diff mbox

[1/2] ARM: dts: OMAP5: add PWM capability to timer nodes missing it

Message ID 1366240995-53538-1-git-send-email-s-anna@ti.com (mailing list archive)
State New, archived
Headers show

Commit Message

Suman Anna April 17, 2013, 11:23 p.m. UTC
OMAP5 has 6 timers (GPTimers 5, 6, 8 to 11) that are capable of PWM.
The PWM capability property is missing from the node definitions of
couple of timers, and this has been fixed.

Signed-off-by: Suman Anna <s-anna@ti.com>
---
 arch/arm/boot/dts/omap5.dtsi | 3 +++
 1 file changed, 3 insertions(+)

Comments

Hunter, Jon April 17, 2013, 11:56 p.m. UTC | #1
On 04/17/2013 06:23 PM, Suman Anna wrote:
> OMAP5 has 6 timers (GPTimers 5, 6, 8 to 11) that are capable of PWM.
> The PWM capability property is missing from the node definitions of
> couple of timers, and this has been fixed.
> 
> Signed-off-by: Suman Anna <s-anna@ti.com>
> ---
>  arch/arm/boot/dts/omap5.dtsi | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
> index 790bb2a..0d155f5 100644
> --- a/arch/arm/boot/dts/omap5.dtsi
> +++ b/arch/arm/boot/dts/omap5.dtsi
> @@ -422,6 +422,7 @@
>  			interrupts = <0 41 0x4>;
>  			ti,hwmods = "timer5";
>  			ti,timer-dsp;
> +			ti,timer-pwm;
>  		};
>  
>  		timer6: timer@4013a000 {
> @@ -458,6 +459,7 @@
>  			reg = <0x4803e000 0x80>;
>  			interrupts = <0 45 0x4>;
>  			ti,hwmods = "timer9";
> +			ti,timer-pwm;
>  		};
>  
>  		timer10: timer@48086000 {
> @@ -465,6 +467,7 @@
>  			reg = <0x48086000 0x80>;
>  			interrupts = <0 46 0x4>;
>  			ti,hwmods = "timer10";
> +			ti,timer-pwm;
>  		};
>  
>  		timer11: timer@48088000 {

Make sure you copy the linux-arm and device-tree mailing lists.

Acked-by: Jon Hunter <jon-hunter@ti.com>

Thanks!
Jon

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Suman Anna May 31, 2013, 10:17 p.m. UTC | #2
Benoit, Tony,
 
> On 04/17/2013 06:23 PM, Suman Anna wrote:

> > OMAP5 has 6 timers (GPTimers 5, 6, 8 to 11) that are capable of PWM.

> > The PWM capability property is missing from the node definitions of

> > couple of timers, and this has been fixed.

> >

> > Signed-off-by: Suman Anna <s-anna@ti.com>

> > ---

> >  arch/arm/boot/dts/omap5.dtsi | 3 +++

> >  1 file changed, 3 insertions(+)

> >

> > diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi

> > index 790bb2a..0d155f5 100644

> > --- a/arch/arm/boot/dts/omap5.dtsi

> > +++ b/arch/arm/boot/dts/omap5.dtsi

> > @@ -422,6 +422,7 @@

> >  			interrupts = <0 41 0x4>;

> >  			ti,hwmods = "timer5";

> >  			ti,timer-dsp;

> > +			ti,timer-pwm;

> >  		};

> >

> >  		timer6: timer@4013a000 {

> > @@ -458,6 +459,7 @@

> >  			reg = <0x4803e000 0x80>;

> >  			interrupts = <0 45 0x4>;

> >  			ti,hwmods = "timer9";

> > +			ti,timer-pwm;

> >  		};

> >

> >  		timer10: timer@48086000 {

> > @@ -465,6 +467,7 @@

> >  			reg = <0x48086000 0x80>;

> >  			interrupts = <0 46 0x4>;

> >  			ti,hwmods = "timer10";

> > +			ti,timer-pwm;

> >  		};

> >

> >  		timer11: timer@48088000 {

> 

> Make sure you copy the linux-arm and device-tree mailing lists.

> 

> Acked-by: Jon Hunter <jon-hunter@ti.com>

> 


Can you include this patch in your next fixes pull request (if you haven't
already done so)? The other patch in the series is a feature change, but
this is a fix, and can go into 3.10 itself.

Regards
Suman
Benoit Cousson June 3, 2013, 12:55 p.m. UTC | #3
Hi Suman,

On 06/01/2013 12:17 AM, Anna, Suman wrote:
> Benoit, Tony,
>  
>> On 04/17/2013 06:23 PM, Suman Anna wrote:
>>> OMAP5 has 6 timers (GPTimers 5, 6, 8 to 11) that are capable of PWM.
>>> The PWM capability property is missing from the node definitions of
>>> couple of timers, and this has been fixed.
>>>
>>> Signed-off-by: Suman Anna <s-anna@ti.com>
>>> ---
>>>  arch/arm/boot/dts/omap5.dtsi | 3 +++
>>>  1 file changed, 3 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
>>> index 790bb2a..0d155f5 100644
>>> --- a/arch/arm/boot/dts/omap5.dtsi
>>> +++ b/arch/arm/boot/dts/omap5.dtsi
>>> @@ -422,6 +422,7 @@
>>>  			interrupts = <0 41 0x4>;
>>>  			ti,hwmods = "timer5";
>>>  			ti,timer-dsp;
>>> +			ti,timer-pwm;
>>>  		};
>>>
>>>  		timer6: timer@4013a000 {
>>> @@ -458,6 +459,7 @@
>>>  			reg = <0x4803e000 0x80>;
>>>  			interrupts = <0 45 0x4>;
>>>  			ti,hwmods = "timer9";
>>> +			ti,timer-pwm;
>>>  		};
>>>
>>>  		timer10: timer@48086000 {
>>> @@ -465,6 +467,7 @@
>>>  			reg = <0x48086000 0x80>;
>>>  			interrupts = <0 46 0x4>;
>>>  			ti,hwmods = "timer10";
>>> +			ti,timer-pwm;
>>>  		};
>>>
>>>  		timer11: timer@48088000 {
>>
>> Make sure you copy the linux-arm and device-tree mailing lists.
>>
>> Acked-by: Jon Hunter <jon-hunter@ti.com>
>>
> 
> Can you include this patch in your next fixes pull request (if you haven't
> already done so)? The other patch in the series is a feature change, but
> this is a fix, and can go into 3.10 itself.

I've just sent a pull request with two fixes to Tony. Since -rc4 is
already there, I'll update it with this fix.

Tony,

I will resend a new pull-request rebased on -rc4 that will include this
new fix.

Regards,
Benoit



--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
index 790bb2a..0d155f5 100644
--- a/arch/arm/boot/dts/omap5.dtsi
+++ b/arch/arm/boot/dts/omap5.dtsi
@@ -422,6 +422,7 @@ 
 			interrupts = <0 41 0x4>;
 			ti,hwmods = "timer5";
 			ti,timer-dsp;
+			ti,timer-pwm;
 		};
 
 		timer6: timer@4013a000 {
@@ -458,6 +459,7 @@ 
 			reg = <0x4803e000 0x80>;
 			interrupts = <0 45 0x4>;
 			ti,hwmods = "timer9";
+			ti,timer-pwm;
 		};
 
 		timer10: timer@48086000 {
@@ -465,6 +467,7 @@ 
 			reg = <0x48086000 0x80>;
 			interrupts = <0 46 0x4>;
 			ti,hwmods = "timer10";
+			ti,timer-pwm;
 		};
 
 		timer11: timer@48088000 {