On Tue, Jan 26, 2016 at 04:52:18PM -0500, Nicolas Pitre wrote:
On Tue, 26 Jan 2016, Rob Herring wrote:
quoted
With the latest dtc I've imported to the kernel, there is a new
dependency on strnlen. This is giving the ARM decompressor an error. I
have a couple of ideas on how to fix it:
- Move string related functions in dtc to separate file
- Add lib/string.c to the decompressor build
- Build the decompressor with function-sections to discard the string
related functions and avoid the linker error. Also, there's probably a
bit of other unused code that would get dropped. function-sections
doesn't work for the kernel in general, but may for the decompressor.
Nico may know.
Use of function-sections turns out to be more involved than I wish. And
we've put many special restrictions on the decompressor already. And
linking dtc (only part of it actually) into the decompressor is only for
legacy stuff and I'd prefer we don't spend too much time on legacy
stuff. Yadda yadda.
So instead I'd simply paste a simple implementation of strnlen into
arch/arm/boot/compressed/string.c and be done with it. The one in
lib/string.c looks dead simple for what it's worth.
I think that would be my preferred option too.
quoted
- Don't update dtc and take the rest of the day off.
That is always an option. Does it really need to be updated? ;-)
Nicolas
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160127/52124933/attachment.sig>