* [PATCH 1/2] ARM: imx27: Add PWM0 to device tree
@ 2013-03-29  8:23 Gwenhael Goavec-Merou
  2013-03-29  8:23 ` [PATCH 2/2] ARM: apf27dev: add PWM0 support Gwenhael Goavec-Merou
  2013-03-30 10:32 ` [PATCH 1/2] ARM: imx27: Add PWM0 to device tree Sascha Hauer
  0 siblings, 2 replies; 3+ messages in thread
From: Gwenhael Goavec-Merou @ 2013-03-29  8:23 UTC (permalink / raw)
  To: linux-arm-kernel
Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@armadeus.com>
---
 arch/arm/boot/dts/imx27.dtsip | 9 +++++++++
 1 file changed, 9 insertions(+)
diff --git a/arch/arm/boot/dts/imx27.dtsip b/arch/arm/boot/dts/imx27.dtsip
index ff4bd48..d4196d1 100644
--- a/arch/arm/boot/dts/imx27.dtsip
+++ b/arch/arm/boot/dts/imx27.dtsip
@@ -89,6 +89,15 @@
 				clock-names = "ipg", "per";
 			};
 
+			pwm0: pwm at 10006000 {
+				compatible = "fsl,imx27-pwm";
+				reg = <0x10006000 0x1000>;
+				interrupts = <23>;
+				clocks = <&clks 34>, <&clks 61>;
+				clock-names = "ipg", "per";
+				status = "disabled";
+			};
+
 			uart1: serial at 1000a000 {
 				compatible = "fsl,imx27-uart", "fsl,imx21-uart";
 				reg = <0x1000a000 0x1000>;
-- 
1.8.1.5
^ permalink raw reply related	[flat|nested] 3+ messages in thread- * [PATCH 2/2] ARM: apf27dev: add PWM0 support
  2013-03-29  8:23 [PATCH 1/2] ARM: imx27: Add PWM0 to device tree Gwenhael Goavec-Merou
@ 2013-03-29  8:23 ` Gwenhael Goavec-Merou
  2013-03-30 10:32 ` [PATCH 1/2] ARM: imx27: Add PWM0 to device tree Sascha Hauer
  1 sibling, 0 replies; 3+ messages in thread
From: Gwenhael Goavec-Merou @ 2013-03-29  8:23 UTC (permalink / raw)
  To: linux-arm-kernel
Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@armadeus.com>
---
 arch/arm/boot/dts/imx27-apf27dev.dtsp | 4 ++++
 1 file changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts/imx27-apf27dev.dtsp b/arch/arm/boot/dts/imx27-apf27dev.dtsp
index 65d1853..3e60eba 100644
--- a/arch/arm/boot/dts/imx27-apf27dev.dtsp
+++ b/arch/arm/boot/dts/imx27-apf27dev.dtsp
@@ -58,3 +58,7 @@
 &i2c2 {
 	status = "okay";
 };
+
+&pwm0 {
+	status = "okay";
+};
-- 
1.8.1.5
^ permalink raw reply related	[flat|nested] 3+ messages in thread
- * [PATCH 1/2] ARM: imx27: Add PWM0 to device tree
  2013-03-29  8:23 [PATCH 1/2] ARM: imx27: Add PWM0 to device tree Gwenhael Goavec-Merou
  2013-03-29  8:23 ` [PATCH 2/2] ARM: apf27dev: add PWM0 support Gwenhael Goavec-Merou
@ 2013-03-30 10:32 ` Sascha Hauer
  1 sibling, 0 replies; 3+ messages in thread
From: Sascha Hauer @ 2013-03-30 10:32 UTC (permalink / raw)
  To: linux-arm-kernel
On Fri, Mar 29, 2013 at 09:23:38AM +0100, Gwenhael Goavec-Merou wrote:
> 
> Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@armadeus.com>
> ---
>  arch/arm/boot/dts/imx27.dtsip | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/imx27.dtsip b/arch/arm/boot/dts/imx27.dtsip
> index ff4bd48..d4196d1 100644
> --- a/arch/arm/boot/dts/imx27.dtsip
> +++ b/arch/arm/boot/dts/imx27.dtsip
> @@ -89,6 +89,15 @@
>  				clock-names = "ipg", "per";
>  			};
>  
> +			pwm0: pwm at 10006000 {
> +				compatible = "fsl,imx27-pwm";
> +				reg = <0x10006000 0x1000>;
> +				interrupts = <23>;
> +				clocks = <&clks 34>, <&clks 61>;
> +				clock-names = "ipg", "per";
> +				status = "disabled";
> +			};
A pwm does not do anything on its own when it's not being used as
backlight, led or something else. So I wonder if it's worth it adding
status = "disabled"; here. Also, what's the result of enabling it in
your board like in patch 2/2? I suppose nothing will happen, right?
Sascha
> +
>  			uart1: serial at 1000a000 {
>  				compatible = "fsl,imx27-uart", "fsl,imx21-uart";
>  				reg = <0x1000a000 0x1000>;
> -- 
> 1.8.1.5
> 
> 
-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-03-30 10:32 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-29  8:23 [PATCH 1/2] ARM: imx27: Add PWM0 to device tree Gwenhael Goavec-Merou
2013-03-29  8:23 ` [PATCH 2/2] ARM: apf27dev: add PWM0 support Gwenhael Goavec-Merou
2013-03-30 10:32 ` [PATCH 1/2] ARM: imx27: Add PWM0 to device tree Sascha Hauer
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).