[PATCH v2 3/7] ARM: dts: kirkwood: gpio-leds fixes for linkstation ls-wxl/wsxl

Andrew Lunn andrew at lunn.ch
Wed Jan 20 16:30:41 PST 2016


On Thu, Jan 21, 2016 at 12:33:54AM +0900, Roger Shimizu wrote:
> Make all leds initially to "off" state except power indicator, so pins below
> change from active_low to active_high:
>   - gpio-leds: "lswxl:red:func" pin
>   - gpio-leds: "lswxl:red:hdderr{0,1}" pin

Hi Roger

So the patch is correct, but the changelog is not really correct. Better to say:

The GPIOs controlling the LEDs are active high, not low.

Please resend, and add my:

Reviewd-by: Andrew Lunn <andrew at lunn.ch>

	    Andrew

> 
> Fixes: e54e4b1b622e ("ARM: dts: add buffalo linkstation ls-wxl/wsxl")
> Signed-off-by: Roger Shimizu <rogershimizu at gmail.com>
> ---
>  arch/arm/boot/dts/kirkwood-lswxl.dts | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/kirkwood-lswxl.dts b/arch/arm/boot/dts/kirkwood-lswxl.dts
> index 0e93f6d19259..f4700a60065b 100644
> --- a/arch/arm/boot/dts/kirkwood-lswxl.dts
> +++ b/arch/arm/boot/dts/kirkwood-lswxl.dts
> @@ -206,18 +206,17 @@
>  
>  		led at 5 {
>  			label = "lswxl:red:func";
> -			gpios = <&gpio1 2 GPIO_ACTIVE_LOW>;
> -			default-state = "keep";
> +			gpios = <&gpio1 2 GPIO_ACTIVE_HIGH>;
>  		};
>  
>  		led at 6 {
>  			label = "lswxl:red:hdderr0";
> -			gpios = <&gpio0 8 GPIO_ACTIVE_LOW>;
> +			gpios = <&gpio0 8 GPIO_ACTIVE_HIGH>;
>  		};
>  
>  		led at 7 {
>  			label = "lswxl:red:hdderr1";
> -			gpios = <&gpio1 14 GPIO_ACTIVE_LOW>;
> +			gpios = <&gpio1 14 GPIO_ACTIVE_HIGH>;
>  		};
>  	};
>  
> -- 
> 2.1.4
> 



More information about the linux-arm-kernel mailing list