On Wed, Aug 30, 2017 at 6:07 AM, Thomas Petazzoni thomas.petazzoni@free-electrons.com wrote:
Hello,
On Tue, 29 Aug 2017 17:18:24 -0400, Tom Rini wrote:
So there's something extra funny going on in the linking. See https://patchwork.ozlabs.org/patch/806263/ which I have included, but haven't been able to push out.
At this point, I don't think it's the correct fix. I don't see why it should be the responsibility of the linker script to include env/embedded.o. Almost no other architecture has env/embedded.o referenced in its linker script. Only a few m68k platforms, with custom linker script in their include/configs/.h file include env/embedded.o in their linker script, plus one Freescale platform.
Again, changing extra- to obj- in env/Makefile also fixes the issue.
Yep, looks to me obj- is the correct way to fix.
Regards, Bin