Fwd: [robh:dt/next 2/2] fdt_ro.c:undefined reference to `strnlen'

[Nicolas Pitre]

On Tue, 26 Jan 2016, Rob Herring wrote:

> 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.

> - 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