[U-Boot] [PATCH 01/16] ARM: remove vpac270 board support

[Masahiro Yamada]

Hi Marek,


2015-07-26 17:49 GMT+09:00 Marek Vasut <marex at denx.de>:
> On Sunday, July 26, 2015 at 10:26:45 AM, Masahiro Yamada wrote:
>> The board-specific linker script board/vpac270/u-boot-spl.lds
>> obstructs further cleanup.  This board has not been converted to
>> Generic Board yet in spite of the long-term warning.  Remove.
>>
>> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
>> ---
>
> Hi!
>
>> Marek,
>>
>> If you want to keep this board:
>>   - Please convert it to Generic Board
>>   - Please use arch-common linker script
>
> Yeah, I want to keep this board, but please give me a bit of time
> to do this. This one and the zipitz2 are the two PXA boards which
> I have and want to keep.
>
> I recall the u-boot-spl.lds was really needed to keep the size of
> the SPL under 1kiB ; without this the board was unbootable when
> using the OneNAND.
>

So, you worked around your problem in a wrong way.

I imagine this linker script means:
We build many objects for SPL, but do not want link them.
To keep the SPL size under 1kiB, only 4 objects should be
linked into the SPL.
The correct way of doing this is to disable CONFIGs for SPL,
by Kconfig or by include/configs/vpac270.h, but it is troubulesome
(or tedious, or impossible at all?), so this weird linker script is needed...

I'd say having per-board linker scripts is a mistake of U-boot.
They should be deprecated in the long-run.


Is it possible to replace

 drivers/mtd/onenand/built-in.o

with

 drivers/built-in.o

Given that any other drivers should be disabled for SPL,
this should work.

And it should be enough for my series.



-- 
Best Regards
Masahiro Yamada