* [PATCH v2] ARM: bcm: Add DEBUG_LL console support
@ 2013-09-18 0:05 ` Christian Daudt
0 siblings, 0 replies; 6+ messages in thread
From: Christian Daudt @ 2013-09-18 0:05 UTC (permalink / raw)
To: Russell King, Olof Johansson, Stephen Warren
Cc: Christian Daudt, linux-arm-kernel, Chris Ball, linux-kernel, bcm
This patch adds low level debug uart support to Broadcom
mobile based SOCs.
Signed-off-by: Christian Daudt <csd@broadcom.com>
Changes from V1:
- Switched to use the common 8250 debug introduced in 3.12-rc1
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 9762c84..0523e71 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm/Kconfig.debug
@@ -94,6 +94,17 @@ choice
depends on ARCH_BCM2835
select DEBUG_UART_PL01X
+ config DEBUG_BCM_KONA_UART
+ bool "Kernel low-level debugging messages via BCM KONA UART"
+ depends on ARCH_BCM
+ select DEBUG_UART_8250
+ help
+ Say Y here if you want kernel low-level debugging support
+ on Broadcom SoC platforms.
+ This low level debug works for Broadcom
+ mobile SoCs in the Kona family of chips (e.g. bcm28155,
+ bcm11351, etc...)
+
config DEBUG_CLPS711X_UART1
bool "Kernel low-level debugging messages via UART1"
depends on ARCH_CLPS711X
@@ -980,6 +991,7 @@ config DEBUG_UART_PHYS
default 0xffd82340 if ARCH_IOP13XX
default 0xfff36000 if DEBUG_HIGHBANK_UART
default 0xfffff700 if ARCH_IOP33X
+ default 0x3e000000 if DEBUG_BCM_KONA_UART
depends on DEBUG_LL_UART_8250 || DEBUG_LL_UART_PL01X || \
DEBUG_UART_8250 || DEBUG_UART_PL01X
@@ -1035,6 +1047,7 @@ config DEBUG_UART_VIRT
default 0xfef00003 if ARCH_IXP4XX && CPU_BIG_ENDIAN
default 0xfefff700 if ARCH_IOP33X
default 0xff003000 if DEBUG_U300_UART
+ default 0xfe300000 if DEBUG_BCM_KONA_UART
default DEBUG_UART_PHYS if !MMU
depends on DEBUG_LL_UART_8250 || DEBUG_LL_UART_PL01X || \
DEBUG_UART_8250 || DEBUG_UART_PL01X
@@ -1052,7 +1065,8 @@ config DEBUG_UART_8250_WORD
default y if DEBUG_PICOXCELL_UART || DEBUG_SOCFPGA_UART || \
ARCH_KEYSTONE || \
DEBUG_DAVINCI_DMx_UART0 || DEBUG_DAVINCI_DA8XX_UART1 || \
- DEBUG_DAVINCI_DA8XX_UART2 || DEBUG_DAVINCI_TNETV107X_UART1
+ DEBUG_DAVINCI_DA8XX_UART2 || DEBUG_DAVINCI_TNETV107X_UART1 || \
+ DEBUG_BCM_KONA_UART
config DEBUG_UART_8250_FLOW_CONTROL
bool "Enable flow control for 8250 UART"
diff --git a/arch/arm/configs/bcm_defconfig b/arch/arm/configs/bcm_defconfig
index 6e49310..5c12c4c 100644
--- a/arch/arm/configs/bcm_defconfig
+++ b/arch/arm/configs/bcm_defconfig
@@ -30,6 +30,8 @@ CONFIG_ARCH_BCM=y
CONFIG_ARM_THUMBEE=y
CONFIG_ARM_ERRATA_743622=y
CONFIG_PREEMPT=y
+CONFIG_DEBUG_LL=y
+CONFIG_DEBUG_BCM_KONA_UART=y
CONFIG_AEABI=y
# CONFIG_OABI_COMPAT is not set
# CONFIG_COMPACTION is not set
--
1.7.10.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2] ARM: bcm: Add DEBUG_LL console support
@ 2013-09-18 0:05 ` Christian Daudt
0 siblings, 0 replies; 6+ messages in thread
From: Christian Daudt @ 2013-09-18 0:05 UTC (permalink / raw)
To: linux-arm-kernel
This patch adds low level debug uart support to Broadcom
mobile based SOCs.
Signed-off-by: Christian Daudt <csd@broadcom.com>
Changes from V1:
- Switched to use the common 8250 debug introduced in 3.12-rc1
diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 9762c84..0523e71 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm/Kconfig.debug
@@ -94,6 +94,17 @@ choice
depends on ARCH_BCM2835
select DEBUG_UART_PL01X
+ config DEBUG_BCM_KONA_UART
+ bool "Kernel low-level debugging messages via BCM KONA UART"
+ depends on ARCH_BCM
+ select DEBUG_UART_8250
+ help
+ Say Y here if you want kernel low-level debugging support
+ on Broadcom SoC platforms.
+ This low level debug works for Broadcom
+ mobile SoCs in the Kona family of chips (e.g. bcm28155,
+ bcm11351, etc...)
+
config DEBUG_CLPS711X_UART1
bool "Kernel low-level debugging messages via UART1"
depends on ARCH_CLPS711X
@@ -980,6 +991,7 @@ config DEBUG_UART_PHYS
default 0xffd82340 if ARCH_IOP13XX
default 0xfff36000 if DEBUG_HIGHBANK_UART
default 0xfffff700 if ARCH_IOP33X
+ default 0x3e000000 if DEBUG_BCM_KONA_UART
depends on DEBUG_LL_UART_8250 || DEBUG_LL_UART_PL01X || \
DEBUG_UART_8250 || DEBUG_UART_PL01X
@@ -1035,6 +1047,7 @@ config DEBUG_UART_VIRT
default 0xfef00003 if ARCH_IXP4XX && CPU_BIG_ENDIAN
default 0xfefff700 if ARCH_IOP33X
default 0xff003000 if DEBUG_U300_UART
+ default 0xfe300000 if DEBUG_BCM_KONA_UART
default DEBUG_UART_PHYS if !MMU
depends on DEBUG_LL_UART_8250 || DEBUG_LL_UART_PL01X || \
DEBUG_UART_8250 || DEBUG_UART_PL01X
@@ -1052,7 +1065,8 @@ config DEBUG_UART_8250_WORD
default y if DEBUG_PICOXCELL_UART || DEBUG_SOCFPGA_UART || \
ARCH_KEYSTONE || \
DEBUG_DAVINCI_DMx_UART0 || DEBUG_DAVINCI_DA8XX_UART1 || \
- DEBUG_DAVINCI_DA8XX_UART2 || DEBUG_DAVINCI_TNETV107X_UART1
+ DEBUG_DAVINCI_DA8XX_UART2 || DEBUG_DAVINCI_TNETV107X_UART1 || \
+ DEBUG_BCM_KONA_UART
config DEBUG_UART_8250_FLOW_CONTROL
bool "Enable flow control for 8250 UART"
diff --git a/arch/arm/configs/bcm_defconfig b/arch/arm/configs/bcm_defconfig
index 6e49310..5c12c4c 100644
--- a/arch/arm/configs/bcm_defconfig
+++ b/arch/arm/configs/bcm_defconfig
@@ -30,6 +30,8 @@ CONFIG_ARCH_BCM=y
CONFIG_ARM_THUMBEE=y
CONFIG_ARM_ERRATA_743622=y
CONFIG_PREEMPT=y
+CONFIG_DEBUG_LL=y
+CONFIG_DEBUG_BCM_KONA_UART=y
CONFIG_AEABI=y
# CONFIG_OABI_COMPAT is not set
# CONFIG_COMPACTION is not set
--
1.7.10.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2] ARM: bcm: Add DEBUG_LL console support
2013-09-18 0:05 ` Christian Daudt
@ 2013-10-03 6:22 ` Christian Daudt
-1 siblings, 0 replies; 6+ messages in thread
From: Christian Daudt @ 2013-10-03 6:22 UTC (permalink / raw)
To: Christian Daudt, Christian Daudt, Russell King
Cc: Olof Johansson, Stephen Warren, Chris Ball, linux-kernel,
Linux ARM Kernel
On Tue, Sep 17, 2013 at 5:05 PM, Christian Daudt <csd@broadcom.com> wrote:
> This patch adds low level debug uart support to Broadcom
> mobile based SOCs.
>
> Signed-off-by: Christian Daudt <csd@broadcom.com>
>
> Changes from V1:
> - Switched to use the common 8250 debug introduced in 3.12-rc1
>
Hi Russell,
Are you okay with v2 of DEBUG_LL support for Broadcom mobile SoC ? If
so, should I submit it through your patchqueue ?
Thanks,
csd
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2] ARM: bcm: Add DEBUG_LL console support
@ 2013-10-03 6:22 ` Christian Daudt
0 siblings, 0 replies; 6+ messages in thread
From: Christian Daudt @ 2013-10-03 6:22 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Sep 17, 2013 at 5:05 PM, Christian Daudt <csd@broadcom.com> wrote:
> This patch adds low level debug uart support to Broadcom
> mobile based SOCs.
>
> Signed-off-by: Christian Daudt <csd@broadcom.com>
>
> Changes from V1:
> - Switched to use the common 8250 debug introduced in 3.12-rc1
>
Hi Russell,
Are you okay with v2 of DEBUG_LL support for Broadcom mobile SoC ? If
so, should I submit it through your patchqueue ?
Thanks,
csd
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] ARM: bcm: Add DEBUG_LL console support
2013-09-18 0:05 ` Christian Daudt
@ 2013-10-03 7:21 ` Russell King - ARM Linux
-1 siblings, 0 replies; 6+ messages in thread
From: Russell King - ARM Linux @ 2013-10-03 7:21 UTC (permalink / raw)
To: Christian Daudt
Cc: Olof Johansson, Stephen Warren, linux-arm-kernel, Chris Ball,
linux-kernel, bcm
On Tue, Sep 17, 2013 at 05:05:11PM -0700, Christian Daudt wrote:
> @@ -980,6 +991,7 @@ config DEBUG_UART_PHYS
> default 0xffd82340 if ARCH_IOP13XX
> default 0xfff36000 if DEBUG_HIGHBANK_UART
> default 0xfffff700 if ARCH_IOP33X
> + default 0x3e000000 if DEBUG_BCM_KONA_UART
Please take a moment to think about this.
> depends on DEBUG_LL_UART_8250 || DEBUG_LL_UART_PL01X || \
> DEBUG_UART_8250 || DEBUG_UART_PL01X
>
> @@ -1035,6 +1047,7 @@ config DEBUG_UART_VIRT
> default 0xfef00003 if ARCH_IXP4XX && CPU_BIG_ENDIAN
> default 0xfefff700 if ARCH_IOP33X
> default 0xff003000 if DEBUG_U300_UART
> + default 0xfe300000 if DEBUG_BCM_KONA_UART
And this.
In particular, study the list which you are adding to and see whether
you can spot what you're doing wrong.
Thanks.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2] ARM: bcm: Add DEBUG_LL console support
@ 2013-10-03 7:21 ` Russell King - ARM Linux
0 siblings, 0 replies; 6+ messages in thread
From: Russell King - ARM Linux @ 2013-10-03 7:21 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Sep 17, 2013 at 05:05:11PM -0700, Christian Daudt wrote:
> @@ -980,6 +991,7 @@ config DEBUG_UART_PHYS
> default 0xffd82340 if ARCH_IOP13XX
> default 0xfff36000 if DEBUG_HIGHBANK_UART
> default 0xfffff700 if ARCH_IOP33X
> + default 0x3e000000 if DEBUG_BCM_KONA_UART
Please take a moment to think about this.
> depends on DEBUG_LL_UART_8250 || DEBUG_LL_UART_PL01X || \
> DEBUG_UART_8250 || DEBUG_UART_PL01X
>
> @@ -1035,6 +1047,7 @@ config DEBUG_UART_VIRT
> default 0xfef00003 if ARCH_IXP4XX && CPU_BIG_ENDIAN
> default 0xfefff700 if ARCH_IOP33X
> default 0xff003000 if DEBUG_U300_UART
> + default 0xfe300000 if DEBUG_BCM_KONA_UART
And this.
In particular, study the list which you are adding to and see whether
you can spot what you're doing wrong.
Thanks.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-10-03 7:27 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-09-18 0:05 [PATCH v2] ARM: bcm: Add DEBUG_LL console support Christian Daudt
2013-09-18 0:05 ` Christian Daudt
2013-10-03 6:22 ` Christian Daudt
2013-10-03 6:22 ` Christian Daudt
2013-10-03 7:21 ` Russell King - ARM Linux
2013-10-03 7:21 ` Russell King - ARM Linux
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.