[openwrt/openwrt] packages/uboot-omap: bugfix: serial using ns16550 fifo not enabled

LEDE Commits lede-commits at lists.infradead.org
Tue Apr 10 18:44:10 PDT 2018


lynxis pushed a commit to openwrt/openwrt.git, branch master:
https://git.lede-project.org/f6e6eadc99c6274207f8f2ebc739063549959a1f

commit f6e6eadc99c6274207f8f2ebc739063549959a1f
Author: Alexander Couzens <lynxis at fe80.eu>
AuthorDate: Wed Apr 11 03:17:44 2018 +0200

    packages/uboot-omap: bugfix: serial using ns16550 fifo not enabled
    
    The serial was working before, but not when doing copy&pasting longer
    commands in a short time.
    
    Fixes: a4def18f29e1 ("uboot-omap: Update to u-boot v2017.01")
    Signed-off-by: Alexander Couzens <lynxis at fe80.eu>
---
 package/boot/uboot-omap/Makefile                   |   2 +-
 ...l-ns16550-bugfix-ns16550-fifo-not-enabled.patch | 309 +++++++++++++++++++++
 2 files changed, 310 insertions(+), 1 deletion(-)

diff --git a/package/boot/uboot-omap/Makefile b/package/boot/uboot-omap/Makefile
index 9084f37..0d5260d 100644
--- a/package/boot/uboot-omap/Makefile
+++ b/package/boot/uboot-omap/Makefile
@@ -9,7 +9,7 @@ include $(TOPDIR)/rules.mk
 include $(INCLUDE_DIR)/kernel.mk
 
 PKG_VERSION:=2017.01
-PKG_RELEASE:=1
+PKG_RELEASE:=2
 
 PKG_HASH:=6c425175f93a4bcf2ec9faf5658ef279633dbd7856a293d95bd1ff516528ecf2
 
diff --git a/package/boot/uboot-omap/patches/105-serial-ns16550-bugfix-ns16550-fifo-not-enabled.patch b/package/boot/uboot-omap/patches/105-serial-ns16550-bugfix-ns16550-fifo-not-enabled.patch
new file mode 100644
index 0000000..513efaf
--- /dev/null
+++ b/package/boot/uboot-omap/patches/105-serial-ns16550-bugfix-ns16550-fifo-not-enabled.patch
@@ -0,0 +1,309 @@
+From 17fa032671f7981628fe16b30399638842a4b1bb Mon Sep 17 00:00:00 2001
+From: Heiko Schocher <hs at denx.de>
+Date: Wed, 18 Jan 2017 08:05:49 +0100
+Subject: [PATCH] serial, ns16550: bugfix: ns16550 fifo not enabled
+
+commit: 65f83802b7a5b "serial: 16550: Add getfcr accessor"
+breaks u-boot commandline working with long commands
+sending to the board.
+
+Since the above patch, you have to setup the fcr register.
+
+For board/archs which enable OF_PLATDATA, the new field
+fcr in struct ns16550_platdata is not filled with a
+default value ...
+
+This leads in not setting up the uarts fifo, which ends
+in problems, when you send long commands to u-boots
+commandline.
+
+Detected this issue with automated tbot tests on am335x
+based shc board.
+
+The error does not popup, if you type commands. You need
+to copy&paste a long command to u-boots commandshell
+(or send a long command with tbot)
+
+Possible boards/plattforms with problems:
+./arch/arm/cpu/arm926ejs/lpc32xx/devices.c
+./arch/arm/mach-tegra/board.c
+./board/overo/overo.c
+./board/quipos/cairo/cairo.c
+./board/logicpd/omap3som/omap3logic.c
+./board/logicpd/zoom1/zoom1.c
+./board/timll/devkit8000/devkit8000.c
+./board/lg/sniper/sniper.c
+./board/ti/beagle/beagle.c
+./drivers/serial/serial_rockchip.c
+
+Signed-off-by: Heiko Schocher <hs at denx.de>
+Signed-off-by: Ladislav Michl <ladis at linux-mips.org>
+Tested-by: Adam Ford <aford173 at gmail.com>
+Reviewed-by: Tom Rini <trini at konsulko.com>
+---
+ arch/arm/cpu/arm926ejs/lpc32xx/devices.c | 12 ++++++++----
+ arch/arm/mach-omap2/am33xx/board.c       | 18 ++++++++++++------
+ arch/arm/mach-tegra/board.c              |  1 +
+ board/isee/igep00x0/igep00x0.c           |  3 ++-
+ board/lg/sniper/sniper.c                 |  3 ++-
+ board/logicpd/omap3som/omap3logic.c      |  3 ++-
+ board/logicpd/zoom1/zoom1.c              |  3 ++-
+ board/overo/overo.c                      |  3 ++-
+ board/quipos/cairo/cairo.c               |  3 ++-
+ board/ti/beagle/beagle.c                 |  3 ++-
+ board/timll/devkit8000/devkit8000.c      |  3 ++-
+ drivers/serial/ns16550.c                 |  9 +++------
+ drivers/serial/serial_rockchip.c         |  1 +
+ include/ns16550.h                        |  5 +++++
+ 14 files changed, 46 insertions(+), 24 deletions(-)
+
+diff --git a/arch/arm/cpu/arm926ejs/lpc32xx/devices.c b/arch/arm/cpu/arm926ejs/lpc32xx/devices.c
+index 399b07c5420a..f744398ca7ad 100644
+--- a/arch/arm/cpu/arm926ejs/lpc32xx/devices.c
++++ b/arch/arm/cpu/arm926ejs/lpc32xx/devices.c
+@@ -45,10 +45,14 @@ void lpc32xx_uart_init(unsigned int uart_id)
+ 
+ #if !CONFIG_IS_ENABLED(OF_CONTROL)
+ static const struct ns16550_platdata lpc32xx_uart[] = {
+-	{ .base = UART3_BASE, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
+-	{ .base = UART4_BASE, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
+-	{ .base = UART5_BASE, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
+-	{ .base = UART6_BASE, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
++	{ .base = UART3_BASE, .reg_shift = 2,
++	  .clock = CONFIG_SYS_NS16550_CLK, .fcr = UART_FCR_DEFVAL, },
++	{ .base = UART4_BASE, .reg_shift = 2,
++	  .clock = CONFIG_SYS_NS16550_CLK, .fcr = UART_FCR_DEFVAL, },
++	{ .base = UART5_BASE, .reg_shift = 2,
++	  .clock = CONFIG_SYS_NS16550_CLK, .fcr = UART_FCR_DEFVAL, },
++	{ .base = UART6_BASE, .reg_shift = 2,
++	  .clock = CONFIG_SYS_NS16550_CLK, .fcr = UART_FCR_DEFVAL, },
+ };
+ 
+ #if defined(CONFIG_LPC32XX_HSUART)
+diff --git a/arch/arm/mach-omap2/am33xx/board.c b/arch/arm/mach-omap2/am33xx/board.c
+index 73824df18fa7..190310fd0079 100644
+--- a/arch/arm/mach-omap2/am33xx/board.c
++++ b/arch/arm/mach-omap2/am33xx/board.c
+@@ -40,14 +40,20 @@ DECLARE_GLOBAL_DATA_PTR;
+ 
+ #if !CONFIG_IS_ENABLED(OF_CONTROL)
+ static const struct ns16550_platdata am33xx_serial[] = {
+-	{ .base = CONFIG_SYS_NS16550_COM1, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
++	{ .base = CONFIG_SYS_NS16550_COM1, .reg_shift = 2,
++	  .clock = CONFIG_SYS_NS16550_CLK, .fcr = UART_FCR_DEFVAL, },
+ # ifdef CONFIG_SYS_NS16550_COM2
+-	{ .base = CONFIG_SYS_NS16550_COM2, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
++	{ .base = CONFIG_SYS_NS16550_COM2, .reg_shift = 2,
++	  .clock = CONFIG_SYS_NS16550_CLK, .fcr = UART_FCR_DEFVAL, },
+ #  ifdef CONFIG_SYS_NS16550_COM3
+-	{ .base = CONFIG_SYS_NS16550_COM3, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
+-	{ .base = CONFIG_SYS_NS16550_COM4, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
+-	{ .base = CONFIG_SYS_NS16550_COM5, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
+-	{ .base = CONFIG_SYS_NS16550_COM6, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
++	{ .base = CONFIG_SYS_NS16550_COM3, .reg_shift = 2,
++	  .clock = CONFIG_SYS_NS16550_CLK, .fcr = UART_FCR_DEFVAL, },
++	{ .base = CONFIG_SYS_NS16550_COM4, .reg_shift = 2,
++	  .clock = CONFIG_SYS_NS16550_CLK, .fcr = UART_FCR_DEFVAL, },
++	{ .base = CONFIG_SYS_NS16550_COM5, .reg_shift = 2,
++	  .clock = CONFIG_SYS_NS16550_CLK, .fcr = UART_FCR_DEFVAL, },
++	{ .base = CONFIG_SYS_NS16550_COM6, .reg_shift = 2,
++	  .clock = CONFIG_SYS_NS16550_CLK, .fcr = UART_FCR_DEFVAL, },
+ #  endif
+ # endif
+ };
+diff --git a/arch/arm/mach-tegra/board.c b/arch/arm/mach-tegra/board.c
+index 3d1d26d13d13..b3a041b539af 100644
+--- a/arch/arm/mach-tegra/board.c
++++ b/arch/arm/mach-tegra/board.c
+@@ -219,6 +219,7 @@ static struct ns16550_platdata ns16550_com1_pdata = {
+ 	.base = CONFIG_SYS_NS16550_COM1,
+ 	.reg_shift = 2,
+ 	.clock = CONFIG_SYS_NS16550_CLK,
++	.fcr = UART_FCR_DEFVAL,
+ };
+ 
+ U_BOOT_DEVICE(ns16550_com1) = {
+diff --git a/board/isee/igep00x0/igep00x0.c b/board/isee/igep00x0/igep00x0.c
+index ae7959b1eb6e..5a3498f570a6 100644
+--- a/board/isee/igep00x0/igep00x0.c
++++ b/board/isee/igep00x0/igep00x0.c
+@@ -32,7 +32,8 @@ DECLARE_GLOBAL_DATA_PTR;
+ static const struct ns16550_platdata igep_serial = {
+ 	.base = OMAP34XX_UART3,
+ 	.reg_shift = 2,
+-	.clock = V_NS16550_CLK
++	.clock = V_NS16550_CLK,
++	.fcr = UART_FCR_DEFVAL,
+ };
+ 
+ U_BOOT_DEVICE(igep_uart) = {
+diff --git a/board/lg/sniper/sniper.c b/board/lg/sniper/sniper.c
+index 0662449c3875..b2b8f8861f11 100644
+--- a/board/lg/sniper/sniper.c
++++ b/board/lg/sniper/sniper.c
+@@ -31,7 +31,8 @@ const omap3_sysinfo sysinfo = {
+ static const struct ns16550_platdata serial_omap_platdata = {
+ 	.base = OMAP34XX_UART3,
+ 	.reg_shift = 2,
+-	.clock = V_NS16550_CLK
++	.clock = V_NS16550_CLK,
++	.fcr = UART_FCR_DEFVAL,
+ };
+ 
+ U_BOOT_DEVICE(sniper_serial) = {
+diff --git a/board/logicpd/omap3som/omap3logic.c b/board/logicpd/omap3som/omap3logic.c
+index 21b3fdcf49cf..b2fcc28f8b4b 100644
+--- a/board/logicpd/omap3som/omap3logic.c
++++ b/board/logicpd/omap3som/omap3logic.c
+@@ -49,7 +49,8 @@ DECLARE_GLOBAL_DATA_PTR;
+ static const struct ns16550_platdata omap3logic_serial = {
+ 	.base = OMAP34XX_UART1,
+ 	.reg_shift = 2,
+-	.clock = V_NS16550_CLK
++	.clock = V_NS16550_CLK,
++	.fcr = UART_FCR_DEFVAL,
+ };
+ 
+ U_BOOT_DEVICE(omap3logic_uart) = {
+diff --git a/board/logicpd/zoom1/zoom1.c b/board/logicpd/zoom1/zoom1.c
+index 2821ee22674f..0fad23af62f6 100644
+--- a/board/logicpd/zoom1/zoom1.c
++++ b/board/logicpd/zoom1/zoom1.c
+@@ -47,7 +47,8 @@ static const u32 gpmc_lab_enet[] = {
+ static const struct ns16550_platdata zoom1_serial = {
+ 	.base = OMAP34XX_UART3,
+ 	.reg_shift = 2,
+-	.clock = V_NS16550_CLK
++	.clock = V_NS16550_CLK,
++	.fcr = UART_FCR_DEFVAL,
+ };
+ 
+ U_BOOT_DEVICE(zoom1_uart) = {
+diff --git a/board/overo/overo.c b/board/overo/overo.c
+index 40f13e5876cc..5e447262bcfd 100644
+--- a/board/overo/overo.c
++++ b/board/overo/overo.c
+@@ -70,7 +70,8 @@ static struct {
+ static const struct ns16550_platdata overo_serial = {
+ 	.base = OMAP34XX_UART3,
+ 	.reg_shift = 2,
+-	.clock = V_NS16550_CLK
++	.clock = V_NS16550_CLK,
++	.fcr = UART_FCR_DEFVAL,
+ };
+ 
+ U_BOOT_DEVICE(overo_uart) = {
+diff --git a/board/quipos/cairo/cairo.c b/board/quipos/cairo/cairo.c
+index 77e4482906f0..793aa9023150 100644
+--- a/board/quipos/cairo/cairo.c
++++ b/board/quipos/cairo/cairo.c
+@@ -93,7 +93,8 @@ void get_board_mem_timings(struct board_sdrc_timings *timings)
+ static const struct ns16550_platdata cairo_serial = {
+ 	.base = OMAP34XX_UART2,
+ 	.reg_shift = 2,
+-	.clock = V_NS16550_CLK
++	.clock = V_NS16550_CLK,
++	.fcr = UART_FCR_DEFVAL,
+ };
+ 
+ U_BOOT_DEVICE(cairo_uart) = {
+diff --git a/board/ti/beagle/beagle.c b/board/ti/beagle/beagle.c
+index cfdab3e34253..23c79333a223 100644
+--- a/board/ti/beagle/beagle.c
++++ b/board/ti/beagle/beagle.c
+@@ -75,7 +75,8 @@ static struct {
+ static const struct ns16550_platdata beagle_serial = {
+ 	.base = OMAP34XX_UART3,
+ 	.reg_shift = 2,
+-	.clock = V_NS16550_CLK
++	.clock = V_NS16550_CLK,
++	.fcr = UART_FCR_DEFVAL,
+ };
+ 
+ U_BOOT_DEVICE(beagle_uart) = {
+diff --git a/board/timll/devkit8000/devkit8000.c b/board/timll/devkit8000/devkit8000.c
+index f785dbe6d732..b2f060b2ddbf 100644
+--- a/board/timll/devkit8000/devkit8000.c
++++ b/board/timll/devkit8000/devkit8000.c
+@@ -48,7 +48,8 @@ static u32 gpmc_net_config[GPMC_MAX_REG] = {
+ static const struct ns16550_platdata devkit8000_serial = {
+ 	.base = OMAP34XX_UART3,
+ 	.reg_shift = 2,
+-	.clock = V_NS16550_CLK
++	.clock = V_NS16550_CLK,
++	.fcr = UART_FCR_DEFVAL,
+ };
+ 
+ U_BOOT_DEVICE(devkit8000_uart) = {
+diff --git a/drivers/serial/ns16550.c b/drivers/serial/ns16550.c
+index 9b423a591d8a..2df4a1f04fe5 100644
+--- a/drivers/serial/ns16550.c
++++ b/drivers/serial/ns16550.c
+@@ -20,9 +20,6 @@ DECLARE_GLOBAL_DATA_PTR;
+ #define UART_LCRVAL UART_LCR_8N1		/* 8 data, 1 stop, no parity */
+ #define UART_MCRVAL (UART_MCR_DTR | \
+ 		     UART_MCR_RTS)		/* RTS/DTR */
+-#define UART_FCRVAL (UART_FCR_FIFO_EN |	\
+-		     UART_FCR_RXSR |	\
+-		     UART_FCR_TXSR)		/* Clear & enable FIFOs */
+ 
+ #ifndef CONFIG_DM_SERIAL
+ #ifdef CONFIG_SYS_NS16550_PORT_MAPPED
+@@ -138,7 +135,7 @@ static u32 ns16550_getfcr(NS16550_t port)
+ #else
+ static u32 ns16550_getfcr(NS16550_t port)
+ {
+-	return UART_FCRVAL;
++	return UART_FCR_DEFVAL;
+ }
+ #endif
+ 
+@@ -275,7 +272,7 @@ static inline void _debug_uart_init(void)
+ 					    CONFIG_BAUDRATE);
+ 	serial_dout(&com_port->ier, CONFIG_SYS_NS16550_IER);
+ 	serial_dout(&com_port->mcr, UART_MCRVAL);
+-	serial_dout(&com_port->fcr, UART_FCRVAL);
++	serial_dout(&com_port->fcr, UART_FCR_DEFVAL);
+ 
+ 	serial_dout(&com_port->lcr, UART_LCR_BKSE | UART_LCRVAL);
+ 	serial_dout(&com_port->dll, baud_divisor & 0xff);
+@@ -440,7 +437,7 @@ int ns16550_serial_ofdata_to_platdata(struct udevice *dev)
+ 		return -EINVAL;
+ 	}
+ 
+-	plat->fcr = UART_FCRVAL;
++	plat->fcr = UART_FCR_DEFVAL;
+ 	if (port_type == PORT_JZ4780)
+ 		plat->fcr |= UART_FCR_UME;
+ 
+diff --git a/drivers/serial/serial_rockchip.c b/drivers/serial/serial_rockchip.c
+index 6bac95a414ce..c06afc58f7ea 100644
+--- a/drivers/serial/serial_rockchip.c
++++ b/drivers/serial/serial_rockchip.c
+@@ -27,6 +27,7 @@ static int rockchip_serial_probe(struct udevice *dev)
+ 	plat->plat.base = plat->dtplat.reg[0];
+ 	plat->plat.reg_shift = plat->dtplat.reg_shift;
+ 	plat->plat.clock = plat->dtplat.clock_frequency;
++	plat->plat.fcr = UART_FCR_DEFVAL;
+ 	dev->platdata = &plat->plat;
+ 
+ 	return ns16550_serial_probe(dev);
+diff --git a/include/ns16550.h b/include/ns16550.h
+index 7c9703683109..5fcbcd2e74e3 100644
+--- a/include/ns16550.h
++++ b/include/ns16550.h
+@@ -121,6 +121,11 @@ typedef struct NS16550 *NS16550_t;
+ /* Ingenic JZ47xx specific UART-enable bit. */
+ #define UART_FCR_UME		0x10
+ 
++/* Clear & enable FIFOs */
++#define UART_FCR_DEFVAL (UART_FCR_FIFO_EN | \
++			UART_FCR_RXSR |	\
++			UART_FCR_TXSR)
++
+ /*
+  * These are the definitions for the Modem Control Register
+  */
+-- 
+2.17.0
+



More information about the lede-commits mailing list