* [PATCH v2] ARM: at91: debug: add lan966 support
@ 2022-03-24 13:10 Kavyasree Kotagiri
2022-03-24 13:38 ` Michael Walle
0 siblings, 1 reply; 2+ messages in thread
From: Kavyasree Kotagiri @ 2022-03-24 13:10 UTC (permalink / raw)
To: alexandre.belloni, linux, linus.walleij
Cc: linux-arm-kernel, linux-kernel, UNGLinuxDriver, michael,
Manohar.Puri, Kavyasree.Kotagiri
Add support for low-level debugging on FLEXCOM3 USART of
LAN966 SoC.
Signed-off-by: Kavyasree Kotagiri <kavyasree.kotagiri@microchip.com>
---
v1 -> v2:
- Corrected flexcom3 base address - 0xe0064200.
arch/arm/Kconfig.debug | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 0c9497d549e3..04501ddc7ab4 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm/Kconfig.debug
@@ -210,6 +210,14 @@ choice
Say Y here if you want kernel low-level debugging support
on the FLEXCOM3 port of SAMA7G5.
+ config DEBUG_AT91_LAN966_FLEXCOM3
+ bool "Kernel low-level debugging on LAN966 FLEXCOM3 USART"
+ select DEBUG_AT91_UART
+ depends on SOC_LAN966
+ help
+ Say Y here if you want kernel low-level debugging support
+ on the FLEXCOM3 port of LAN966.
+
config DEBUG_BCM2835
bool "Kernel low-level debugging on BCM2835 PL011 UART"
depends on ARCH_BCM2835 && ARCH_MULTI_V6
@@ -1685,6 +1693,7 @@ config DEBUG_UART_PHYS
default 0xd4017000 if DEBUG_MMP_UART2
default 0xd4018000 if DEBUG_MMP_UART3
default 0xe0000000 if DEBUG_SPEAR13XX
+ default 0xe0064200 if DEBUG_AT91_LAN966_FLEXCOM3
default 0xe1824200 if DEBUG_AT91_SAMA7G5_FLEXCOM3
default 0xe4007000 if DEBUG_HIP04_UART
default 0xe6c40000 if DEBUG_RMOBILE_SCIFA0
@@ -1805,6 +1814,7 @@ config DEBUG_UART_VIRT
default 0xfb10c000 if DEBUG_REALVIEW_PB1176_PORT
default 0xfcfe8600 if DEBUG_BCM63XX_UART
default 0xfd000000 if DEBUG_SPEAR3XX || DEBUG_SPEAR13XX
+ default 0xfd064200 if DEBUG_AT91_LAN966_FLEXCOM3
default 0xfd531000 if DEBUG_STIH41X_SBC_ASC1
default 0xfd883000 if DEBUG_ALPINE_UART0
default 0xfdd32000 if DEBUG_STIH41X_ASC2
--
2.17.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] ARM: at91: debug: add lan966 support
2022-03-24 13:10 [PATCH v2] ARM: at91: debug: add lan966 support Kavyasree Kotagiri
@ 2022-03-24 13:38 ` Michael Walle
0 siblings, 0 replies; 2+ messages in thread
From: Michael Walle @ 2022-03-24 13:38 UTC (permalink / raw)
To: Kavyasree Kotagiri
Cc: alexandre.belloni, linux, linus.walleij, linux-arm-kernel,
linux-kernel, UNGLinuxDriver, Manohar.Puri
Hi Kavyasree,
Am 2022-03-24 14:10, schrieb Kavyasree Kotagiri:
> Add support for low-level debugging on FLEXCOM3 USART of
> LAN966 SoC.
>
> Signed-off-by: Kavyasree Kotagiri <kavyasree.kotagiri@microchip.com>
> ---
> v1 -> v2:
> - Corrected flexcom3 base address - 0xe0064200.
Unfortunately, you didn't respond to my previous remarks. Not
everyone has the debug console on FLEXCOM3. Therefore, I suggested
to use DEBUG_AT91_LAN966_FLEXCOM (please note the missing '3' at
the end) and add a table with all the offsets to the help text.
I'm fine with having the flexcom3 base address by default. But
that should also be noted in the help text.
-michael
> arch/arm/Kconfig.debug | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
> index 0c9497d549e3..04501ddc7ab4 100644
> --- a/arch/arm/Kconfig.debug
> +++ b/arch/arm/Kconfig.debug
> @@ -210,6 +210,14 @@ choice
> Say Y here if you want kernel low-level debugging support
> on the FLEXCOM3 port of SAMA7G5.
>
> + config DEBUG_AT91_LAN966_FLEXCOM3
> + bool "Kernel low-level debugging on LAN966 FLEXCOM3
> USART"
> + select DEBUG_AT91_UART
> + depends on SOC_LAN966
> + help
> + Say Y here if you want kernel low-level debugging
> support
> + on the FLEXCOM3 port of LAN966.
> +
> config DEBUG_BCM2835
> bool "Kernel low-level debugging on BCM2835 PL011 UART"
> depends on ARCH_BCM2835 && ARCH_MULTI_V6
> @@ -1685,6 +1693,7 @@ config DEBUG_UART_PHYS
> default 0xd4017000 if DEBUG_MMP_UART2
> default 0xd4018000 if DEBUG_MMP_UART3
> default 0xe0000000 if DEBUG_SPEAR13XX
> + default 0xe0064200 if DEBUG_AT91_LAN966_FLEXCOM3
> default 0xe1824200 if DEBUG_AT91_SAMA7G5_FLEXCOM3
> default 0xe4007000 if DEBUG_HIP04_UART
> default 0xe6c40000 if DEBUG_RMOBILE_SCIFA0
> @@ -1805,6 +1814,7 @@ config DEBUG_UART_VIRT
> default 0xfb10c000 if DEBUG_REALVIEW_PB1176_PORT
> default 0xfcfe8600 if DEBUG_BCM63XX_UART
> default 0xfd000000 if DEBUG_SPEAR3XX || DEBUG_SPEAR13XX
> + default 0xfd064200 if DEBUG_AT91_LAN966_FLEXCOM3
> default 0xfd531000 if DEBUG_STIH41X_SBC_ASC1
> default 0xfd883000 if DEBUG_ALPINE_UART0
> default 0xfdd32000 if DEBUG_STIH41X_ASC2
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-03-24 13:40 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-03-24 13:10 [PATCH v2] ARM: at91: debug: add lan966 support Kavyasree Kotagiri
2022-03-24 13:38 ` Michael Walle
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).