[U-Boot] [PATCH v2 1/2] CONFIG_SYS_BAUDRATE_TABLE: Add <config_fallbacks.h>, place there

Tom Rini trini at ti.com
Fri Apr 27 16:53:34 CEST 2012


On 04/26/2012 11:31 PM, Wolfgang Denk wrote:
> Dear Tom Rini,
>
> In message<1335480396-29478-1-git-send-email-trini at ti.com>  you wrote:
>> We provide a default table of { 9600, 19200, 38400, 57600, 115200 }
>> in<config_fallbacks.h>  which mkconfig places after<configs/...h>  in
>> the generated config file.  This is used when a board has not set its
>> own table.  A number of boards set values below 9600 but it is my belief
>> that this was done for completeness rather than as a requirement so they
>> have opted to the new default table.
>
> Argh... But you cannot change this without explicit ACK from the
> respective board maintainers.

Then we're really SOL here.  Even if we limit it to "I dropped 
300/600/900/1200/2400/4800 baud rate support, is that ok".

> Technically, I don't like the name "config_fallbacks.h"; how about
> "config_defaults.h" instead?

Sure.

> Finally, your patch does not apply (I guess it is an incremental patch
> over obsolete code?).

That's odd actually, it was on v2012.04'ish and a pull and rebase -q 
went quietly.  It was build-tested on MAKEALL -a powerpc/arm/x86.

-- 
Tom


More information about the U-Boot mailing list