Skip to contentSkip to conversation

ARM: lpc18xx: define low-level debug symbol for LPC18xx/43xx

PATCH New Joachim Eastwood 3 participants 3 messages May 19, 2015 – May 20, 2015 Arm (32-bit)

lore
Share & tools
lore

Joachim Eastwood

author
#0 lore raw

PATCH 1/1 ARM: lpc18xx: define low-level debug symbol for LPC18xx/43xx

Using a dedicated symbol for low-level debugging instead of the arch symbol will make this platform play nice when enabled on a kernel that supports multiple platforms.

Signed-off-by Joachim Eastwood <[email protected]>

---
Hi Arnd,

As disscussed on the mailing list:
http://marc.info/?l=linux-arm-kernel&m=143202771500659&w=2

Patch should apply nicely on top of next/soc.

 arch/arm/Kconfig.debug | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)
arch/arm/Kconfig.debug +9 -1
index b3b6469350c1..689b524f8428 100644
@@ -433,6 +433,14 @@ choice
 		  Say Y here if you want kernel low-level debugging support
 		  on KS8695.
 
+	config DEBUG_LPC18XX_UART0
+		bool "Kernel low-level debugging via LPC18xx/43xx UART0"
+		depends on ARCH_LPC18XX
+		select DEBUG_UART_8250
+		help
+		  Say Y here if you want kernel low-level debugging support
+		  on NXP LPC18xx/43xx UART0.
+
 	config DEBUG_MESON_UARTAO
 		bool "Kernel low-level debugging via Meson6 UARTAO"
 		depends on ARCH_MESON

@@ -1359,7 +1367,7 @@ config DEBUG_UART_PHYS
 	default 0x20201000 if DEBUG_BCM2835
 	default 0x3e000000 if DEBUG_BCM_KONA_UART
 	default 0x4000e400 if DEBUG_LL_UART_EFM32
-	default 0x40081000 if ARCH_LPC18XX
+	default 0x40081000 if DEBUG_LPC18XX_UART0
 	default 0x40090000 if ARCH_LPC32XX
 	default 0x40100000 if DEBUG_PXA_UART1
 	default 0x42000000 if ARCH_GEMINI
Signature · 1 line
1.8.0
#1 lore raw

PATCH 1/1 ARM: lpc18xx: define low-level debug symbol for LPC18xx/43xx

Hi Joachim,

Quoted 3 lines · Joachim Eastwood

Using a dedicated symbol for low-level debugging instead of the arch symbol will make this platform play nice when enabled on a kernel that supports multiple platforms.

Thanks a lot!

Quoted 1 line
Signed-off-by Joachim Eastwood <[email protected]>
Acked-by Geert Uytterhoeven <[email protected]>

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

Applied
#2 lore raw

PATCH 1/1 ARM: lpc18xx: define low-level debug symbol for LPC18xx/43xx

Quoted 7 lines · On Tuesday 19 May 2015 21:37:32 Geert Uytterhoeven wrote:

Hi Joachim,

Quoted 3 lines · Joachim Eastwood

Using a dedicated symbol for low-level debugging instead of the arch symbol will make this platform play nice when enabled on a kernel that supports multiple platforms.

Quoted 7 lines

Thanks a lot!

Quoted 1 line
Signed-off-by Joachim Eastwood <[email protected]>
Acked-by Geert Uytterhoeven <[email protected]>

Applied to next/soc, with Geert's Ack. Thanks!

Arnd