public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: apq8096-db820c: disable uart0 by default
@ 2018-06-12 14:48 Srinivas Kandagatla
  2018-06-12 15:49 ` Bjorn Andersson
  2018-06-13  7:22 ` Andy Gross
  0 siblings, 2 replies; 3+ messages in thread
From: Srinivas Kandagatla @ 2018-06-12 14:48 UTC (permalink / raw)
  To: andy.gross
  Cc: mark.rutland, linux-arm-msm, david.brown, bjorn.andersson,
	linux-kernel, Srinivas Kandagatla

Access to UART0 is disabled by bootloaders. By leaving it enabled by
default would reboot the board.
Disable this for now, this would alteast give a board which boots.

Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
---
 arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
index 0f829db33efe..4d5ef01f43a3 100644
--- a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
+++ b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
@@ -75,7 +75,7 @@
 
 		serial@75b1000 {
 			label = "LS-UART0";
-			status = "okay";
+			status = "disabled";
 			pinctrl-names = "default", "sleep";
 			pinctrl-0 = <&blsp2_uart2_4pins_default>;
 			pinctrl-1 = <&blsp2_uart2_4pins_sleep>;
-- 
2.16.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: dts: apq8096-db820c: disable uart0 by default
  2018-06-12 14:48 [PATCH] arm64: dts: apq8096-db820c: disable uart0 by default Srinivas Kandagatla
@ 2018-06-12 15:49 ` Bjorn Andersson
  2018-06-13  7:22 ` Andy Gross
  1 sibling, 0 replies; 3+ messages in thread
From: Bjorn Andersson @ 2018-06-12 15:49 UTC (permalink / raw)
  To: Srinivas Kandagatla
  Cc: andy.gross, mark.rutland, linux-arm-msm, david.brown,
	linux-kernel

On Tue 12 Jun 07:48 PDT 2018, Srinivas Kandagatla wrote:

> Access to UART0 is disabled by bootloaders. By leaving it enabled by
> default would reboot the board.
> Disable this for now, this would alteast give a board which boots.
> 
> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>

Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>

Regards,
Bjorn

> ---
>  arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
> index 0f829db33efe..4d5ef01f43a3 100644
> --- a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
> +++ b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
> @@ -75,7 +75,7 @@
>  
>  		serial@75b1000 {
>  			label = "LS-UART0";
> -			status = "okay";
> +			status = "disabled";
>  			pinctrl-names = "default", "sleep";
>  			pinctrl-0 = <&blsp2_uart2_4pins_default>;
>  			pinctrl-1 = <&blsp2_uart2_4pins_sleep>;
> -- 
> 2.16.2
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: dts: apq8096-db820c: disable uart0 by default
  2018-06-12 14:48 [PATCH] arm64: dts: apq8096-db820c: disable uart0 by default Srinivas Kandagatla
  2018-06-12 15:49 ` Bjorn Andersson
@ 2018-06-13  7:22 ` Andy Gross
  1 sibling, 0 replies; 3+ messages in thread
From: Andy Gross @ 2018-06-13  7:22 UTC (permalink / raw)
  To: Srinivas Kandagatla
  Cc: mark.rutland, linux-arm-msm, david.brown, bjorn.andersson,
	linux-kernel

On Tue, Jun 12, 2018 at 03:48:06PM +0100, Srinivas Kandagatla wrote:
> Access to UART0 is disabled by bootloaders. By leaving it enabled by
> default would reboot the board.
> Disable this for now, this would alteast give a board which boots.
> 
> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>

Thanks for this patch.  I'll queue it up for a fixes-for-rc1


Andy

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-06-13  7:22 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-06-12 14:48 [PATCH] arm64: dts: apq8096-db820c: disable uart0 by default Srinivas Kandagatla
2018-06-12 15:49 ` Bjorn Andersson
2018-06-13  7:22 ` Andy Gross

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox