[U-Boot] [PATCH] x86: extend baudrate table with 57600

[Bin Meng]

Hi Christian,

On Tue, Sep 4, 2018 at 6:07 PM Christian Gmeiner
<christian.gmeiner at gmail.com> wrote:
>
> Hi all,
>
> I totally forgot to answer that mail - but as I need this change my
> motivation is back.
>
> Am Do., 22. Juni 2017 um 08:50 Uhr schrieb Bin Meng <bmeng.cn at gmail.com>:
> >
> > Hi Christian,
> >
> > On Fri, Jun 9, 2017 at 8:58 AM, Tom Rini <trini at konsulko.com> wrote:
> > > On Thu, Jun 08, 2017 at 09:38:49AM +0200, Christian Gmeiner wrote:
> > >> Hi Tom
> > >>
> > >> 2017-06-06 22:04 GMT+02:00 Tom Rini <trini at konsulko.com>:
> > >> > On Tue, Jun 06, 2017 at 01:51:38PM +0200, Christian Gmeiner wrote:
> > >> >
> > >> >> Signed-off-by: Christian Gmeiner <christian.gmeiner at gmail.com>
> > >> >> ---
> > >> >>  include/configs/x86-common.h | 2 +-
> > >> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> > >> >>
> > >> >> diff --git a/include/configs/x86-common.h b/include/configs/x86-common.h
> > >> >> index b5ef8b5..9a97714 100644
> > >> >> --- a/include/configs/x86-common.h
> > >> >> +++ b/include/configs/x86-common.h
> > >> >> @@ -56,7 +56,7 @@
> > >> >>   * Serial Configuration
> > >> >>   */
> > >> >>  #define CONFIG_SYS_BAUDRATE_TABLE    {300, 600, 1200, 2400, 4800, \
> > >> >> -                                      9600, 19200, 38400, 115200}
> > >> >> +                                      9600, 19200, 38400, 57600, 115200}
> > >> >>  #define CONFIG_SYS_NS16550_PORT_MAPPED
> > >> >>
> > >> >>  #define CONFIG_CMDLINE_EDITING
> > >> >
> > >> > Can we just use the fall back table instead?  Thanks!
> > >>
> > >> Can you point me to the "fall back table"?
> > >
> > > If one isn't defined the one in <config_fallbacks.h> is used.
> >
>
> The fallback table looks okay and has 57600 as baud rate in it. The problematic
> part is the xf86-common.h which defines CONFIG_SYS_BAUDRATE_TABLE and
> as a result the fallback table plays no role here.
>
> > Can you respin the patch per Tom's suggestion? Thanks!
>
> I think that the current patch is okay as Tom's suggestion does not fly.
>

I believe what Tom asked was to update config_fallbacks.h directly,
and then no need to keep x86-specific one.

Regards,
Bin