* [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings @ 2016-09-23 8:20 Simon Horman 2016-09-23 8:35 ` Geert Uytterhoeven 0 siblings, 1 reply; 6+ messages in thread From: Simon Horman @ 2016-09-23 8:20 UTC (permalink / raw) To: Daniel Lezcano, Thomas Gleixner, linux-renesas-soc Cc: Magnus Damm, linux-kernel, 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 <horms+renesas@verge.net.au> --- 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 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings 2016-09-23 8:20 [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings Simon Horman @ 2016-09-23 8:35 ` Geert Uytterhoeven 2016-09-23 9:03 ` Simon Horman 0 siblings, 1 reply; 6+ messages in thread From: Geert Uytterhoeven @ 2016-09-23 8:35 UTC (permalink / raw) To: Simon Horman, Magnus Damm Cc: Daniel Lezcano, Thomas Gleixner, Linux-Renesas, linux-kernel Hi Simon, Magnus, On Fri, Sep 23, 2016 at 10:20 AM, Simon Horman <horms+renesas@verge.net.au> 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 <horms+renesas@verge.net.au> > --- > 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 -- geert@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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings 2016-09-23 8:35 ` Geert Uytterhoeven @ 2016-09-23 9:03 ` Simon Horman 2016-09-30 7:07 ` Magnus Damm 0 siblings, 1 reply; 6+ messages in thread From: Simon Horman @ 2016-09-23 9:03 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Magnus Damm, Daniel Lezcano, Thomas Gleixner, Linux-Renesas, linux-kernel 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 > <horms+renesas@verge.net.au> 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 <horms+renesas@verge.net.au> > > --- > > 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? ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings 2016-09-23 9:03 ` Simon Horman @ 2016-09-30 7:07 ` Magnus Damm 2016-09-30 7:20 ` Geert Uytterhoeven 0 siblings, 1 reply; 6+ messages in thread From: Magnus Damm @ 2016-09-30 7:07 UTC (permalink / raw) To: Simon Horman Cc: Geert Uytterhoeven, Daniel Lezcano, Thomas Gleixner, Linux-Renesas, linux-kernel On Fri, Sep 23, 2016 at 6:03 PM, Simon Horman <horms@verge.net.au> 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 >> <horms+renesas@verge.net.au> 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 <horms+renesas@verge.net.au> >> > --- >> > 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings 2016-09-30 7:07 ` Magnus Damm @ 2016-09-30 7:20 ` Geert Uytterhoeven 2016-09-30 7:25 ` Magnus Damm 0 siblings, 1 reply; 6+ messages in thread From: Geert Uytterhoeven @ 2016-09-30 7:20 UTC (permalink / raw) To: Magnus Damm Cc: Simon Horman, Daniel Lezcano, Thomas Gleixner, Linux-Renesas, linux-kernel Hi Magnus, On Fri, Sep 30, 2016 at 9:07 AM, Magnus Damm <magnus.damm@gmail.com> wrote: > On Fri, Sep 23, 2016 at 6:03 PM, Simon Horman <horms@verge.net.au> 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 -- geert@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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings 2016-09-30 7:20 ` Geert Uytterhoeven @ 2016-09-30 7:25 ` Magnus Damm 0 siblings, 0 replies; 6+ messages in thread From: Magnus Damm @ 2016-09-30 7:25 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Simon Horman, Daniel Lezcano, Thomas Gleixner, Linux-Renesas, linux-kernel Hi Geert, On Fri, Sep 30, 2016 at 4:20 PM, Geert Uytterhoeven <geert@linux-m68k.org> wrote: > Hi Magnus, > > On Fri, Sep 30, 2016 at 9:07 AM, Magnus Damm <magnus.damm@gmail.com> wrote: >> On Fri, Sep 23, 2016 at 6:03 PM, Simon Horman <horms@verge.net.au> 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-09-30 7:25 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2016-09-23 8:20 [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings Simon Horman 2016-09-23 8:35 ` Geert Uytterhoeven 2016-09-23 9:03 ` Simon Horman 2016-09-30 7:07 ` Magnus Damm 2016-09-30 7:20 ` Geert Uytterhoeven 2016-09-30 7:25 ` Magnus Damm
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.