Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1493917 > unrolled thread

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

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings Magnus Damm <magnus.damm@gmail.com> - 2016-09-30 09:10 +0200
    Re: [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings Geert Uytterhoeven <geert@linux-m68k.org> - 2016-09-30 09:30 +0200
      Re: [PATCH] clocksource: sh_cmt: Document r8a779[34] SoC specific bindings Magnus Damm <magnus.damm@gmail.com> - 2016-09-30 09:30 +0200

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

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

[toc] | [next] | [standalone]

#1493924

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

[toc] | [prev] | [next] | [standalone]

#1493925

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

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web