* Daniel Thompson <daniel.thomp...@linaro.org> [140916 16:58]: > The omap1's debug-macro.S is similar to the generic 8250 code. Compared to > the 8520 code the omap1 macro automatically determines what UART to use > based on breadcrumbs left by the bootloader and automatically copes with > the eccentric register layout on OMAP7XX. > > This patch drops both these features and relies instead on the generic > 8250 macros: > > 1. Dropping support for the bootloader breadcrumbs is identical to the > way the migration was handled for OMAP2 (see 808b7e07464d...). > > 2. Support for OMAP7XX still exists but it must be configured by hand > (DEBUG_OMAP7XXUART1/2/3) rather than handled at runtime.
I assume you'll be queueing this along with other Kconfig.debug changes, so: Acked-by: Tony Lindgren <t...@atomide.com> > Signed-off-by: Daniel Thompson <daniel.thomp...@linaro.org> > Cc: Russell King <li...@arm.linux.org.uk> > Cc: linux-arm-ker...@lists.infradead.org > Cc: Tony Lindgren <t...@atomide.com> > Cc: Arnd Bergmann <arnd.bergm...@linaro.org> > Cc: linux-o...@vger.kernel.org > Tested-by: Aaro Koskinen <aaro.koski...@iki.fi> > --- > arch/arm/Kconfig.debug | 57 +++++++++++++- > arch/arm/mach-omap1/include/mach/debug-macro.S | 101 > ------------------------- > 2 files changed, 56 insertions(+), 102 deletions(-) > delete mode 100644 arch/arm/mach-omap1/include/mach/debug-macro.S > > diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug > index 97c3058..6540036 100644 > --- a/arch/arm/Kconfig.debug > +++ b/arch/arm/Kconfig.debug > @@ -471,6 +471,30 @@ choice > Say Y here if you want kernel low-level debugging support > on TI-NSPIRE CX models. > > + config DEBUG_OMAP1UART1 > + bool "Kernel low-level debugging via OMAP1 UART1" > + depends on ARCH_OMAP1 > + select DEBUG_UART_8250 > + help > + Say Y here if you want kernel low-level debugging support > + on OMAP1 based platforms (except OMAP730) on the UART1. > + > + config DEBUG_OMAP1UART2 > + bool "Kernel low-level debugging via OMAP1 UART2" > + depends on ARCH_OMAP1 > + select DEBUG_UART_8250 > + help > + Say Y here if you want kernel low-level debugging support > + on OMAP1 based platforms (except OMAP730) on the UART2. > + > + config DEBUG_OMAP1UART3 > + bool "Kernel low-level debugging via OMAP1 UART3" > + depends on ARCH_OMAP1 > + select DEBUG_UART_8250 > + help > + Say Y here if you want kernel low-level debugging support > + on OMAP1 based platforms (except OMAP730) on the UART3. > + > config DEBUG_OMAP2UART1 > bool "OMAP2/3/4 UART1 (omap2/3 sdp boards and some omap3 > boards)" > depends on ARCH_OMAP2PLUS > @@ -513,6 +537,30 @@ choice > depends on ARCH_OMAP2PLUS > select DEBUG_OMAP2PLUS_UART > > + config DEBUG_OMAP7XXUART1 > + bool "Kernel low-level debugging via OMAP730 UART1" > + depends on ARCH_OMAP730 > + select DEBUG_UART_8250 > + help > + Say Y here if you want kernel low-level debugging support > + on OMAP730 based platforms on the UART1. > + > + config DEBUG_OMAP7XXUART2 > + bool "Kernel low-level debugging via OMAP730 UART2" > + depends on ARCH_OMAP730 > + select DEBUG_UART_8250 > + help > + Say Y here if you want kernel low-level debugging support > + on OMAP730 based platforms on the UART2. > + > + config DEBUG_OMAP7XXUART3 > + bool "Kernel low-level debugging via OMAP730 UART3" > + depends on ARCH_OMAP730 > + select DEBUG_UART_8250 > + help > + Say Y here if you want kernel low-level debugging support > + on OMAP730 based platforms on the UART3. > + > config DEBUG_TI81XXUART1 > bool "Kernel low-level debugging messages via TI81XX UART1 > (ti8148evm)" > depends on ARCH_OMAP2PLUS > @@ -1120,6 +1168,9 @@ config DEBUG_UART_PHYS > default 0xff690000 if DEBUG_RK32_UART2 > default 0xffc02000 if DEBUG_SOCFPGA_UART > default 0xffd82340 if ARCH_IOP13XX > + default 0xfffb0000 if DEBUG_OMAP1UART1 || DEBUG_OMAP7XXUART1 > + default 0xfffb0800 if DEBUG_OMAP1UART2 || DEBUG_OMAP7XXUART2 > + default 0xfffb9800 if DEBUG_OMAP1UART3 || DEBUG_OMAP7XXUART3 > default 0xfff36000 if DEBUG_HIGHBANK_UART > default 0xfffff700 if ARCH_IOP33X > depends on DEBUG_LL_UART_8250 || DEBUG_LL_UART_PL01X || \ > @@ -1188,6 +1239,9 @@ config DEBUG_UART_VIRT > default 0xfef00000 if ARCH_IXP4XX && !CPU_BIG_ENDIAN > default 0xfef00003 if ARCH_IXP4XX && CPU_BIG_ENDIAN > default 0xfef36000 if DEBUG_HIGHBANK_UART > + default 0xfefb0000 if DEBUG_OMAP1UART1 || DEBUG_OMAP7XXUART1 > + default 0xfefb0800 if DEBUG_OMAP1UART2 || DEBUG_OMAP7XXUART2 > + default 0xfefb9800 if DEBUG_OMAP1UART3 || DEBUG_OMAP7XXUART3 > default 0xfefff700 if ARCH_IOP33X > default 0xff003000 if DEBUG_U300_UART > default DEBUG_UART_PHYS if !MMU > @@ -1198,7 +1252,8 @@ config DEBUG_UART_VIRT > config DEBUG_UART_8250_SHIFT > int "Register offset shift for the 8250 debug UART" > depends on DEBUG_LL_UART_8250 || DEBUG_UART_8250 > - default 0 if FOOTBRIDGE || ARCH_IOP32X > + default 0 if FOOTBRIDGE || ARCH_IOP32X || \ > + DEBUG_OMAP7XXUART1 || DEBUG_OMAP7XXUART2 || DEBUG_OMAP7XXUART3 > default 2 > > config DEBUG_UART_8250_WORD > diff --git a/arch/arm/mach-omap1/include/mach/debug-macro.S > b/arch/arm/mach-omap1/include/mach/debug-macro.S > deleted file mode 100644 > index 5c1a26c..0000000 > --- a/arch/arm/mach-omap1/include/mach/debug-macro.S > +++ /dev/null > @@ -1,101 +0,0 @@ > -/* arch/arm/mach-omap1/include/mach/debug-macro.S > - * > - * Debugging macro include header > - * > - * Copyright (C) 1994-1999 Russell King > - * Moved from linux/arch/arm/kernel/debug.S by Ben Dooks > - * > - * This program is free software; you can redistribute it and/or modify > - * it under the terms of the GNU General Public License version 2 as > - * published by the Free Software Foundation. > - * > -*/ > - > -#include <linux/serial_reg.h> > - > -#include "serial.h" > - > - .pushsection .data > -omap_uart_phys: .word 0x0 > -omap_uart_virt: .word 0x0 > - .popsection > - > - /* > - * Note that this code won't work if the bootloader passes > - * a wrong machine ID number in r1. To debug, just hardcode > - * the desired UART phys and virt addresses temporarily into > - * the omap_uart_phys and omap_uart_virt above. > - */ > - .macro addruart, rp, rv, tmp > - > - /* Use omap_uart_phys/virt if already configured */ > -9: adr \rp, 99f @ get effective addr of 99f > - ldr \rv, [\rp] @ get absolute addr of 99f > - sub \rv, \rv, \rp @ offset between the two > - ldr \rp, [\rp, #4] @ abs addr of omap_uart_phys > - sub \tmp, \rp, \rv @ make it effective > - ldr \rp, [\tmp, #0] @ omap_uart_phys > - ldr \rv, [\tmp, #4] @ omap_uart_virt > - cmp \rp, #0 @ is port configured? > - cmpne \rv, #0 > - bne 100f @ already configured > - > - /* Check the debug UART configuration set in uncompress.h */ > - and \rp, pc, #0xff000000 > - ldr \rv, =OMAP_UART_INFO_OFS > - ldr \rp, [\rp, \rv] > - > - /* Select the UART to use based on the UART1 scratchpad value */ > -10: cmp \rp, #0 @ no port configured? > - beq 11f @ if none, try to use UART1 > - cmp \rp, #OMAP1UART1 > - beq 11f @ configure OMAP1UART1 > - cmp \rp, #OMAP1UART2 > - beq 12f @ configure OMAP1UART2 > - cmp \rp, #OMAP1UART3 > - beq 13f @ configure OMAP2UART3 > - > - /* Configure the UART offset from the phys/virt base */ > -11: mov \rp, #0x00fb0000 @ OMAP1UART1 > - b 98f > -12: mov \rp, #0x00fb0000 @ OMAP1UART1 > - orr \rp, \rp, #0x00000800 @ OMAP1UART2 > - b 98f > -13: mov \rp, #0x00fb0000 @ OMAP1UART1 > - orr \rp, \rp, #0x00000800 @ OMAP1UART2 > - orr \rp, \rp, #0x00009000 @ OMAP1UART3 > - > - /* Store both phys and virt address for the uart */ > -98: add \rp, \rp, #0xff000000 @ phys base > - str \rp, [\tmp, #0] @ omap_uart_phys > - sub \rp, \rp, #0xff000000 @ phys base > - add \rp, \rp, #0xfe000000 @ virt base > - str \rp, [\tmp, #4] @ omap_uart_virt > - b 9b > - > - .align > -99: .word . > - .word omap_uart_phys > - .ltorg > - > -100: > - .endm > - > - .macro senduart,rd,rx > - strb \rd, [\rx] > - .endm > - > - .macro busyuart,rd,rx > -1001: ldrb \rd, [\rx, #(UART_LSR << OMAP_PORT_SHIFT)] > - and \rd, \rd, #(UART_LSR_TEMT | UART_LSR_THRE) > - teq \rd, #(UART_LSR_TEMT | UART_LSR_THRE) > - beq 1002f > - ldrb \rd, [\rx, #(UART_LSR << OMAP7XX_PORT_SHIFT)] > - and \rd, \rd, #(UART_LSR_TEMT | UART_LSR_THRE) > - teq \rd, #(UART_LSR_TEMT | UART_LSR_THRE) > - bne 1001b > -1002: > - .endm > - > - .macro waituart,rd,rx > - .endm > -- > 1.9.3 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
