
22 Jun
2009
22 Jun
'09
8:12 p.m.
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
- 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