On Sep 2, 2011, at 8:41 AM, Timur Tabi wrote:
> Kumar Gala wrote:
>> Both. I'm think for your patch we'd add some general config option for
>> extra print info.
>
> So you want to see this instead:
>
> /*
> * Display whether this is a 32-bit build or a 36-bit build.
> */
> #ifdef CONFIG_DISPLAY_ADDR_SIZE
> #ifdef CONFIG_PHYS_64BIT
> puts("ADDR: 36-bit address map\n");
> #else
> puts("ADDR: 32-bit address map\n");
> #endif
> #endif
>
> I still like my way better. It eliminates the need to think about another
> CONFIG option. I think adding another CONFIG option is worse than adding
> another line of text.
I think we could introduce kernel style "printk" levels that would allow more
control over something like this.
- k
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
