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

Signed-off-by: Christian Gmeiner christian.gmeiner@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

On Tue, Jun 06, 2017 at 01:51:38PM +0200, Christian Gmeiner wrote:
Signed-off-by: Christian Gmeiner christian.gmeiner@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!

Hi Tom
2017-06-06 22:04 GMT+02:00 Tom Rini trini@konsulko.com:
On Tue, Jun 06, 2017 at 01:51:38PM +0200, Christian Gmeiner wrote:
Signed-off-by: Christian Gmeiner christian.gmeiner@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"?
thanks -- Christian Gmeiner, MSc
https://www.youtube.com/user/AloryOFFICIAL https://soundcloud.com/christian-gmeiner

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@konsulko.com:
On Tue, Jun 06, 2017 at 01:51:38PM +0200, Christian Gmeiner wrote:
Signed-off-by: Christian Gmeiner christian.gmeiner@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.

Hi Christian,
On Fri, Jun 9, 2017 at 8:58 AM, Tom Rini trini@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@konsulko.com:
On Tue, Jun 06, 2017 at 01:51:38PM +0200, Christian Gmeiner wrote:
Signed-off-by: Christian Gmeiner christian.gmeiner@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.
Can you respin the patch per Tom's suggestion? Thanks!
Regards, Bin

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@gmail.com:
Hi Christian,
On Fri, Jun 9, 2017 at 8:58 AM, Tom Rini trini@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@konsulko.com:
On Tue, Jun 06, 2017 at 01:51:38PM +0200, Christian Gmeiner wrote:
Signed-off-by: Christian Gmeiner christian.gmeiner@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.

Hi Christian,
On Tue, Sep 4, 2018 at 6:07 PM Christian Gmeiner christian.gmeiner@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@gmail.com:
Hi Christian,
On Fri, Jun 9, 2017 at 8:58 AM, Tom Rini trini@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@konsulko.com:
On Tue, Jun 06, 2017 at 01:51:38PM +0200, Christian Gmeiner wrote:
Signed-off-by: Christian Gmeiner christian.gmeiner@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

On Wed, Sep 05, 2018 at 12:57:39PM +0800, Bin Meng wrote:
Hi Christian,
On Tue, Sep 4, 2018 at 6:07 PM Christian Gmeiner christian.gmeiner@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@gmail.com:
Hi Christian,
On Fri, Jun 9, 2017 at 8:58 AM, Tom Rini trini@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@konsulko.com:
On Tue, Jun 06, 2017 at 01:51:38PM +0200, Christian Gmeiner wrote:
> Signed-off-by: Christian Gmeiner christian.gmeiner@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.
To be extra clear, drop the table from the x86 file and then the default will be used.

Am Mi., 5. Sep. 2018 um 17:10 Uhr schrieb Tom Rini trini@konsulko.com:
On Wed, Sep 05, 2018 at 12:57:39PM +0800, Bin Meng wrote:
Hi Christian,
On Tue, Sep 4, 2018 at 6:07 PM Christian Gmeiner christian.gmeiner@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@gmail.com:
Hi Christian,
On Fri, Jun 9, 2017 at 8:58 AM, Tom Rini trini@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@konsulko.com: > On Tue, Jun 06, 2017 at 01:51:38PM +0200, Christian Gmeiner wrote: > >> Signed-off-by: Christian Gmeiner christian.gmeiner@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.
To be extra clear, drop the table from the x86 file and then the default will be used.
Got it - thanks...
participants (3)
-
Bin Meng
-
Christian Gmeiner
-
Tom Rini