linux-kernel - Re: [PATCH 1/2] dt-bindings: timer: renesas,ostm: Document RZ/V2N (R9A09G056) support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CA+V-a8s-+x1-OAJCbizOdW1ygtTYON6AybPg=e-x7M-AL+4F0A@mail.gmail.com>
Date: Fri, 2 May 2025 13:04:34 +0100
From: "Lad, Prabhakar" <prabhakar.csengg@...il.com>
To: Geert Uytterhoeven <geert@...ux-m68k.org>
Cc: Daniel Lezcano <daniel.lezcano@...aro.org>, Thomas Gleixner <tglx@...utronix.de>, 
	Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, 
	Maxime Coquelin <mcoquelin.stm32@...il.com>, Alexandre Torgue <alexandre.torgue@...s.st.com>, 
	Magnus Damm <magnus.damm@...il.com>, Chris Brandt <chris.brandt@...esas.com>, 
	linux-kernel@...r.kernel.org, devicetree@...r.kernel.org, 
	linux-stm32@...md-mailman.stormreply.com, 
	linux-arm-kernel@...ts.infradead.org, linux-renesas-soc@...r.kernel.org, 
	Biju Das <biju.das.jz@...renesas.com>, 
	Fabrizio Castro <fabrizio.castro.jz@...esas.com>, 
	Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
Subject: Re: [PATCH 1/2] dt-bindings: timer: renesas,ostm: Document RZ/V2N
 (R9A09G056) support

Hi Geert,

Thank you for the review.

On Fri, May 2, 2025 at 12:53 PM Geert Uytterhoeven <geert@...ux-m68k.org> wrote:
>
> On Thu, 1 May 2025 at 14:37, Prabhakar <prabhakar.csengg@...il.com> wrote:
> > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
> >
> > Document support for the Renesas OS Timer (OSTM) found on the Renesas
> > RZ/V2N (R9A09G056) SoC. The OSTM IP on RZ/V2N is identical to that on
> > other RZ families, so no driver changes are required as `renesas,ostm`
> > will be used as fallback compatible.
> >
> > Also include RZ/V2N in the list of compatibles for which the `resets`
> > property is required.
> >
> > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@...der.be>
>
> > --- a/Documentation/devicetree/bindings/timer/renesas,ostm.yaml
> > +++ b/Documentation/devicetree/bindings/timer/renesas,ostm.yaml
> > @@ -26,6 +26,7 @@ properties:
> >            - renesas,r9a07g043-ostm # RZ/G2UL and RZ/Five
> >            - renesas,r9a07g044-ostm # RZ/G2{L,LC}
> >            - renesas,r9a07g054-ostm # RZ/V2L
> > +          - renesas,r9a09g056-ostm # RZ/V2N
> >            - renesas,r9a09g057-ostm # RZ/V2H(P)
> >        - const: renesas,ostm        # Generic
> >
> > @@ -59,6 +60,7 @@ if:
> >            - renesas,r9a07g043-ostm
> >            - renesas,r9a07g044-ostm
> >            - renesas,r9a07g054-ostm
> > +          - renesas,r9a09g056-ostm
> >            - renesas,r9a09g057-ostm
>
> Looks like we have reached critical mass to invert the logic?
>
Agreed.

>     not:
>       contains:
>         - renesas,r7s72100-ostm
>         - renesas,r7s9210-ostm
>
> ?
>
OK, I will update it as above and send a v2.

Cheers,
Prabhakar