Re: [PATCH] dt-bindings: rpc: renesas-rpc-if: Add support for the R8A779A0 RPC-IF

[Geert Uytterhoeven]

Hi Krzysztof,

On Fri, Sep 24, 2021 at 1:48 PM Krzysztof Kozlowski
<krzysztof.kozlowski@xxxxxxxxxxxxx> wrote:
> On 22/09/2021 10:58, Wolfram Sang wrote:
> > --- a/Documentation/devicetree/bindings/memory-controllers/renesas,rpc-if.yaml
> > +++ b/Documentation/devicetree/bindings/memory-controllers/renesas,rpc-if.yaml
> > @@ -33,6 +33,7 @@ properties:
> >            - renesas,r8a77970-rpc-if       # R-Car V3M
> >            - renesas,r8a77980-rpc-if       # R-Car V3H
> >            - renesas,r8a77995-rpc-if       # R-Car D3
> > +          - renesas,r8a779a0-rpc-if       # R-Car V3U
> >        - const: renesas,rcar-gen3-rpc-if   # a generic R-Car gen3 or RZ/G2 device
>
> Where is the driver or DTS using these compatibles? The bindings can
> come even without users, but are you sure this is what you want?
> Changing them later, if ever needed, will be more difficult.

Driver:
    drivers/memory/renesas-rpc-if.c:        { .compatible =
"renesas,rcar-gen3-rpc-if", },

DTS:
    [RFC PATCH 3/4] arm64: dts: renesas: r8a779a0: Add RPC node
    https://lore.kernel.org/linux-renesas-soc/20210913065317.2297-4-wsa+renesas@xxxxxxxxxxxxxxxxxxxx/

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

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