[U-Boot] [PATCH] ARMv7: Add missing boards to MAKEALL

[Albert ARIBAUD <albert.u.boot@aribaud.net>]

On 03/08/2011 16:49, Albert ARIBAUD wrote:
> On 03/08/2011 16:43, Albert ARIBAUD wrote:
>> Hi Dirk,
>>
>> On 06/07/2011 17:35, Dirk Behme wrote:
>>> From: Dirk Behme<dirk.behme@googlemail.com>
>>>
>>> Add all missing boards with CPU==armv7 from boards.cfg
>>> to MAKEALL.
>>>
>>> Signed-off-by: Dirk Behme<dirk.behme@googlemail.com>
>>>
>>> ---
>>> MAKEALL | 12 ++++++++++++
>>> 1 file changed, 12 insertions(+)
>>>
>>> Index: u-boot.git/MAKEALL
>>> ===================================================================
>>> --- u-boot.git.orig/MAKEALL
>>> +++ u-boot.git/MAKEALL
>>> @@ -423,11 +423,18 @@ LIST_ARMV7=" \
>>> am3517_crane \
>>> am3517_evm \
>>> ca9x4_ct_vxp \
>>> + cm_t35 \
>>> devkit8000 \
>>> dig297 \
>>> + efikamx \
>>> + harmony \
>>> igep0020 \
>>> igep0030 \
>>> mx51evk \
>>> + mx53ard \
>>> + mx53evk \
>>> + mx53loco \
>>> + mx53smd \
>>> omap3_beagle \
>>> omap3_overo \
>>> omap3_evm \
>>> @@ -437,8 +444,13 @@ LIST_ARMV7=" \
>>> omap3_zoom2 \
>>> omap4_panda \
>>> omap4_sdp4430 \
>>> + s5pc210_universal \
>>> s5p_goni \
>>> + seaboard \
>>> smdkc100 \
>>> + smdkv310 \
>>> + u8500_href \
>>> + vision2 \
>>> "
>>>
>>> #########################################################################
>>>
>>> _______________________________________________
>>> U-Boot mailing list
>>> U-Boot at lists.denx.de
>>> http://lists.denx.de/mailman/listinfo/u-boot
>>>
>>>
>> Applied to u-boot-arm/master, thanks.
>>
>> This raises the number of boards built / built with errors or warnings
>> from 201 / 97 to 213 / 100 -- which means three of these new boards do
>> not build cleanly: cm_t53 and smdkv310 fail building with errors, and
>> u8500 has warnings.
>>
>> The boards which are still in error at the time of the release will be
>> removed by way of an additional patch above Wolfgang's 'card deck' of 52
>> board removal patches (well actually less than 52 now since several
>> boards have been made to build since).
>
> After Reinhard's remark on boards.cfg vs MAKEALL, I may have accepted
> the patch a bit hastily. All of these boards are already in boards.cfg,
> and need not be in makeall IIUC.
>
> Thus, I am rolling back master one notch and will mark the patch as
> Supersedes (could not find a better match for "not needed any more")

... or not: here, adding this patch should have resulted in 12 more 
boards built and 12 more boards failing to build, but actuelly 9 succeed.

Reinhard, are you sure that boards that are in boards.cfg *and* in 
MAKEALL will fail to build? This is not consistent with what I see here 
with this patch.

Amicalement,
-- 
Albert.