linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] Fix STM32 timers dt-bindings
@ 2017-06-09  7:37 Fabrice Gasnier
  2017-06-09  7:37 ` [PATCH 1/2] dt-bindings: mfd: update STM32 timers clock names Fabrice Gasnier
  2017-06-09  7:37 ` [PATCH 2/2] dt-bindings: iio: " Fabrice Gasnier
  0 siblings, 2 replies; 8+ messages in thread
From: Fabrice Gasnier @ 2017-06-09  7:37 UTC (permalink / raw)
  To: lee.jones, benjamin.gaignard, jic23, robh+dt
  Cc: mark.rutland, alexandre.torgue, mcoquelin.stm32, fabrice.gasnier,
	benjamin.gaignard, linux-iio, devicetree, linux-arm-kernel,
	linux-kernel

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

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

* [PATCH 1/2] dt-bindings: mfd: update STM32 timers clock names
  2017-06-09  7:37 [PATCH 0/2] Fix STM32 timers dt-bindings Fabrice Gasnier
@ 2017-06-09  7:37 ` Fabrice Gasnier
  2017-06-09  7:43   ` Benjamin Gaignard
  2017-06-09  7:37 ` [PATCH 2/2] dt-bindings: iio: " Fabrice Gasnier
  1 sibling, 1 reply; 8+ messages in thread
From: Fabrice Gasnier @ 2017-06-09  7:37 UTC (permalink / raw)
  To: lee.jones, benjamin.gaignard, jic23, robh+dt
  Cc: mark.rutland, alexandre.torgue, mcoquelin.stm32, fabrice.gasnier,
	benjamin.gaignard, linux-iio, devicetree, linux-arm-kernel,
	linux-kernel

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 <fabrice.gasnier@st.com>
---
 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

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

* [PATCH 2/2] dt-bindings: iio: update STM32 timers clock names
  2017-06-09  7:37 [PATCH 0/2] Fix STM32 timers dt-bindings Fabrice Gasnier
  2017-06-09  7:37 ` [PATCH 1/2] dt-bindings: mfd: update STM32 timers clock names Fabrice Gasnier
@ 2017-06-09  7:37 ` Fabrice Gasnier
  2017-06-09  7:43   ` Benjamin Gaignard
  1 sibling, 1 reply; 8+ messages in thread
From: Fabrice Gasnier @ 2017-06-09  7:37 UTC (permalink / raw)
  To: lee.jones, benjamin.gaignard, jic23, robh+dt
  Cc: mark.rutland, alexandre.torgue, mcoquelin.stm32, fabrice.gasnier,
	benjamin.gaignard, linux-iio, devicetree, linux-arm-kernel,
	linux-kernel

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 <fabrice.gasnier@st.com>
---
 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

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

* Re: [PATCH 1/2] dt-bindings: mfd: update STM32 timers clock names
  2017-06-09  7:37 ` [PATCH 1/2] dt-bindings: mfd: update STM32 timers clock names Fabrice Gasnier
@ 2017-06-09  7:43   ` Benjamin Gaignard
  2017-06-11 11:15     ` Jonathan Cameron
  0 siblings, 1 reply; 8+ messages in thread
From: Benjamin Gaignard @ 2017-06-09  7:43 UTC (permalink / raw)
  To: Fabrice Gasnier
  Cc: Lee Jones, Jonathan Cameron, Rob Herring, Mark Rutland,
	Alexandre Torgue, Maxime Coquelin, Benjamin GAIGNARD, linux-iio,
	devicetree, linux-arm-kernel, Linux Kernel Mailing List

2017-06-09 9:37 GMT+02:00 Fabrice Gasnier <fabrice.gasnier@st.com>:
> 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 <fabrice.gasnier@st.com>
> ---
>  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 <benjamin.gaignard@linaro.org>

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

* Re: [PATCH 2/2] dt-bindings: iio: update STM32 timers clock names
  2017-06-09  7:37 ` [PATCH 2/2] dt-bindings: iio: " Fabrice Gasnier
@ 2017-06-09  7:43   ` Benjamin Gaignard
  2017-06-11 11:20     ` Jonathan Cameron
  0 siblings, 1 reply; 8+ messages in thread
From: Benjamin Gaignard @ 2017-06-09  7:43 UTC (permalink / raw)
  To: Fabrice Gasnier
  Cc: Lee Jones, Jonathan Cameron, Rob Herring, Mark Rutland,
	Alexandre Torgue, Maxime Coquelin, Benjamin GAIGNARD, linux-iio,
	devicetree, linux-arm-kernel, Linux Kernel Mailing List

2017-06-09 9:37 GMT+02:00 Fabrice Gasnier <fabrice.gasnier@st.com>:
> 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 <fabrice.gasnier@st.com>
> ---
>  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 <benjamin.gaignard@linaro.org>

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

* Re: [PATCH 1/2] dt-bindings: mfd: update STM32 timers clock names
  2017-06-09  7:43   ` Benjamin Gaignard
@ 2017-06-11 11:15     ` Jonathan Cameron
  0 siblings, 0 replies; 8+ messages in thread
From: Jonathan Cameron @ 2017-06-11 11:15 UTC (permalink / raw)
  To: Benjamin Gaignard
  Cc: Fabrice Gasnier, Lee Jones, Rob Herring, Mark Rutland,
	Alexandre Torgue, Maxime Coquelin, Benjamin GAIGNARD, linux-iio,
	devicetree, linux-arm-kernel, Linux Kernel Mailing List

On Fri, 9 Jun 2017 09:43:21 +0200
Benjamin Gaignard <benjamin.gaignard@linaro.org> wrote:

> 2017-06-09 9:37 GMT+02:00 Fabrice Gasnier <fabrice.gasnier@st.com>:
> > 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 <fabrice.gasnier@st.com>
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 <benjamin.gaignard@linaro.org>

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

* Re: [PATCH 2/2] dt-bindings: iio: update STM32 timers clock names
  2017-06-09  7:43   ` Benjamin Gaignard
@ 2017-06-11 11:20     ` Jonathan Cameron
  2017-06-14 15:16       ` Fabrice Gasnier
  0 siblings, 1 reply; 8+ messages in thread
From: Jonathan Cameron @ 2017-06-11 11:20 UTC (permalink / raw)
  To: Benjamin Gaignard
  Cc: Fabrice Gasnier, Lee Jones, Rob Herring, Mark Rutland,
	Alexandre Torgue, Maxime Coquelin, Benjamin GAIGNARD, linux-iio,
	devicetree, linux-arm-kernel, Linux Kernel Mailing List

On Fri, 9 Jun 2017 09:43:42 +0200
Benjamin Gaignard <benjamin.gaignard@linaro.org> wrote:

> 2017-06-09 9:37 GMT+02:00 Fabrice Gasnier <fabrice.gasnier@st.com>:
> > 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 <fabrice.gasnier@st.com>
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 <jic23@kernel.org>

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 <benjamin.gaignard@linaro.org>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* Re: [PATCH 2/2] dt-bindings: iio: update STM32 timers clock names
  2017-06-11 11:20     ` Jonathan Cameron
@ 2017-06-14 15:16       ` Fabrice Gasnier
  0 siblings, 0 replies; 8+ messages in thread
From: Fabrice Gasnier @ 2017-06-14 15:16 UTC (permalink / raw)
  To: Jonathan Cameron, Benjamin Gaignard
  Cc: Lee Jones, Rob Herring, Mark Rutland, Alexandre Torgue,
	Maxime Coquelin, Benjamin GAIGNARD, linux-iio, devicetree,
	linux-arm-kernel, Linux Kernel Mailing List

On 06/11/2017 01:20 PM, Jonathan Cameron wrote:
> On Fri, 9 Jun 2017 09:43:42 +0200
> Benjamin Gaignard <benjamin.gaignard@linaro.org> wrote:
> 
>> 2017-06-09 9:37 GMT+02:00 Fabrice Gasnier <fabrice.gasnier@st.com>:
>>> 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 <fabrice.gasnier@st.com>
> 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 <jic23@kernel.org>
> 
> 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 <benjamin.gaignard@linaro.org>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

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

end of thread, other threads:[~2017-06-14 15:17 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-06-09  7:37 [PATCH 0/2] Fix STM32 timers dt-bindings Fabrice Gasnier
2017-06-09  7:37 ` [PATCH 1/2] dt-bindings: mfd: update STM32 timers clock names Fabrice Gasnier
2017-06-09  7:43   ` Benjamin Gaignard
2017-06-11 11:15     ` Jonathan Cameron
2017-06-09  7:37 ` [PATCH 2/2] dt-bindings: iio: " Fabrice Gasnier
2017-06-09  7:43   ` Benjamin Gaignard
2017-06-11 11:20     ` Jonathan Cameron
2017-06-14 15:16       ` Fabrice Gasnier

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).