diff mbox

ARM: ux500: disable msp2 node on Snowball

Message ID 1409661681-31490-1-git-send-email-linus.walleij@linaro.org
State Accepted
Commit dbd366fdf2650531045ce47833b1efcdeec4dd86
Headers show

Commit Message

Linus Walleij Sept. 2, 2014, 12:41 p.m. UTC
Analogous to commit 8858d88a25142544843869f0cd3e6654aa7b4aec
that fixed commit 70b41abc151f9
"ARM: ux500: move MSP pin control to the device tree"
accidentally activated MSP2, giving rise to a boot scroll
scream as the kernel attempts to probe a driver for it and
fails to obtain DMA channel 14.

For some reason I forgot to fix this on the Snowball. Fix
this up by marking the node disabled again.

Cc: Lee Jones <lee.jones@linaro.org>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
ARM SoC folks: please apply this patch directly to fixes for
v3.17, I have no other pending Ux500 fixes.
---
 arch/arm/boot/dts/ste-snowball.dts | 1 -
 1 file changed, 1 deletion(-)

Comments

Lee Jones Sept. 2, 2014, 1:19 p.m. UTC | #1
On Tue, 02 Sep 2014, Linus Walleij wrote:

> Analogous to commit 8858d88a25142544843869f0cd3e6654aa7b4aec
> that fixed commit 70b41abc151f9
> "ARM: ux500: move MSP pin control to the device tree"
> accidentally activated MSP2, giving rise to a boot scroll
> scream as the kernel attempts to probe a driver for it and
> fails to obtain DMA channel 14.
> 
> For some reason I forgot to fix this on the Snowball. Fix
> this up by marking the node disabled again.
> 
> Cc: Lee Jones <lee.jones@linaro.org>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> ARM SoC folks: please apply this patch directly to fixes for
> v3.17, I have no other pending Ux500 fixes.
> ---
>  arch/arm/boot/dts/ste-snowball.dts | 1 -
>  1 file changed, 1 deletion(-)

Ah I see.  You fixed the HREF back in Feb, but I guess Snowball didn't
benefit from it.

Acked-by: Lee Jones <lee.jones@linaro.org>
 
> diff --git a/arch/arm/boot/dts/ste-snowball.dts b/arch/arm/boot/dts/ste-snowball.dts
> index 4a2000c620ad..3e97a669f15e 100644
> --- a/arch/arm/boot/dts/ste-snowball.dts
> +++ b/arch/arm/boot/dts/ste-snowball.dts
> @@ -116,7 +116,6 @@
>  		msp2: msp@80117000 {
>  			pinctrl-names = "default";
>  			pinctrl-0 = <&msp2_default_mode>;
> -			status = "okay";
>  		};
>  
>  		msp3: msp@80125000 {
Kevin Hilman Sept. 2, 2014, 6:18 p.m. UTC | #2
Linus Walleij <linus.walleij@linaro.org> writes:

> Analogous to commit 8858d88a25142544843869f0cd3e6654aa7b4aec
> that fixed commit 70b41abc151f9
> "ARM: ux500: move MSP pin control to the device tree"
> accidentally activated MSP2, giving rise to a boot scroll
> scream as the kernel attempts to probe a driver for it and
> fails to obtain DMA channel 14.
>
> For some reason I forgot to fix this on the Snowball. Fix
> this up by marking the node disabled again.
>
> Cc: Lee Jones <lee.jones@linaro.org>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> ARM SoC folks: please apply this patch directly to fixes for
> v3.17, I have no other pending Ux500 fixes.

Tested on snowball, and applied to fixes.

Thanks,

Kevin
diff mbox

Patch

diff --git a/arch/arm/boot/dts/ste-snowball.dts b/arch/arm/boot/dts/ste-snowball.dts
index 4a2000c620ad..3e97a669f15e 100644
--- a/arch/arm/boot/dts/ste-snowball.dts
+++ b/arch/arm/boot/dts/ste-snowball.dts
@@ -116,7 +116,6 @@ 
 		msp2: msp@80117000 {
 			pinctrl-names = "default";
 			pinctrl-0 = <&msp2_default_mode>;
-			status = "okay";
 		};
 
 		msp3: msp@80125000 {