* [PATCH v11 0/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X)
@ 2014-09-16 23:57 Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 1/9] arm: versatile: Enable DEBUG_LL_UART_PL01X Daniel Thompson
` (8 more replies)
0 siblings, 9 replies; 11+ messages in thread
From: Daniel Thompson @ 2014-09-16 23:57 UTC (permalink / raw)
To: linux-arm-kernel
This patchset removes some single-platform compatibility tricks related
to DEBUG_LL and, as a result, allows multi_v7_defconfig derived builds
to enable DEBUG_LL. Currently the user selected kbuild setting is
ignored and the PL01X's DEBUG_LL stub is silently selected instead. This
is a pain if your hardware doesn't have this cell, not least because it
takes a little time to figure out that kbuild built the wrong code.
Warning:
silentoldconfig on PL011 single platforms will result in DEBUG_LL
being directed at ICEDCC rather than PL011. This should only
affect silentoldconfig since oldconfig prompts and the defconfigs have
been updated to avoid this problem.
Changes since v10:
- Fixed typos in the omap1 config help texts (review of Aaro Koskinen).
Changes since v9:
- Removed a spuriously introduced (and unused) config variable from
the netx patch (review of Paul Bolle).
- Fix netx macro to ensure \rv is correctly initialized.
- Rebased on v3.17rc4.
Changes since v8:
- Regenerated patch series with --find-renames to properly track code
movement.
Changes since v7:
- Killed DEBUG_LL_UART_NONE completely and migrated over the four remaining
platforms that relied on this (Arnd Bergmann, Russell King). This
fixes some problems with oldconfig builds.
- Rebased on latest mainline (v3.17rc1).
Changes since v6:
- Removed a patch to limit DEBUG_LL_UART_NONE to platforms that support
it because it almost time to drop DEBUG_LL_UART_NONE anyway (only
four platforms still use it).
- Rebased on latest mainline (v3.16rc3).
Changes since v5:
- Shortened the list of platforms that can select DEBUG_LL_UART_NONE
(changes Arnd Bergmann)
- Rebased on latest mainline (v3.15rc7).
Changes since v4:
- Rebased to latest mainline (and tested again). No functional changes.
Changes since v3:
- Converted from a single patch to a series.
- Tested defconfig builds of all impacted platforms.
Changes since v2:
- Switch from def_bool to bool (thanks Russell King)
Changes since v1:
- Remove pointless single platform support (thanks Arnd Bergmann)
Daniel Thompson (9):
arm: versatile: Enable DEBUG_LL_UART_PL01X
arm: ep93xx: Enable DEBUG_LL_UART_PL01X
arm: Remove DEBUG_LL_UART_NONE
arm: ks8695: Migrate debug_ll macros to shared directory
arm: omap1: Migrate debug_ll macros to use 8250.S
arm: netx: Migrate DEBUG_LL macros to shared directory
arm: sa1100: Migrate DEBUG_LL macros to shared directory
arm: Seperate DEBUG_UART_PHYS from DEBUG_LL on EP93XX
arm: Fix DEBUG_LL for multi-platform kernels (without PL01X)
arch/arm/Kconfig.debug | 137 ++++++++++++++++++---
arch/arm/configs/ep93xx_defconfig | 1 +
arch/arm/configs/versatile_defconfig | 1 +
.../mach/debug-macro.S => include/debug/ks8695.S} | 10 +-
.../mach/debug-macro.S => include/debug/netx.S} | 22 ++--
arch/arm/include/debug/sa1100.S | 37 ++++++
arch/arm/mach-omap1/include/mach/debug-macro.S | 101 ---------------
arch/arm/mach-sa1100/include/mach/debug-macro.S | 62 ----------
8 files changed, 174 insertions(+), 197 deletions(-)
rename arch/arm/{mach-ks8695/include/mach/debug-macro.S => include/debug/ks8695.S} (80%)
rename arch/arm/{mach-netx/include/mach/debug-macro.S => include/debug/netx.S} (62%)
create mode 100644 arch/arm/include/debug/sa1100.S
delete mode 100644 arch/arm/mach-omap1/include/mach/debug-macro.S
delete mode 100644 arch/arm/mach-sa1100/include/mach/debug-macro.S
--
1.9.3
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v11 1/9] arm: versatile: Enable DEBUG_LL_UART_PL01X
2014-09-16 23:57 [PATCH v11 0/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X) Daniel Thompson
@ 2014-09-16 23:57 ` Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 2/9] arm: ep93xx: " Daniel Thompson
` (7 subsequent siblings)
8 siblings, 0 replies; 11+ messages in thread
From: Daniel Thompson @ 2014-09-16 23:57 UTC (permalink / raw)
To: linux-arm-kernel
This defconfig already enables DEBUG_LL and by default DEBUG_LL_UART_NONE
will be selected (but due to some back compability magic I'd like to
remove is not actually honoured). DEBUG_LL_UART_PL01X is a much saner
default.
Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
Acked-by: Arnd Bergmann <arnd@arndb.de>
---
arch/arm/configs/versatile_defconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/configs/versatile_defconfig b/arch/arm/configs/versatile_defconfig
index d52b4ff..ea49d37 100644
--- a/arch/arm/configs/versatile_defconfig
+++ b/arch/arm/configs/versatile_defconfig
@@ -82,5 +82,6 @@ CONFIG_MAGIC_SYSRQ=y
CONFIG_DEBUG_KERNEL=y
CONFIG_DEBUG_USER=y
CONFIG_DEBUG_LL=y
+CONFIG_DEBUG_LL_UART_PL01X=y
CONFIG_FONTS=y
CONFIG_FONT_ACORN_8x8=y
--
1.9.3
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v11 2/9] arm: ep93xx: Enable DEBUG_LL_UART_PL01X
2014-09-16 23:57 [PATCH v11 0/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X) Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 1/9] arm: versatile: Enable DEBUG_LL_UART_PL01X Daniel Thompson
@ 2014-09-16 23:57 ` Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 3/9] arm: Remove DEBUG_LL_UART_NONE Daniel Thompson
` (6 subsequent siblings)
8 siblings, 0 replies; 11+ messages in thread
From: Daniel Thompson @ 2014-09-16 23:57 UTC (permalink / raw)
To: linux-arm-kernel
This defconfig already enables DEBUG_LL and by default DEBUG_LL_UART_NONE
will be selected (but due to some back compability magic I'd like to
remove is not actually honoured). DEBUG_LL_UART_PL01X is a much saner
default.
Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
Acked-by: Arnd Bergmann <arnd@arndb.de>
---
arch/arm/configs/ep93xx_defconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/configs/ep93xx_defconfig b/arch/arm/configs/ep93xx_defconfig
index 1b650c8..72233b9 100644
--- a/arch/arm/configs/ep93xx_defconfig
+++ b/arch/arm/configs/ep93xx_defconfig
@@ -107,5 +107,6 @@ CONFIG_DEBUG_SPINLOCK=y
CONFIG_DEBUG_MUTEXES=y
CONFIG_DEBUG_USER=y
CONFIG_DEBUG_LL=y
+CONFIG_DEBUG_LL_UART_PL01X=y
# CONFIG_CRYPTO_ANSI_CPRNG is not set
CONFIG_LIBCRC32C=y
--
1.9.3
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v11 3/9] arm: Remove DEBUG_LL_UART_NONE
2014-09-16 23:57 [PATCH v11 0/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X) Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 1/9] arm: versatile: Enable DEBUG_LL_UART_PL01X Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 2/9] arm: ep93xx: " Daniel Thompson
@ 2014-09-16 23:57 ` Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 4/9] arm: ks8695: Migrate debug_ll macros to shared directory Daniel Thompson
` (5 subsequent siblings)
8 siblings, 0 replies; 11+ messages in thread
From: Daniel Thompson @ 2014-09-16 23:57 UTC (permalink / raw)
To: linux-arm-kernel
Only a very small handful of platforms support DEBUG_LL_UART_NONE but it
lurks in the menus of every single platform config ready to break the
build. This is an especial problem for defconfig/oldconfig since it is
often selected by default.
This patch solves the problem by removing this option. Any platforms
still depending upon this option must be migrated.
Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Russell King <linux@arm.linux.org.uk>
---
arch/arm/Kconfig.debug | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index b11ad54..9c46729 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm/Kconfig.debug
@@ -877,15 +877,6 @@ choice
This option selects UART0 on VIA/Wondermedia System-on-a-chip
devices, including VT8500, WM8505, WM8650 and WM8850.
- config DEBUG_LL_UART_NONE
- bool "No low-level debugging UART"
- depends on !ARCH_MULTIPLATFORM
- help
- Say Y here if your platform doesn't provide a UART option
- above. This relies on your platform choosing the right UART
- definition internally in order for low-level debugging to
- work.
-
config DEBUG_ICEDCC
bool "Kernel low-level debugging via EmbeddedICE DCC channel"
help
--
1.9.3
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v11 4/9] arm: ks8695: Migrate debug_ll macros to shared directory
2014-09-16 23:57 [PATCH v11 0/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X) Daniel Thompson
` (2 preceding siblings ...)
2014-09-16 23:57 ` [PATCH v11 3/9] arm: Remove DEBUG_LL_UART_NONE Daniel Thompson
@ 2014-09-16 23:57 ` Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 5/9] arm: omap1: Migrate debug_ll macros to use 8250.S Daniel Thompson
` (4 subsequent siblings)
8 siblings, 0 replies; 11+ messages in thread
From: Daniel Thompson @ 2014-09-16 23:57 UTC (permalink / raw)
To: linux-arm-kernel
As part of the migration a couple of uart definitions have been copied
from of the platform specific header files.
Note that, in order to keep oldconfig working nicely we must defer the
removal of arch/arm/mach-ks8695/include/mach/debug-macro.S until
DEBUG_LL_UART_NONE has been removed.
Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Russell King <linux@arm.linux.org.uk>
Cc: Greg Ungerer <gerg@uclinux.org>
Cc: Arnd Bergmann <arnd.bergmann@linaro.org>
---
arch/arm/Kconfig.debug | 8 ++++++++
.../include/mach/debug-macro.S => include/debug/ks8695.S} | 10 +++++++---
2 files changed, 15 insertions(+), 3 deletions(-)
rename arch/arm/{mach-ks8695/include/mach/debug-macro.S => include/debug/ks8695.S} (80%)
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 9c46729..97c3058 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm/Kconfig.debug
@@ -348,6 +348,13 @@ choice
Say Y here if you want the debug print routines to direct
their output to UART1 serial port on KEYSTONE2 devices.
+ config DEBUG_KS8695_UART
+ bool "KS8695 Debug UART"
+ depends on ARCH_KS8695
+ help
+ Say Y here if you want kernel low-level debugging support
+ on KS8695.
+
config DEBUG_MMP_UART2
bool "Kernel low-level debugging message via MMP UART2"
depends on ARCH_MMP
@@ -1017,6 +1024,7 @@ config DEBUG_LL_INCLUDE
DEBUG_IMX6Q_UART || \
DEBUG_IMX6SL_UART || \
DEBUG_IMX6SX_UART
+ default "debug/ks8695.S" if DEBUG_KS8695_UART
default "debug/msm.S" if DEBUG_MSM_UART || DEBUG_QCOM_UARTDM
default "debug/omap2plus.S" if DEBUG_OMAP2PLUS_UART
default "debug/s3c24xx.S" if DEBUG_S3C24XX_UART
diff --git a/arch/arm/mach-ks8695/include/mach/debug-macro.S b/arch/arm/include/debug/ks8695.S
similarity index 80%
rename from arch/arm/mach-ks8695/include/mach/debug-macro.S
rename to arch/arm/include/debug/ks8695.S
index a79e489..961da1f 100644
--- a/arch/arm/mach-ks8695/include/mach/debug-macro.S
+++ b/arch/arm/include/debug/ks8695.S
@@ -1,5 +1,5 @@
/*
- * arch/arm/mach-ks8695/include/mach/debug-macro.S
+ * arch/arm/include/debug/ks8695.S
*
* Copyright (C) 2006 Ben Dooks <ben@simtec.co.uk>
* Copyright (C) 2006 Simtec Electronics
@@ -11,8 +11,12 @@
* published by the Free Software Foundation.
*/
-#include <mach/hardware.h>
-#include <mach/regs-uart.h>
+#define KS8695_UART_PA 0x03ffe000
+#define KS8695_UART_VA 0xf00fe000
+#define KS8695_URTH (0x04)
+#define KS8695_URLS (0x14)
+#define URLS_URTE (1 << 6)
+#define URLS_URTHRE (1 << 5)
.macro addruart, rp, rv, tmp
ldr \rp, =KS8695_UART_PA @ physical base address
--
1.9.3
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v11 5/9] arm: omap1: Migrate debug_ll macros to use 8250.S
2014-09-16 23:57 [PATCH v11 0/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X) Daniel Thompson
` (3 preceding siblings ...)
2014-09-16 23:57 ` [PATCH v11 4/9] arm: ks8695: Migrate debug_ll macros to shared directory Daniel Thompson
@ 2014-09-16 23:57 ` Daniel Thompson
2014-09-18 16:05 ` Tony Lindgren
2014-09-16 23:57 ` [PATCH v11 6/9] arm: netx: Migrate DEBUG_LL macros to shared directory Daniel Thompson
` (3 subsequent siblings)
8 siblings, 1 reply; 11+ messages in thread
From: Daniel Thompson @ 2014-09-16 23:57 UTC (permalink / raw)
To: linux-arm-kernel
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.
Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Russell King <linux@arm.linux.org.uk>
Cc: linux-arm-kernel at lists.infradead.org
Cc: Tony Lindgren <tony@atomide.com>
Cc: Arnd Bergmann <arnd.bergmann@linaro.org>
Cc: linux-omap at vger.kernel.org
Tested-by: Aaro Koskinen <aaro.koskinen@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
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v11 6/9] arm: netx: Migrate DEBUG_LL macros to shared directory
2014-09-16 23:57 [PATCH v11 0/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X) Daniel Thompson
` (4 preceding siblings ...)
2014-09-16 23:57 ` [PATCH v11 5/9] arm: omap1: Migrate debug_ll macros to use 8250.S Daniel Thompson
@ 2014-09-16 23:57 ` Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 7/9] arm: sa1100: " Daniel Thompson
` (2 subsequent siblings)
8 siblings, 0 replies; 11+ messages in thread
From: Daniel Thompson @ 2014-09-16 23:57 UTC (permalink / raw)
To: linux-arm-kernel
As part of the migration we introduce DEBUG_UART_PHYS/DEBUG_UART_VIRT
which default to UART1 but allow a user to configure UART2 or UART3.
We also introduce symbolic names for the registers and flags.
Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Russell King <linux@arm.linux.org.uk>
Acked-by: Arnd Bergmann <arnd.bergmann@linaro.org>
Cc: linux-arm-kernel at lists.infradead.org
---
arch/arm/Kconfig.debug | 16 ++++++++++++++--
.../mach/debug-macro.S => include/debug/netx.S} | 22 +++++++++++-----------
2 files changed, 25 insertions(+), 13 deletions(-)
rename arch/arm/{mach-netx/include/mach/debug-macro.S => include/debug/netx.S} (62%)
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 6540036..ab5f60b 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm/Kconfig.debug
@@ -447,6 +447,13 @@ choice
Say Y here if you want kernel low-level debugging support
on Vybrid based platforms.
+ config DEBUG_NETX_UART
+ bool "Kernel low-level debugging messages via NetX UART"
+ depends on ARCH_NETX
+ help
+ Say Y here if you want kernel low-level debugging support
+ on Hilscher NetX based platforms.
+
config DEBUG_NOMADIK_UART
bool "Kernel low-level debugging messages via NOMADIK UART"
depends on ARCH_NOMADIK
@@ -1074,6 +1081,7 @@ config DEBUG_LL_INCLUDE
DEBUG_IMX6SX_UART
default "debug/ks8695.S" if DEBUG_KS8695_UART
default "debug/msm.S" if DEBUG_MSM_UART || DEBUG_QCOM_UARTDM
+ default "debug/netx.S" if DEBUG_NETX_UART
default "debug/omap2plus.S" if DEBUG_OMAP2PLUS_UART
default "debug/s3c24xx.S" if DEBUG_S3C24XX_UART
default "debug/s5pv210.S" if DEBUG_S5PV210_UART
@@ -1106,6 +1114,7 @@ config DEBUG_UART_8250
config DEBUG_UART_PHYS
hex "Physical base address of debug UART"
+ default 0x00100a00 if DEBUG_NETX_UART
default 0x01c20000 if DEBUG_DAVINCI_DMx_UART0
default 0x01c28000 if DEBUG_SUNXI_UART0
default 0x01c28400 if DEBUG_SUNXI_UART1
@@ -1176,10 +1185,12 @@ config DEBUG_UART_PHYS
depends on DEBUG_LL_UART_8250 || DEBUG_LL_UART_PL01X || \
DEBUG_LL_UART_EFM32 || \
DEBUG_UART_8250 || DEBUG_UART_PL01X || \
- DEBUG_MSM_UART || DEBUG_QCOM_UARTDM || DEBUG_S3C24XX_UART
+ DEBUG_MSM_UART || DEBUG_NETX_UART || DEBUG_QCOM_UARTDM || \
+ DEBUG_S3C24XX_UART
config DEBUG_UART_VIRT
hex "Virtual base address of debug UART"
+ default 0xe0000a00 if DEBUG_NETX_UART
default 0xe0010fe0 if ARCH_RPC
default 0xe1000000 if DEBUG_MSM_UART
default 0xf0000be0 if ARCH_EBSA110
@@ -1247,7 +1258,8 @@ config DEBUG_UART_VIRT
default DEBUG_UART_PHYS if !MMU
depends on DEBUG_LL_UART_8250 || DEBUG_LL_UART_PL01X || \
DEBUG_UART_8250 || DEBUG_UART_PL01X || \
- DEBUG_MSM_UART || DEBUG_QCOM_UARTDM || DEBUG_S3C24XX_UART
+ DEBUG_MSM_UART || DEBUG_NETX_UART || DEBUG_QCOM_UARTDM || \
+ DEBUG_S3C24XX_UART
config DEBUG_UART_8250_SHIFT
int "Register offset shift for the 8250 debug UART"
diff --git a/arch/arm/mach-netx/include/mach/debug-macro.S b/arch/arm/include/debug/netx.S
similarity index 62%
rename from arch/arm/mach-netx/include/mach/debug-macro.S
rename to arch/arm/include/debug/netx.S
index 247781e..81e1b2a 100644
--- a/arch/arm/mach-netx/include/mach/debug-macro.S
+++ b/arch/arm/include/debug/netx.S
@@ -1,5 +1,4 @@
-/* arch/arm/mach-netx/include/mach/debug-macro.S
- *
+/*
* Debugging macro include header
*
* Copyright (C) 1994-1999 Russell King
@@ -11,26 +10,27 @@
*
*/
-#include "hardware.h"
+#define UART_DATA 0
+#define UART_FLAG 0x18
+#define UART_FLAG_BUSY (1 << 3)
.macro addruart, rp, rv, tmp
- mov \rp, #0x00000a00
- orr \rv, \rp, #io_p2v(0x00100000) @ virtual
- orr \rp, \rp, #0x00100000 @ physical
+ ldr \rp, =CONFIG_DEBUG_UART_PHYS
+ ldr \rv, =CONFIG_DEBUG_UART_VIRT
.endm
.macro senduart,rd,rx
- str \rd, [\rx, #0]
+ str \rd, [\rx, #UART_DATA]
.endm
.macro busyuart,rd,rx
-1002: ldr \rd, [\rx, #0x18]
- tst \rd, #(1 << 3)
+1002: ldr \rd, [\rx, #UART_FLAG]
+ tst \rd, #UART_FLAG_BUSY
bne 1002b
.endm
.macro waituart,rd,rx
-1001: ldr \rd, [\rx, #0x18]
- tst \rd, #(1 << 3)
+1001: ldr \rd, [\rx, #UART_FLAG]
+ tst \rd, #UART_FLAG_BUSY
bne 1001b
.endm
--
1.9.3
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v11 7/9] arm: sa1100: Migrate DEBUG_LL macros to shared directory
2014-09-16 23:57 [PATCH v11 0/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X) Daniel Thompson
` (5 preceding siblings ...)
2014-09-16 23:57 ` [PATCH v11 6/9] arm: netx: Migrate DEBUG_LL macros to shared directory Daniel Thompson
@ 2014-09-16 23:57 ` Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 8/9] arm: Seperate DEBUG_UART_PHYS from DEBUG_LL on EP93XX Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 9/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X) Daniel Thompson
8 siblings, 0 replies; 11+ messages in thread
From: Daniel Thompson @ 2014-09-16 23:57 UTC (permalink / raw)
To: linux-arm-kernel
As part of the migration we copy a few definitions from the mach-sa1100
headers and replace the automatic serial port detectiion with explicit
configuration based on DEBUG_UART_PHYS/DEBUG_UART_VIRT.
The removal of the automatic configuration is similar to the way migration
was handled for OMAP2 (see 808b7e07464d...).
Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Russell King <linux@arm.linux.org.uk>
---
arch/arm/Kconfig.debug | 41 ++++++++++++++--
arch/arm/include/debug/sa1100.S | 37 +++++++++++++++
arch/arm/mach-sa1100/include/mach/debug-macro.S | 62 -------------------------
3 files changed, 75 insertions(+), 65 deletions(-)
create mode 100644 arch/arm/include/debug/sa1100.S
delete mode 100644 arch/arm/mach-sa1100/include/mach/debug-macro.S
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index ab5f60b..01bf5c7 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm/Kconfig.debug
@@ -761,6 +761,30 @@ choice
their output to UART 2. The port must have been initialised
by the boot-loader before use.
+ config DEBUG_SA1100_UART1
+ depends on ARCH_SA1100
+ select DEBUG_SA1100_UART
+ bool "Kernel low-level debugging messages via SA1100 Ser1"
+ help
+ Say Y here if you want kernel low-level debugging support
+ on SA1100 based platforms.
+
+ config DEBUG_SA1100_UART2
+ depends on ARCH_SA1100
+ select DEBUG_SA1100_UART
+ bool "Kernel low-level debugging messages via SA1100 Ser2"
+ help
+ Say Y here if you want kernel low-level debugging support
+ on SA1100 based platforms.
+
+ config DEBUG_SA1100_UART3
+ depends on ARCH_SA1100
+ select DEBUG_SA1100_UART
+ bool "Kernel low-level debugging messages via SA1100 Ser3"
+ help
+ Say Y here if you want kernel low-level debugging support
+ on SA1100 based platforms.
+
config DEBUG_SOCFPGA_UART
depends on ARCH_SOCFPGA
bool "Use SOCFPGA UART for low-level debug"
@@ -1056,6 +1080,10 @@ config DEBUG_TEGRA_UART
bool
depends on ARCH_TEGRA
+config DEBUG_SA1100_UART
+ bool
+ depends on ARCH_SA1100
+
config DEBUG_STI_UART
bool
depends on ARCH_STI
@@ -1085,6 +1113,7 @@ config DEBUG_LL_INCLUDE
default "debug/omap2plus.S" if DEBUG_OMAP2PLUS_UART
default "debug/s3c24xx.S" if DEBUG_S3C24XX_UART
default "debug/s5pv210.S" if DEBUG_S5PV210_UART
+ default "debug/sa1100.S" if DEBUG_SA1100_UART
default "debug/sirf.S" if DEBUG_SIRFPRIMA2_UART1 || DEBUG_SIRFMARCO_UART1
default "debug/sti.S" if DEBUG_STI_UART
default "debug/tegra.S" if DEBUG_TEGRA_UART
@@ -1104,7 +1133,7 @@ config DEBUG_UART_PL01X
ARCH_SPEAR13XX || \
ARCH_VERSATILE
-# Compatibility options for 8250
+#Compatibility options for 8250
config DEBUG_UART_8250
def_bool ARCH_DOVE || ARCH_EBSA110 || \
(FOOTBRIDGE && !DEBUG_DC21285_PORT) || \
@@ -1150,6 +1179,9 @@ config DEBUG_UART_PHYS
default 0x50008000 if DEBUG_S3C24XX_UART && (DEBUG_S3C_UART2 || \
DEBUG_S3C2410_UART2)
default 0x7c0003f8 if FOOTBRIDGE
+ default 0x80010000 if DEBUG_SA1100_UART1
+ default 0x80030000 if DEBUG_SA1100_UART2
+ default 0x80050000 if DEBUG_SA1100_UART3
default 0x80070000 if DEBUG_IMX23_UART
default 0x80074000 if DEBUG_IMX28_UART
default 0x80230000 if DEBUG_PICOXCELL_UART
@@ -1186,7 +1218,7 @@ config DEBUG_UART_PHYS
DEBUG_LL_UART_EFM32 || \
DEBUG_UART_8250 || DEBUG_UART_PL01X || \
DEBUG_MSM_UART || DEBUG_NETX_UART || DEBUG_QCOM_UARTDM || \
- DEBUG_S3C24XX_UART
+ DEBUG_S3C24XX_UART || DEBUG_SA1100_UART
config DEBUG_UART_VIRT
hex "Virtual base address of debug UART"
@@ -1214,6 +1246,9 @@ config DEBUG_UART_VIRT
DEBUG_S3C2410_UART2)
default 0xf7fc9000 if DEBUG_BERLIN_UART
default 0xf8009000 if DEBUG_VEXPRESS_UART0_CA9
+ default 0xf8010000 if DEBUG_SA1100_UART1
+ default 0xf8030000 if DEBUG_SA1100_UART2
+ default 0xf8050000 if DEBUG_SA1100_UART3
default 0xf8090000 if DEBUG_VEXPRESS_UART0_RS1
default 0xfa71e000 if DEBUG_QCOM_UARTDM
default 0xfb009000 if DEBUG_REALVIEW_STD_PORT
@@ -1259,7 +1294,7 @@ config DEBUG_UART_VIRT
depends on DEBUG_LL_UART_8250 || DEBUG_LL_UART_PL01X || \
DEBUG_UART_8250 || DEBUG_UART_PL01X || \
DEBUG_MSM_UART || DEBUG_NETX_UART || DEBUG_QCOM_UARTDM || \
- DEBUG_S3C24XX_UART
+ DEBUG_S3C24XX_UART || DEBUG_SA1100_UART
config DEBUG_UART_8250_SHIFT
int "Register offset shift for the 8250 debug UART"
diff --git a/arch/arm/include/debug/sa1100.S b/arch/arm/include/debug/sa1100.S
new file mode 100644
index 0000000..6b5e1ce
--- /dev/null
+++ b/arch/arm/include/debug/sa1100.S
@@ -0,0 +1,37 @@
+/*
+ * 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.
+ *
+*/
+
+#define UTDR 0x14
+#define UTSR1 0x20
+#define UTSR1_TBY 0x00000001 /* Transmitter BusY (read) */
+#define UTSR1_TNF 0x00000004 /* Transmit FIFO Not Full (read) */
+
+ .macro addruart, rp, rv, tmp
+ ldr \rp, =CONFIG_DEBUG_UART_PHYS
+ ldr \rv, =CONFIG_DEBUG_UART_VIRT
+ .endm
+
+ .macro senduart,rd,rx
+ str \rd, [\rx, #UTDR]
+ .endm
+
+ .macro waituart,rd,rx
+1001: ldr \rd, [\rx, #UTSR1]
+ tst \rd, #UTSR1_TNF
+ beq 1001b
+ .endm
+
+ .macro busyuart,rd,rx
+1001: ldr \rd, [\rx, #UTSR1]
+ tst \rd, #UTSR1_TBY
+ bne 1001b
+ .endm
diff --git a/arch/arm/mach-sa1100/include/mach/debug-macro.S b/arch/arm/mach-sa1100/include/mach/debug-macro.S
deleted file mode 100644
index 530772d..0000000
--- a/arch/arm/mach-sa1100/include/mach/debug-macro.S
+++ /dev/null
@@ -1,62 +0,0 @@
-/* arch/arm/mach-sa1100/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 <mach/hardware.h>
-
- .macro addruart, rp, rv, tmp
- mrc p15, 0, \rp, c1, c0
- tst \rp, #1 @ MMU enabled?
- moveq \rp, #0x80000000 @ physical base address
- movne \rp, #0xf8000000 @ virtual address
-
- @ We probe for the active serial port here, coherently with
- @ the comment in arch/arm/mach-sa1100/include/mach/uncompress.h.
- @ We assume r1 can be clobbered.
-
- @ see if Ser3 is active
- add \rp, \rp, #0x00050000
- ldr \rv, [\rp, #UTCR3]
- tst \rv, #UTCR3_TXE
-
- @ if Ser3 is inactive, then try Ser1
- addeq \rp, \rp, #(0x00010000 - 0x00050000)
- ldreq \rv, [\rp, #UTCR3]
- tsteq \rv, #UTCR3_TXE
-
- @ if Ser1 is inactive, then try Ser2
- addeq \rp, \rp, #(0x00030000 - 0x00010000)
- ldreq \rv, [\rp, #UTCR3]
- tsteq \rv, #UTCR3_TXE
-
- @ clear top bits, and generate both phys and virt addresses
- lsl \rp, \rp, #8
- lsr \rp, \rp, #8
- orr \rv, \rp, #0xf8000000 @ virtual
- orr \rp, \rp, #0x80000000 @ physical
-
- .endm
-
- .macro senduart,rd,rx
- str \rd, [\rx, #UTDR]
- .endm
-
- .macro waituart,rd,rx
-1001: ldr \rd, [\rx, #UTSR1]
- tst \rd, #UTSR1_TNF
- beq 1001b
- .endm
-
- .macro busyuart,rd,rx
-1001: ldr \rd, [\rx, #UTSR1]
- tst \rd, #UTSR1_TBY
- bne 1001b
- .endm
--
1.9.3
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v11 8/9] arm: Seperate DEBUG_UART_PHYS from DEBUG_LL on EP93XX
2014-09-16 23:57 [PATCH v11 0/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X) Daniel Thompson
` (6 preceding siblings ...)
2014-09-16 23:57 ` [PATCH v11 7/9] arm: sa1100: " Daniel Thompson
@ 2014-09-16 23:57 ` Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 9/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X) Daniel Thompson
8 siblings, 0 replies; 11+ messages in thread
From: Daniel Thompson @ 2014-09-16 23:57 UTC (permalink / raw)
To: linux-arm-kernel
On EP93XX uncompress.h uses CONFIG_DEBUG_UART_PHYS instead of a hard
coded serial port. This means the build breaks when DEBUG_LL
(and DEBUG_LL_UART_PL01X) is not enabled.
This is fixed by adding a new dependancy.
Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
Acked-by: Arnd Bergmann <arnd@arndb.de>
---
arch/arm/Kconfig.debug | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 01bf5c7..9fa93ea 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm/Kconfig.debug
@@ -1218,7 +1218,8 @@ config DEBUG_UART_PHYS
DEBUG_LL_UART_EFM32 || \
DEBUG_UART_8250 || DEBUG_UART_PL01X || \
DEBUG_MSM_UART || DEBUG_NETX_UART || DEBUG_QCOM_UARTDM || \
- DEBUG_S3C24XX_UART || DEBUG_SA1100_UART
+ DEBUG_S3C24XX_UART || DEBUG_SA1100_UART || \
+ ARCH_EP93XX
config DEBUG_UART_VIRT
hex "Virtual base address of debug UART"
--
1.9.3
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v11 9/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X)
2014-09-16 23:57 [PATCH v11 0/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X) Daniel Thompson
` (7 preceding siblings ...)
2014-09-16 23:57 ` [PATCH v11 8/9] arm: Seperate DEBUG_UART_PHYS from DEBUG_LL on EP93XX Daniel Thompson
@ 2014-09-16 23:57 ` Daniel Thompson
8 siblings, 0 replies; 11+ messages in thread
From: Daniel Thompson @ 2014-09-16 23:57 UTC (permalink / raw)
To: linux-arm-kernel
When building a multi_v7_defconfig kernel it is not possible to configure
DEBUG_LL to use any serial device except a ARM Primecell PL01X, or more
accurately and worse, it is possible to configure a different serial
device but KConfig does not honour this request. In fact this also
overrides the user selection for some of the single platform kernels, for
example I don't think DEBUG_LL can be targetted at ICE or semihosted
supervisor for ARCH_VERSATILE.
This happens because DEBUG_UART_PL01X is automatically enabled by
some architectures and this means user decisions made regarding
the DEBUG_LL backend will be overriden. Problem is fixed by removing the
automatic enabling of this option.
Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
Acked-by: Arnd Bergmann <arnd@arndb.de>
---
arch/arm/Kconfig.debug | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 9fa93ea..4fcff77 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm/Kconfig.debug
@@ -1126,12 +1126,7 @@ config DEBUG_LL_INCLUDE
# Compatibility options for PL01x
config DEBUG_UART_PL01X
- def_bool ARCH_EP93XX || \
- ARCH_INTEGRATOR || \
- ARCH_SPEAR3XX || \
- ARCH_SPEAR6XX || \
- ARCH_SPEAR13XX || \
- ARCH_VERSATILE
+ bool
#Compatibility options for 8250
config DEBUG_UART_8250
--
1.9.3
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v11 5/9] arm: omap1: Migrate debug_ll macros to use 8250.S
2014-09-16 23:57 ` [PATCH v11 5/9] arm: omap1: Migrate debug_ll macros to use 8250.S Daniel Thompson
@ 2014-09-18 16:05 ` Tony Lindgren
0 siblings, 0 replies; 11+ messages in thread
From: Tony Lindgren @ 2014-09-18 16:05 UTC (permalink / raw)
To: linux-arm-kernel
* Daniel Thompson <daniel.thompson@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 <tony@atomide.com>
> Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Russell King <linux@arm.linux.org.uk>
> Cc: linux-arm-kernel at lists.infradead.org
> Cc: Tony Lindgren <tony@atomide.com>
> Cc: Arnd Bergmann <arnd.bergmann@linaro.org>
> Cc: linux-omap at vger.kernel.org
> Tested-by: Aaro Koskinen <aaro.koskinen@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
>
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2014-09-18 16:05 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-09-16 23:57 [PATCH v11 0/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X) Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 1/9] arm: versatile: Enable DEBUG_LL_UART_PL01X Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 2/9] arm: ep93xx: " Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 3/9] arm: Remove DEBUG_LL_UART_NONE Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 4/9] arm: ks8695: Migrate debug_ll macros to shared directory Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 5/9] arm: omap1: Migrate debug_ll macros to use 8250.S Daniel Thompson
2014-09-18 16:05 ` Tony Lindgren
2014-09-16 23:57 ` [PATCH v11 6/9] arm: netx: Migrate DEBUG_LL macros to shared directory Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 7/9] arm: sa1100: " Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 8/9] arm: Seperate DEBUG_UART_PHYS from DEBUG_LL on EP93XX Daniel Thompson
2014-09-16 23:57 ` [PATCH v11 9/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X) Daniel Thompson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).