linux-kernel - RE: [PATCH 1/2] dt-bindings: memory: Renesas RPC-IF: Add support for RZ/G2 Series

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: <TYBPR01MB5309CC39CE7896473819E7C486D40@TYBPR01MB5309.jpnprd01.prod.outlook.com>
Date:   Sat, 2 Jan 2021 08:13:26 +0000
From:   Biju Das <biju.das.jz@...renesas.com>
To:     Adam Ford <aford173@...il.com>
CC:     "linux-renesas-soc@...r.kernel.org" 
        <linux-renesas-soc@...r.kernel.org>,
        "aford@...conembedded.com" <aford@...conembedded.com>,
        Krzysztof Kozlowski <krzk@...nel.org>,
        Rob Herring <robh+dt@...nel.org>,
        Geert Uytterhoeven <geert+renesas@...der.be>,
        Magnus Damm <magnus.damm@...il.com>,
        Sergei Shtylyov <sergei.shtylyov@...il.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>
Subject: RE: [PATCH 1/2] dt-bindings: memory: Renesas RPC-IF: Add support for
 RZ/G2 Series



> -----Original Message-----
> From: Adam Ford <aford173@...il.com>
> Sent: 01 January 2021 21:34
> To: Biju Das <biju.das.jz@...renesas.com>
> Cc: linux-renesas-soc@...r.kernel.org; aford@...conembedded.com; Krzysztof
> Kozlowski <krzk@...nel.org>; Rob Herring <robh+dt@...nel.org>; Geert
> Uytterhoeven <geert+renesas@...der.be>; Magnus Damm
> <magnus.damm@...il.com>; Sergei Shtylyov <sergei.shtylyov@...il.com>;
> linux-kernel@...r.kernel.org; devicetree@...r.kernel.org
> Subject: Re: [PATCH 1/2] dt-bindings: memory: Renesas RPC-IF: Add support
> for RZ/G2 Series
> 
> On Fri, Jan 1, 2021 at 12:58 PM Biju Das <biju.das.jz@...renesas.com>
> wrote:
> >
> > Hi Adam,
> >
> > Thanks for the patch.
> >
> > > -----Original Message-----
> > > From: Adam Ford <aford173@...il.com>
> > > Sent: 01 January 2021 11:39
> > > To: linux-renesas-soc@...r.kernel.org
> > > Cc: aford@...conembedded.com; Adam Ford <aford173@...il.com>;
> > > Krzysztof Kozlowski <krzk@...nel.org>; Rob Herring
> > > <robh+dt@...nel.org>; Geert Uytterhoeven <geert+renesas@...der.be>;
> > > Magnus Damm <magnus.damm@...il.com>; Sergei Shtylyov
> > > <sergei.shtylyov@...il.com>; linux-kernel@...r.kernel.org;
> > > devicetree@...r.kernel.org
> > > Subject: [PATCH 1/2] dt-bindings: memory: Renesas RPC-IF: Add
> > > support for
> > > RZ/G2 Series
> > >
> > > The RZ/G2 Series has the RPC-IF interface.
> > > Update bindings to support: r8a774a1, r8a774b1, r8a774c0, and
> > > r8a774e1
> > >
> > > Signed-off-by: Adam Ford <aford173@...il.com>
> > > ---
> > >  .../bindings/memory-controllers/renesas,rpc-if.yaml           | 4
> ++++
> > >  1 file changed, 4 insertions(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/memory-
> > > controllers/renesas,rpc-if.yaml
> > > b/Documentation/devicetree/bindings/memory-controllers/renesas,rpc-i
> > > f.yaml index 6d6ba608fd22..050c66af8c2c 100644
> > > ---
> > > a/Documentation/devicetree/bindings/memory-controllers/renesas,rpc-
> > > if.yaml
> > > +++ b/Documentation/devicetree/bindings/memory-controllers/renesas,r
> > > +++ pc-
> > > if.yaml
> > > @@ -26,6 +26,10 @@ properties:
> > >    compatible:
> > >      items:
> > >        - enum:
> > > +          - renesas,r8a774a1-rpc-if       # RZ/G2M
> > > +          - renesas,r8a774b1-rpc-if       # RZ/G2N
> > > +          - renesas,r8a774c0-rpc-if       # RZ/G2E
> > > +          - renesas,r8a774e1-rpc-if       # RZ/G2H
> > >            - renesas,r8a77970-rpc-if       # R-Car V3M
> > >            - renesas,r8a77980-rpc-if       # R-Car V3H
> > >            - renesas,r8a77995-rpc-if       # R-Car D3
> >
> > May be we need to update the below description as well to cover RZ/G2
> device??
> >
> > - const: renesas,rcar-gen3-rpc-if   # a generic R-Car gen3 device
> 
> How do you want it to read?

Since it is generic compatible string for both R-Car gen3 and RZ/G2 device, I would update the description as

- const: renesas,rcar-gen3-rpc-if   # a generic R-Car gen3 or RZ/G2 device

Also may be we need to update the description of config RENESAS_RPCIF in drivers/memory/Kconfig to taken care of RZ/G2 devices in a separate patch.

Cheers,
Biju