Re: [PATCH v2 7/7] arm: mediatek: Add earlyprintk support for mt8127 & mt8135

From: Matthias Brugger
Date: Thu Sep 18 2014 - 09:01:35 EST




On 17/09/14 16:24, Joe.C wrote:
> From: "Joe.C" <yingjoe.chen@xxxxxxxxxxxx>
>
> Enable low-level debug for Mediatek mt8127 & mt8135 SoC.
>
> Signed-off-by: Joe.C <yingjoe.chen@xxxxxxxxxxxx>
> ---
> arch/arm/Kconfig.debug | 20 ++++++++++++++++++++
> 1 file changed, 20 insertions(+)
>
> diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
> index cfdd1c0..3888d17 100644
> --- a/arch/arm/Kconfig.debug
> +++ b/arch/arm/Kconfig.debug
> @@ -842,6 +842,22 @@ choice
> Say Y here if you want kernel low-level debugging support
> for Mediatek mt6589 based platforms on UART0.
>
> + config DEBUG_MT8127_UART0
> + bool "Mediatek mt8127 UART0"
> + depends on ARCH_MEDIATEK
> + select DEBUG_UART_8250
> + help
> + Say Y here if you want kernel low-level debugging support
> + for Mediatek mt8127 based platforms on UART0.
> +
> + config DEBUG_MT8135_UART3
> + bool "Mediatek mt8135 UART3"
> + depends on ARCH_MEDIATEK
> + select DEBUG_UART_8250
> + help
> + Say Y here if you want kernel low-level debugging support
> + for Mediatek mt8135 based platforms on UART3.
> +
> config DEBUG_VEXPRESS_UART0_DETECT
> bool "Autodetect UART0 on Versatile Express Cortex-A core tiles"
> depends on ARCH_VEXPRESS && CPU_CP15_MMU
> @@ -1084,6 +1100,8 @@ config DEBUG_UART_PHYS
> default 0x101f1000 if ARCH_VERSATILE
> default 0x101fb000 if DEBUG_NOMADIK_UART
> default 0x11006000 if DEBUG_MT6589_UART0
> + default 0x11002000 if DEBUG_MT8127_UART0
> + default 0x11009000 if DEBUG_MT8135_UART3

These lines are ordered by the address.

> default 0x16000000 if ARCH_INTEGRATOR
> default 0x18000300 if DEBUG_BCM_5301X
> default 0x1c090000 if DEBUG_VEXPRESS_UART0_RS1
> @@ -1147,6 +1165,8 @@ config DEBUG_UART_VIRT
> default 0xf0201000 if DEBUG_BCM2835
> default 0xf1000300 if DEBUG_BCM_5301X
> default 0xf1006000 if DEBUG_MT6589_UART0
> + default 0xf1002000 if DEBUG_MT8127_UART0
> + default 0xf1009000 if DEBUG_MT8135_UART3

Same here.
Apart from that the patches look fine to me.

Cheers,
Matthias

> default 0xf11f1000 if ARCH_VERSATILE
> default 0xf1600000 if ARCH_INTEGRATOR
> default 0xf1c28000 if DEBUG_SUNXI_UART0
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/