linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH V2] ARM: dts: BCM5301X: add missing LEDs for Buffalo WZR-900DHP
@ 2018-03-07 11:33 musashino.open
  2018-03-07 11:40 ` Rafał Miłecki
  2018-03-09 19:07 ` Florian Fainelli
  0 siblings, 2 replies; 3+ messages in thread
From: musashino.open @ 2018-03-07 11:33 UTC (permalink / raw)
  To: f.fainelli
  Cc: hauke, zajec5, jonmason, bcm-kernel-feedback-list, robh+dt,
	mark.rutland, linux, linux-arm-kernel, devicetree, linux-kernel,
	INAGAKI Hiroshi

From: INAGAKI Hiroshi <musashino.open@gmail.com>

Buffalo WZR-900DHP has 8 LEDs, but there is not LED definitions in the
dts and cannot configure these LEDs.
I Added missing LED definitions for WZR-900DHP.

Signed-off-by: INAGAKI Hiroshi <musashino.open@gmail.com>
---
 arch/arm/boot/dts/bcm47081-buffalo-wzr-900dhp.dts | 68 +++++++++++++++++++++++
 1 file changed, 68 insertions(+)

diff --git a/arch/arm/boot/dts/bcm47081-buffalo-wzr-900dhp.dts b/arch/arm/boot/dts/bcm47081-buffalo-wzr-900dhp.dts
index 8bef642..87ea6ba 100644
--- a/arch/arm/boot/dts/bcm47081-buffalo-wzr-900dhp.dts
+++ b/arch/arm/boot/dts/bcm47081-buffalo-wzr-900dhp.dts
@@ -35,6 +35,74 @@
 		       0x88000000 0x08000000>;
 	};
 
+	spi {
+		compatible = "spi-gpio";
+		num-chipselects = <1>;
+		gpio-sck = <&chipcommon 7 0>;
+		gpio-mosi = <&chipcommon 4 0>;
+		cs-gpios = <&chipcommon 6 0>;
+		#address-cells = <1>;
+		#size-cells = <0>;
+
+		hc595: gpio_spi@0 {
+			compatible = "fairchild,74hc595";
+			reg = <0>;
+			registers-number = <1>;
+			spi-max-frequency = <100000>;
+
+			gpio-controller;
+			#gpio-cells = <2>;
+
+		};
+	};
+
+	leds {
+		compatible = "gpio-leds";
+
+		usb {
+			label = "bcm53xx:green:usb";
+			gpios = <&hc595 0 GPIO_ACTIVE_HIGH>;
+		};
+
+		power0 {
+			label = "bcm53xx:green:power";
+			gpios = <&hc595 1 GPIO_ACTIVE_HIGH>;
+			linux,default-trigger = "default-on";
+		};
+
+		power1 {
+			label = "bcm53xx:red:power";
+			gpios = <&hc595 2 GPIO_ACTIVE_HIGH>;
+		};
+
+		router0 {
+			label = "bcm53xx:green:router";
+			gpios = <&hc595 3 GPIO_ACTIVE_HIGH>;
+			linux,default-trigger = "default-on";
+		};
+
+		router1 {
+			label = "bcm53xx:amber:router";
+			gpios = <&hc595 4 GPIO_ACTIVE_HIGH>;
+		};
+
+		wan {
+			label = "bcm53xx:green:wan";
+			gpios = <&hc595 5 GPIO_ACTIVE_HIGH>;
+			linux,default-trigger = "default-on";
+		};
+
+		wireless0 {
+			label = "bcm53xx:green:wireless";
+			gpios = <&hc595 6 GPIO_ACTIVE_HIGH>;
+		};
+
+		wireless1 {
+			label = "bcm53xx:amber:wireless";
+			gpios = <&hc595 7 GPIO_ACTIVE_HIGH>;
+		};
+	};
+
 	gpio-keys {
 		compatible = "gpio-keys";
 		#address-cells = <1>;
-- 
2.7.4

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

* Re: [PATCH V2] ARM: dts: BCM5301X: add missing LEDs for Buffalo WZR-900DHP
  2018-03-07 11:33 [PATCH V2] ARM: dts: BCM5301X: add missing LEDs for Buffalo WZR-900DHP musashino.open
@ 2018-03-07 11:40 ` Rafał Miłecki
  2018-03-09 19:07 ` Florian Fainelli
  1 sibling, 0 replies; 3+ messages in thread
From: Rafał Miłecki @ 2018-03-07 11:40 UTC (permalink / raw)
  To: musashino.open
  Cc: Florian Fainelli, Hauke Mehrtens, Jon Mason,
	bcm-kernel-feedback-list, Rob Herring, Mark Rutland,
	Russell King, linux-arm-kernel, devicetree,
	Linux Kernel Mailing List

On 7 March 2018 at 12:33,  <musashino.open@gmail.com> wrote:
> From: INAGAKI Hiroshi <musashino.open@gmail.com>
>
> Buffalo WZR-900DHP has 8 LEDs, but there is not LED definitions in the
> dts and cannot configure these LEDs.
> I Added missing LED definitions for WZR-900DHP.
>
> Signed-off-by: INAGAKI Hiroshi <musashino.open@gmail.com>

Nice work, thanks!

Reviewed-by: Rafał Miłecki <rafal@milecki.pl>

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

* Re: [PATCH V2] ARM: dts: BCM5301X: add missing LEDs for Buffalo WZR-900DHP
  2018-03-07 11:33 [PATCH V2] ARM: dts: BCM5301X: add missing LEDs for Buffalo WZR-900DHP musashino.open
  2018-03-07 11:40 ` Rafał Miłecki
@ 2018-03-09 19:07 ` Florian Fainelli
  1 sibling, 0 replies; 3+ messages in thread
From: Florian Fainelli @ 2018-03-09 19:07 UTC (permalink / raw)
  To: bcm-kernel-feedback-list, musashino.open
  Cc: hauke, zajec5, jonmason, robh+dt, mark.rutland, linux,
	linux-arm-kernel, devicetree, linux-kernel

On Wed,  7 Mar 2018 20:33:56 +0900, musashino.open@gmail.com wrote:
> From: INAGAKI Hiroshi <musashino.open@gmail.com>
> 
> Buffalo WZR-900DHP has 8 LEDs, but there is not LED definitions in the
> dts and cannot configure these LEDs.
> I Added missing LED definitions for WZR-900DHP.
> 
> Signed-off-by: INAGAKI Hiroshi <musashino.open@gmail.com>
> ---

Applied to devicetree/next, thanks!
--
Florian

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

end of thread, other threads:[~2018-03-09 19:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-07 11:33 [PATCH V2] ARM: dts: BCM5301X: add missing LEDs for Buffalo WZR-900DHP musashino.open
2018-03-07 11:40 ` Rafał Miłecki
2018-03-09 19:07 ` Florian Fainelli

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).