Re: [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings

2016-09-30 Thread Magnus Damm 
Hi Geert,

On Fri, Sep 30, 2016 at 4:20 PM, Geert Uytterhoeven
 wrote:
> Hi Magnus,
>
> On Fri, Sep 30, 2016 at 9:07 AM, Magnus Damm  wrote:
>> On Fri, Sep 23, 2016 at 6:03 PM, Simon Horman  wrote:
>>> On Fri, Sep 23, 2016 at 10:35:06AM +0200, Geert Uytterhoeven wrote:
 And these are planned to be removed again with Magnus'
 "devicetree: bindings: r8a73a4 and R-Car Gen2 CMT bindings"
 (https://patchwork.kernel.org/patch/8579481/)?
>>>
>>> Sorry, that slipped my mind.
>>>
>>> Magnus, what is the status of that work?
>>
>> Banged my head against DT bindings too long, and I felt it never got
>> picked up so I gave up. =)
>>
>> It would make sense to update and resend, I do however wonder how to
>> improve our chances to get it merged?
>
> Looking in email history, the only comment you got on v4 was an accidental
> word duplicate. As you already had plenty of acks, resending with that
> fixed (and the code included again :-) should be enough.
>
> Note that I've been running that code since the first half of 2015 ;-)

Will fix up and resend. I recall that some more effort is needed to
clean up other parts of the bindings too, but that can be done
incrementally.

Cheers,

/ magnus

Re: [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings

2016-09-30 Thread Geert Uytterhoeven 
Hi Magnus,

On Fri, Sep 30, 2016 at 9:07 AM, Magnus Damm  wrote:
> On Fri, Sep 23, 2016 at 6:03 PM, Simon Horman  wrote:
>> On Fri, Sep 23, 2016 at 10:35:06AM +0200, Geert Uytterhoeven wrote:
>>> And these are planned to be removed again with Magnus'
>>> "devicetree: bindings: r8a73a4 and R-Car Gen2 CMT bindings"
>>> (https://patchwork.kernel.org/patch/8579481/)?
>>
>> Sorry, that slipped my mind.
>>
>> Magnus, what is the status of that work?
>
> Banged my head against DT bindings too long, and I felt it never got
> picked up so I gave up. =)
>
> It would make sense to update and resend, I do however wonder how to
> improve our chances to get it merged?

Looking in email history, the only comment you got on v4 was an accidental
word duplicate. As you already had plenty of acks, resending with that
fixed (and the code included again :-) should be enough.

Note that I've been running that code since the first half of 2015 ;-)

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

Re: [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings

2016-09-30 Thread Magnus Damm 
On Fri, Sep 23, 2016 at 6:03 PM, Simon Horman  wrote:
> On Fri, Sep 23, 2016 at 10:35:06AM +0200, Geert Uytterhoeven wrote:
>> Hi Simon, Magnus,
>>
>> On Fri, Sep 23, 2016 at 10:20 AM, Simon Horman
>>  wrote:
>> > This documents the SoC specific binding for the r8a779[34] SoCs.
>> > This is in keeping with the documentation of other R-Car Gen2 SoCs.
>> >
>> > Signed-off-by: Simon Horman 
>> > ---
>> >  Documentation/devicetree/bindings/timer/renesas,cmt.txt | 9 +++--
>> >  1 file changed, 7 insertions(+), 2 deletions(-)
>> >
>> > diff --git a/Documentation/devicetree/bindings/timer/renesas,cmt.txt 
>> > b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
>> > index 1a05c1b243c1..566fb599ea39 100644
>> > --- a/Documentation/devicetree/bindings/timer/renesas,cmt.txt
>> > +++ b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
>> > @@ -48,10 +48,15 @@ Required Properties:
>> > (CMT[01])
>> >  - "renesas,cmt-48-r8a7791" for the r8a7791 48-bit CMT
>> > (CMT[01])
>> > +- "renesas,cmt-48-r8a7793" for the r8a7793 48-bit CMT
>> > +   (CMT[01])
>> > +- "renesas,cmt-48-r8a7794" for the r8a7793 48-bit CMT
>> > +   (CMT[01])
>> >  - "renesas,cmt-48-gen2" for all second generation 48-bit CMT
>> > -   (CMT[01] on r8a73a4, r8a7790 and r8a7791)
>> > +   (CMT[01] on r8a73a4, r8a7790, r8a7791, r8a7793, r8a7794)
>> > This is a fallback for the renesas,cmt-48-r8a73a4,
>> > -   renesas,cmt-48-r8a7790 and renesas,cmt-48-r8a7791 entries.
>> > +   renesas,cmt-48-r8a7790, renesas,cmt-48-r8a7791,
>> > +   renesas,cmt-48-r8a7793 and renesas,cmt-48-r8a7794 entries.
>> >
>> >- reg: base address and length of the registers block for the timer 
>> > module.
>> >- interrupts: interrupt-specifier for the timer, one per channel.
>>
>> And these are planned to be removed again with Magnus'
>> "devicetree: bindings: r8a73a4 and R-Car Gen2 CMT bindings"
>> (https://patchwork.kernel.org/patch/8579481/)?
>
> Sorry, that slipped my mind.
>
> Magnus, what is the status of that work?

Banged my head against DT bindings too long, and I felt it never got
picked up so I gave up. =)

It would make sense to update and resend, I do however wonder how to
improve our chances to get it merged?

Cheers,

/ magnus

Re: [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings

2016-09-23 Thread Simon Horman 
On Fri, Sep 23, 2016 at 10:35:06AM +0200, Geert Uytterhoeven wrote:
> Hi Simon, Magnus,
> 
> On Fri, Sep 23, 2016 at 10:20 AM, Simon Horman
>  wrote:
> > This documents the SoC specific binding for the r8a779[34] SoCs.
> > This is in keeping with the documentation of other R-Car Gen2 SoCs.
> >
> > Signed-off-by: Simon Horman 
> > ---
> >  Documentation/devicetree/bindings/timer/renesas,cmt.txt | 9 +++--
> >  1 file changed, 7 insertions(+), 2 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/timer/renesas,cmt.txt 
> > b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> > index 1a05c1b243c1..566fb599ea39 100644
> > --- a/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> > +++ b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> > @@ -48,10 +48,15 @@ Required Properties:
> > (CMT[01])
> >  - "renesas,cmt-48-r8a7791" for the r8a7791 48-bit CMT
> > (CMT[01])
> > +- "renesas,cmt-48-r8a7793" for the r8a7793 48-bit CMT
> > +   (CMT[01])
> > +- "renesas,cmt-48-r8a7794" for the r8a7793 48-bit CMT
> > +   (CMT[01])
> >  - "renesas,cmt-48-gen2" for all second generation 48-bit CMT
> > -   (CMT[01] on r8a73a4, r8a7790 and r8a7791)
> > +   (CMT[01] on r8a73a4, r8a7790, r8a7791, r8a7793, r8a7794)
> > This is a fallback for the renesas,cmt-48-r8a73a4,
> > -   renesas,cmt-48-r8a7790 and renesas,cmt-48-r8a7791 entries.
> > +   renesas,cmt-48-r8a7790, renesas,cmt-48-r8a7791,
> > +   renesas,cmt-48-r8a7793 and renesas,cmt-48-r8a7794 entries.
> >
> >- reg: base address and length of the registers block for the timer 
> > module.
> >- interrupts: interrupt-specifier for the timer, one per channel.
> 
> And these are planned to be removed again with Magnus'
> "devicetree: bindings: r8a73a4 and R-Car Gen2 CMT bindings"
> (https://patchwork.kernel.org/patch/8579481/)?

Sorry, that slipped my mind.

Magnus, what is the status of that work?

Re: [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings

2016-09-23 Thread Geert Uytterhoeven 
Hi Simon, Magnus,

On Fri, Sep 23, 2016 at 10:20 AM, Simon Horman
 wrote:
> This documents the SoC specific binding for the r8a779[34] SoCs.
> This is in keeping with the documentation of other R-Car Gen2 SoCs.
>
> Signed-off-by: Simon Horman 
> ---
>  Documentation/devicetree/bindings/timer/renesas,cmt.txt | 9 +++--
>  1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/timer/renesas,cmt.txt 
> b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> index 1a05c1b243c1..566fb599ea39 100644
> --- a/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> +++ b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
> @@ -48,10 +48,15 @@ Required Properties:
> (CMT[01])
>  - "renesas,cmt-48-r8a7791" for the r8a7791 48-bit CMT
> (CMT[01])
> +- "renesas,cmt-48-r8a7793" for the r8a7793 48-bit CMT
> +   (CMT[01])
> +- "renesas,cmt-48-r8a7794" for the r8a7793 48-bit CMT
> +   (CMT[01])
>  - "renesas,cmt-48-gen2" for all second generation 48-bit CMT
> -   (CMT[01] on r8a73a4, r8a7790 and r8a7791)
> +   (CMT[01] on r8a73a4, r8a7790, r8a7791, r8a7793, r8a7794)
> This is a fallback for the renesas,cmt-48-r8a73a4,
> -   renesas,cmt-48-r8a7790 and renesas,cmt-48-r8a7791 entries.
> +   renesas,cmt-48-r8a7790, renesas,cmt-48-r8a7791,
> +   renesas,cmt-48-r8a7793 and renesas,cmt-48-r8a7794 entries.
>
>- reg: base address and length of the registers block for the timer module.
>- interrupts: interrupt-specifier for the timer, one per channel.

And these are planned to be removed again with Magnus'
"devicetree: bindings: r8a73a4 and R-Car Gen2 CMT bindings"
(https://patchwork.kernel.org/patch/8579481/)?

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

[PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings

2016-09-23 Thread Simon Horman 
This documents the SoC specific binding for the r8a779[34] SoCs.
This is in keeping with the documentation of other R-Car Gen2 SoCs.

Signed-off-by: Simon Horman 
---
 Documentation/devicetree/bindings/timer/renesas,cmt.txt | 9 +++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/timer/renesas,cmt.txt 
b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
index 1a05c1b243c1..566fb599ea39 100644
--- a/Documentation/devicetree/bindings/timer/renesas,cmt.txt
+++ b/Documentation/devicetree/bindings/timer/renesas,cmt.txt
@@ -48,10 +48,15 @@ Required Properties:
(CMT[01])
 - "renesas,cmt-48-r8a7791" for the r8a7791 48-bit CMT
(CMT[01])
+- "renesas,cmt-48-r8a7793" for the r8a7793 48-bit CMT
+   (CMT[01])
+- "renesas,cmt-48-r8a7794" for the r8a7793 48-bit CMT
+   (CMT[01])
 - "renesas,cmt-48-gen2" for all second generation 48-bit CMT
-   (CMT[01] on r8a73a4, r8a7790 and r8a7791)
+   (CMT[01] on r8a73a4, r8a7790, r8a7791, r8a7793, r8a7794)
This is a fallback for the renesas,cmt-48-r8a73a4,
-   renesas,cmt-48-r8a7790 and renesas,cmt-48-r8a7791 entries.
+   renesas,cmt-48-r8a7790, renesas,cmt-48-r8a7791,
+   renesas,cmt-48-r8a7793 and renesas,cmt-48-r8a7794 entries.
 
   - reg: base address and length of the registers block for the timer module.
   - interrupts: interrupt-specifier for the timer, one per channel.
-- 
2.7.0.rc3.207.g0ac5344