2017-06-09 07:38:42

by Fabrice Gasnier

[permalink] [raw]
Subject: [PATCH 0/2] Fix STM32 timers dt-bindings

STM32 timers clock names has been updated during driver/DT binding review:
https://lkml.org/lkml/2016/12/13/718

STM32 timers driver has been fixed with these remarks. It looks like it's
been missed on DT binding part.
Update DT binding doc to reflect this as well.

Fabrice Gasnier (2):
dt-bindings: mfd: update STM32 timers clock names
dt-bindings: iio: update STM32 timers clock names

Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt | 2 +-
Documentation/devicetree/bindings/mfd/stm32-timers.txt | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)

--
1.9.1


2017-06-09 07:38:46

by Fabrice Gasnier

[permalink] [raw]
Subject: [PATCH 2/2] dt-bindings: iio: update STM32 timers clock names

Clock name has been updated during driver/DT binding review:
https://lkml.org/lkml/2016/12/13/718
Update DT binding doc to reflect this.

Signed-off-by: Fabrice Gasnier <[email protected]>
---
Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt b/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt
index 55a653d..6abc755 100644
--- a/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt
+++ b/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt
@@ -14,7 +14,7 @@ Example:
compatible = "st,stm32-timers";
reg = <0x40010000 0x400>;
clocks = <&rcc 0 160>;
- clock-names = "clk_int";
+ clock-names = "int";

timer@0 {
compatible = "st,stm32-timer-trigger";
--
1.9.1

2017-06-09 07:38:57

by Fabrice Gasnier

[permalink] [raw]
Subject: [PATCH 1/2] dt-bindings: mfd: update STM32 timers clock names

Clock name has been updated during driver/DT binding review:
https://lkml.org/lkml/2016/12/13/718
Update DT binding doc to reflect this.

Signed-off-by: Fabrice Gasnier <[email protected]>
---
Documentation/devicetree/bindings/mfd/stm32-timers.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/mfd/stm32-timers.txt b/Documentation/devicetree/bindings/mfd/stm32-timers.txt
index bbd083f..1db6e00 100644
--- a/Documentation/devicetree/bindings/mfd/stm32-timers.txt
+++ b/Documentation/devicetree/bindings/mfd/stm32-timers.txt
@@ -31,7 +31,7 @@ Example:
compatible = "st,stm32-timers";
reg = <0x40010000 0x400>;
clocks = <&rcc 0 160>;
- clock-names = "clk_int";
+ clock-names = "int";

pwm {
compatible = "st,stm32-pwm";
--
1.9.1

2017-06-09 07:43:24

by Benjamin Gaignard

[permalink] [raw]
Subject: Re: [PATCH 1/2] dt-bindings: mfd: update STM32 timers clock names

2017-06-09 9:37 GMT+02:00 Fabrice Gasnier <[email protected]>:
> Clock name has been updated during driver/DT binding review:
> https://lkml.org/lkml/2016/12/13/718
> Update DT binding doc to reflect this.
>
> Signed-off-by: Fabrice Gasnier <[email protected]>
> ---
> Documentation/devicetree/bindings/mfd/stm32-timers.txt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/mfd/stm32-timers.txt b/Documentation/devicetree/bindings/mfd/stm32-timers.txt
> index bbd083f..1db6e00 100644
> --- a/Documentation/devicetree/bindings/mfd/stm32-timers.txt
> +++ b/Documentation/devicetree/bindings/mfd/stm32-timers.txt
> @@ -31,7 +31,7 @@ Example:
> compatible = "st,stm32-timers";
> reg = <0x40010000 0x400>;
> clocks = <&rcc 0 160>;
> - clock-names = "clk_int";
> + clock-names = "int";
>
> pwm {
> compatible = "st,stm32-pwm";
> --
> 1.9.1
>

Acked-by: Benjamin Gaignard <[email protected]>

2017-06-09 07:43:46

by Benjamin Gaignard

[permalink] [raw]
Subject: Re: [PATCH 2/2] dt-bindings: iio: update STM32 timers clock names

2017-06-09 9:37 GMT+02:00 Fabrice Gasnier <[email protected]>:
> Clock name has been updated during driver/DT binding review:
> https://lkml.org/lkml/2016/12/13/718
> Update DT binding doc to reflect this.
>
> Signed-off-by: Fabrice Gasnier <[email protected]>
> ---
> Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt b/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt
> index 55a653d..6abc755 100644
> --- a/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt
> +++ b/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt
> @@ -14,7 +14,7 @@ Example:
> compatible = "st,stm32-timers";
> reg = <0x40010000 0x400>;
> clocks = <&rcc 0 160>;
> - clock-names = "clk_int";
> + clock-names = "int";
>
> timer@0 {
> compatible = "st,stm32-timer-trigger";
> --
> 1.9.1
>
Acked-by: Benjamin Gaignard <[email protected]>

2017-06-11 11:15:46

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH 1/2] dt-bindings: mfd: update STM32 timers clock names

On Fri, 9 Jun 2017 09:43:21 +0200
Benjamin Gaignard <[email protected]> wrote:

> 2017-06-09 9:37 GMT+02:00 Fabrice Gasnier <[email protected]>:
> > Clock name has been updated during driver/DT binding review:
> > https://lkml.org/lkml/2016/12/13/718
> > Update DT binding doc to reflect this.
> >
> > Signed-off-by: Fabrice Gasnier <[email protected]>
Should have a fixes tag as we should probably be pushing this
to stable... Rob / Mark - up to you guys on both of these for
the path they take (assuming Lee is happy with patch!)

Fixes: d0f949e220fd ("mfd: Add STM32 Timers driver")
I think...
> > ---
> > Documentation/devicetree/bindings/mfd/stm32-timers.txt | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/Documentation/devicetree/bindings/mfd/stm32-timers.txt b/Documentation/devicetree/bindings/mfd/stm32-timers.txt
> > index bbd083f..1db6e00 100644
> > --- a/Documentation/devicetree/bindings/mfd/stm32-timers.txt
> > +++ b/Documentation/devicetree/bindings/mfd/stm32-timers.txt
> > @@ -31,7 +31,7 @@ Example:
> > compatible = "st,stm32-timers";
> > reg = <0x40010000 0x400>;
> > clocks = <&rcc 0 160>;
> > - clock-names = "clk_int";
> > + clock-names = "int";
> >
> > pwm {
> > compatible = "st,stm32-pwm";
> > --
> > 1.9.1
> >
>
> Acked-by: Benjamin Gaignard <[email protected]>

2017-06-11 11:20:29

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH 2/2] dt-bindings: iio: update STM32 timers clock names

On Fri, 9 Jun 2017 09:43:42 +0200
Benjamin Gaignard <[email protected]> wrote:

> 2017-06-09 9:37 GMT+02:00 Fabrice Gasnier <[email protected]>:
> > Clock name has been updated during driver/DT binding review:
> > https://lkml.org/lkml/2016/12/13/718
> > Update DT binding doc to reflect this.
> >
> > Signed-off-by: Fabrice Gasnier <[email protected]>
Same fixes tag as previous as this is an example of the binding for the
mfd rather than actually part of the IIO binding.

Fixes: d0f949e220fd ("mfd: Add STM32 Timers driver")
Acked-by: Jonathan Cameron <[email protected]>

I'm assuming Rob / Mark or Lee will pick this up. If people
do want it to go through the IIO tree let me know.

Jonathan
> > ---
> > Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt b/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt
> > index 55a653d..6abc755 100644
> > --- a/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt
> > +++ b/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt
> > @@ -14,7 +14,7 @@ Example:
> > compatible = "st,stm32-timers";
> > reg = <0x40010000 0x400>;
> > clocks = <&rcc 0 160>;
> > - clock-names = "clk_int";
> > + clock-names = "int";
> >
> > timer@0 {
> > compatible = "st,stm32-timer-trigger";
> > --
> > 1.9.1
> >
> Acked-by: Benjamin Gaignard <[email protected]>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html

2017-06-14 15:17:03

by Fabrice Gasnier

[permalink] [raw]
Subject: Re: [PATCH 2/2] dt-bindings: iio: update STM32 timers clock names

On 06/11/2017 01:20 PM, Jonathan Cameron wrote:
> On Fri, 9 Jun 2017 09:43:42 +0200
> Benjamin Gaignard <[email protected]> wrote:
>
>> 2017-06-09 9:37 GMT+02:00 Fabrice Gasnier <[email protected]>:
>>> Clock name has been updated during driver/DT binding review:
>>> https://lkml.org/lkml/2016/12/13/718
>>> Update DT binding doc to reflect this.
>>>
>>> Signed-off-by: Fabrice Gasnier <[email protected]>
> Same fixes tag as previous as this is an example of the binding for the
> mfd rather than actually part of the IIO binding.
>
> Fixes: d0f949e220fd ("mfd: Add STM32 Timers driver")
> Acked-by: Jonathan Cameron <[email protected]>
>
> I'm assuming Rob / Mark or Lee will pick this up. If people
> do want it to go through the IIO tree let me know.

Hi Jonathan, all,

I just sent an updated version with Fixes tag.
I also missed pwm dt-binding in v1. I added it in V2.

Thanks for reviewing,
Best Regards,
Fabrice

>
> Jonathan
>>> ---
>>> Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt b/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt
>>> index 55a653d..6abc755 100644
>>> --- a/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt
>>> +++ b/Documentation/devicetree/bindings/iio/timer/stm32-timer-trigger.txt
>>> @@ -14,7 +14,7 @@ Example:
>>> compatible = "st,stm32-timers";
>>> reg = <0x40010000 0x400>;
>>> clocks = <&rcc 0 160>;
>>> - clock-names = "clk_int";
>>> + clock-names = "int";
>>>
>>> timer@0 {
>>> compatible = "st,stm32-timer-trigger";
>>> --
>>> 1.9.1
>>>
>> Acked-by: Benjamin Gaignard <[email protected]>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
>> the body of a message to [email protected]
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>