All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm: kirkwood: readynas duo v2: Add GMT G762 Fan Controller DT bindings
@ 2013-07-01 20:23 Arnaud Ebalard
  2013-07-01 20:28 ` Thomas Petazzoni
  0 siblings, 1 reply; 4+ messages in thread
From: Arnaud Ebalard @ 2013-07-01 20:23 UTC (permalink / raw)
  To: linux-arm-kernel


Now that GMT G762 PWM Fan Controller is available upstream, let's add
missing bindings in Netgear ReadyNAS Duo v2 .dts file. Let's also take
the opportunity to fix space issues at the end of the file.

Signed-off-by: Arnaud Ebalard <arno@natisbad.org>
---
Hi Jason and Andrew,

This one applies and has been tested against Stephen Rothwell tree.
Can you make it flow to Linus tree during the merge window? Thanks
in advance.

Cheers,

a+

 .../boot/dts/kirkwood-netgear_readynas_duo_v2.dts  | 54 +++++++++++++++-------
 1 file changed, 37 insertions(+), 17 deletions(-)

diff --git a/arch/arm/boot/dts/kirkwood-netgear_readynas_duo_v2.dts b/arch/arm/boot/dts/kirkwood-netgear_readynas_duo_v2.dts
index ad6ade7..796a307 100644
--- a/arch/arm/boot/dts/kirkwood-netgear_readynas_duo_v2.dts
+++ b/arch/arm/boot/dts/kirkwood-netgear_readynas_duo_v2.dts
@@ -52,6 +52,17 @@
 			};
 		};
 
+		clocks {
+		       #address-cells = <1>;
+		       #size-cells = <0>;
+
+		       g762_clk: fixedclk {
+				 compatible = "fixed-clock";
+				 #clock-cells = <0>;
+				 clock-frequency = <8192>;
+		       };
+		};
+
 		i2c at 11000 {
 			status = "okay";
 
@@ -59,6 +70,15 @@
 				compatible = "ricoh,rs5c372a";
 				reg = <0x32>;
 			};
+
+			g762: g762 at 3e {
+				compatible = "gmt,g762";
+				reg = <0x3e>;
+				clocks = <&g762_clk>; /* input clock */
+				fan_gear_mode = <0>;
+				fan_startv = <1>;
+				pwm_polarity = <0>;
+			};
 		};
 
 		serial at 12000 {
@@ -166,21 +186,21 @@
 		};
 	};
 
-        regulators {
-                compatible = "simple-bus";
-                #address-cells = <1>;
-                #size-cells = <0>;
-
-                usb_power: regulator at 1 {
-                        compatible = "regulator-fixed";
-                        reg = <1>;
-                        regulator-name = "USB 3.0 Power";
-                        regulator-min-microvolt = <5000000>;
-                        regulator-max-microvolt = <5000000>;
-                        enable-active-high;
-                        regulator-always-on;
-                        regulator-boot-on;
-                        gpio = <&gpio1 14 0>;
-                };
-        };
+	regulators {
+		compatible = "simple-bus";
+		#address-cells = <1>;
+		#size-cells = <0>;
+
+		usb_power: regulator at 1 {
+			compatible = "regulator-fixed";
+			reg = <1>;
+			regulator-name = "USB 3.0 Power";
+			regulator-min-microvolt = <5000000>;
+			regulator-max-microvolt = <5000000>;
+			enable-active-high;
+			regulator-always-on;
+			regulator-boot-on;
+			gpio = <&gpio1 14 0>;
+		};
+	};
 };
-- 
1.8.3.1

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

* [PATCH] arm: kirkwood: readynas duo v2: Add GMT G762 Fan Controller DT bindings
  2013-07-01 20:23 [PATCH] arm: kirkwood: readynas duo v2: Add GMT G762 Fan Controller DT bindings Arnaud Ebalard
@ 2013-07-01 20:28 ` Thomas Petazzoni
  2013-07-01 20:31   ` Arnaud Ebalard
  0 siblings, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2013-07-01 20:28 UTC (permalink / raw)
  To: linux-arm-kernel

Dear Arnaud Ebalard,

On Mon, 01 Jul 2013 22:23:14 +0200, Arnaud Ebalard wrote:

> -        regulators {
> -                compatible = "simple-bus";
> -                #address-cells = <1>;
> -                #size-cells = <0>;
> -
> -                usb_power: regulator at 1 {
> -                        compatible = "regulator-fixed";
> -                        reg = <1>;
> -                        regulator-name = "USB 3.0 Power";
> -                        regulator-min-microvolt = <5000000>;
> -                        regulator-max-microvolt = <5000000>;
> -                        enable-active-high;
> -                        regulator-always-on;
> -                        regulator-boot-on;
> -                        gpio = <&gpio1 14 0>;
> -                };
> -        };
> +	regulators {
> +		compatible = "simple-bus";
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +
> +		usb_power: regulator at 1 {
> +			compatible = "regulator-fixed";
> +			reg = <1>;
> +			regulator-name = "USB 3.0 Power";
> +			regulator-min-microvolt = <5000000>;
> +			regulator-max-microvolt = <5000000>;
> +			enable-active-high;
> +			regulator-always-on;
> +			regulator-boot-on;
> +			gpio = <&gpio1 14 0>;
> +		};
> +	};
>  };

Those white-space changes are not related to the addition of the G762
Device Tree informations.

Thanks,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

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

* [PATCH] arm: kirkwood: readynas duo v2: Add GMT G762 Fan Controller DT bindings
  2013-07-01 20:28 ` Thomas Petazzoni
@ 2013-07-01 20:31   ` Arnaud Ebalard
  2013-07-01 21:24     ` Jason Cooper
  0 siblings, 1 reply; 4+ messages in thread
From: Arnaud Ebalard @ 2013-07-01 20:31 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Thomas,

Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

> On Mon, 01 Jul 2013 22:23:14 +0200, Arnaud Ebalard wrote:
>
>> -        regulators {
>> -                compatible = "simple-bus";
>> -                #address-cells = <1>;
>> -                #size-cells = <0>;
>> -
>> -                usb_power: regulator at 1 {
>> -                        compatible = "regulator-fixed";
>> -                        reg = <1>;
>> -                        regulator-name = "USB 3.0 Power";
>> -                        regulator-min-microvolt = <5000000>;
>> -                        regulator-max-microvolt = <5000000>;
>> -                        enable-active-high;
>> -                        regulator-always-on;
>> -                        regulator-boot-on;
>> -                        gpio = <&gpio1 14 0>;
>> -                };
>> -        };
>> +	regulators {
>> +		compatible = "simple-bus";
>> +		#address-cells = <1>;
>> +		#size-cells = <0>;
>> +
>> +		usb_power: regulator at 1 {
>> +			compatible = "regulator-fixed";
>> +			reg = <1>;
>> +			regulator-name = "USB 3.0 Power";
>> +			regulator-min-microvolt = <5000000>;
>> +			regulator-max-microvolt = <5000000>;
>> +			enable-active-high;
>> +			regulator-always-on;
>> +			regulator-boot-on;
>> +			gpio = <&gpio1 14 0>;
>> +		};
>> +	};
>>  };
>
> Those white-space changes are not related to the addition of the G762
> Device Tree informations.

Should I make two unrelated patches (one for G762 and another cosmetic
one) or simply let the spaces as they are in .dts?

Cheers,

a+

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

* [PATCH] arm: kirkwood: readynas duo v2: Add GMT G762 Fan Controller DT bindings
  2013-07-01 20:31   ` Arnaud Ebalard
@ 2013-07-01 21:24     ` Jason Cooper
  0 siblings, 0 replies; 4+ messages in thread
From: Jason Cooper @ 2013-07-01 21:24 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Jul 01, 2013 at 10:31:54PM +0200, Arnaud Ebalard wrote:
> Hi Thomas,
> 
> Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:
> 
> > On Mon, 01 Jul 2013 22:23:14 +0200, Arnaud Ebalard wrote:
> >
> >> -        regulators {
> >> -                compatible = "simple-bus";
> >> -                #address-cells = <1>;
> >> -                #size-cells = <0>;
> >> -
> >> -                usb_power: regulator at 1 {
> >> -                        compatible = "regulator-fixed";
> >> -                        reg = <1>;
> >> -                        regulator-name = "USB 3.0 Power";
> >> -                        regulator-min-microvolt = <5000000>;
> >> -                        regulator-max-microvolt = <5000000>;
> >> -                        enable-active-high;
> >> -                        regulator-always-on;
> >> -                        regulator-boot-on;
> >> -                        gpio = <&gpio1 14 0>;
> >> -                };
> >> -        };
> >> +	regulators {
> >> +		compatible = "simple-bus";
> >> +		#address-cells = <1>;
> >> +		#size-cells = <0>;
> >> +
> >> +		usb_power: regulator at 1 {
> >> +			compatible = "regulator-fixed";
> >> +			reg = <1>;
> >> +			regulator-name = "USB 3.0 Power";
> >> +			regulator-min-microvolt = <5000000>;
> >> +			regulator-max-microvolt = <5000000>;
> >> +			enable-active-high;
> >> +			regulator-always-on;
> >> +			regulator-boot-on;
> >> +			gpio = <&gpio1 14 0>;
> >> +		};
> >> +	};
> >>  };
> >
> > Those white-space changes are not related to the addition of the G762
> > Device Tree informations.
> 
> Should I make two unrelated patches (one for G762 and another cosmetic
> one) or simply let the spaces as they are in .dts?

Leave them as they are.  I'm intending to do a cleanup series for all
the whitespace issues in the dts files.

thx,

Jason.

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

end of thread, other threads:[~2013-07-01 21:24 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-01 20:23 [PATCH] arm: kirkwood: readynas duo v2: Add GMT G762 Fan Controller DT bindings Arnaud Ebalard
2013-07-01 20:28 ` Thomas Petazzoni
2013-07-01 20:31   ` Arnaud Ebalard
2013-07-01 21:24     ` Jason Cooper

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.