* [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.