Stefan Roese wrote:
> Hi Dirk,
> 
> I forgot to address your first issue. Please see below:
> 
> <snip>
> 
>>> +#if !defined(CONFIG_SYS_64BIT_VSPRINTF)
>>> +#warning Please define CONFIG_SYS_64BIT_VSPRINTF for correct output!
>>> +#endif
>>> +
>> 1. This preprocessor warning is annoying. Why not enable it in all
>> configs where needed and then remove this warning here?
> 
> I could have done that, but this would not have been so easy. Especially with 
> so many new board ports pending in the u-boot-arch/next branches at that 
> time. 
> And upcoming board ports would probably fallen through the crack as well. So 
> I 
> decided to do it this way.

Is this a long term solution or only waiting for all u-boot-arch/next 
branches to be merged?

For a long term solution this would be annoying. Maybe we can change 
this after merge window closed?

Thanks and best regards

Dirk

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to