[U-Boot] [STATUS] WARNING: huge ARM board removal pending

Igor Grinberg grinberg at compulab.co.il
Wed Jun 8 22:00:53 CEST 2011


Hi Albert,

On 06/07/11 21:21, Albert ARIBAUD wrote:
> Hi Igor,
>
> Le 07/06/2011 07:42, Igor Grinberg a écrit :
>> On 06/06/11 23:17, Albert ARIBAUD wrote:
>>
>>> Hi,
>>>
>>> Le 03/06/2011 15:10, Igor Grinberg a écrit :
>>>> On 06/03/11 00:51, Wolfgang Denk wrote:
>>>>
>>>>> Hi,
>>>>
>>>> Hi Wolfgang,
>>>>
>>>> [...]
>>>>
>>>>> omap1610h2
>>>>> omap1610inn
>>>>
>>>>> Please submit fixes to prevent removal.
>>>>
>>>> I've sent a compile fix [1] for omap1610h2 and omap1610inn about a month ago,
>>>> but still it hasn't been picked up.
>>>>
>>>> Sandeep said, he is on vacation, may be Albert should pick this up instead?
>>>>
>>>> [1] http://patchwork.ozlabs.org/patch/93545/
>>>
>>> What exactly do the FIXMEs mean? Are the definitions of CONFIG_SYS_INIT_SP_ADDR somehow incorrect in this patch, or just non-optimal?
>>
>> This means, that I could not test those, because I don't have the hardware.
>> I've taken those from the same SoC, so they should be fine, but again could not test,
>> so I've added a comment.
>
> FIXME implies the writer know something was wrong and mention it. Here, it is less of a FIXME and more of a TODO: test <whatever>; and I don't want readers to think there is a known error where there is not. Can you resubmit with the FIXMEs turned into TODOs?

I can see Aneesh has posted several fixes of that kind (also for these two boards),
so may be his values are better justified and don't need either TODO or FIXME.
If it is so, then my patch can be dropped.


-- 
Regards,
Igor.



More information about the U-Boot mailing list