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

[Christian Gmeiner]

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.

-- 
greets
--
Christian Gmeiner, MSc

https://christian-gmeiner.info