RE: [PATCH v2 1/3] dt-bindings: arm: renesas: Document Renesas RZ/V2M System Configuration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Krzysztof Kozlowski,

> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> Sent: 11 November 2022 08:34
> To: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>;
> Krzysztof Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx>
> Cc: Phil Edworthy <phil.edworthy@xxxxxxxxxxx>; Geert Uytterhoeven
> <geert+renesas@xxxxxxxxx>; Magnus Damm <magnus.damm@xxxxxxxxx>; linux-
> renesas-soc@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; Chris Paterson
> <Chris.Paterson2@xxxxxxxxxxx>; Fabrizio Castro
> <fabrizio.castro.jz@xxxxxxxxxxx>
> Subject: Re: [PATCH v2 1/3] dt-bindings: arm: renesas: Document Renesas
> RZ/V2M System Configuration
> 
> On 10/11/2022 17:21, Biju Das wrote:
> > From: Phil Edworthy <phil.edworthy@xxxxxxxxxxx>
> >
> > Add DT binding documentation for System Configuration (SYS) found on
> > RZ/V2M SoC's.
> >
> > SYS block contains the SYS_VERSION register which can be used to
> > retrieve SoC version information.
> >
> > Signed-off-by: Phil Edworthy <phil.edworthy@xxxxxxxxxxx>
> > [biju: Updated the example ]
> > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> > ---
> > v1->v2:
> >  * Moved the file from arm->soc/renesas
> >  * Updated the path for binding file
> >  * Updated the example
> > ---
> >  .../soc/renesas/renesas,rzv2m-sys.yaml        | 39 +++++++++++++++++++
> >  1 file changed, 39 insertions(+)
> >  create mode 100644
> > Documentation/devicetree/bindings/soc/renesas/renesas,rzv2m-sys.yaml
> >
> > diff --git
> > a/Documentation/devicetree/bindings/soc/renesas/renesas,rzv2m-sys.yaml
> > b/Documentation/devicetree/bindings/soc/renesas/renesas,rzv2m-sys.yaml
> > new file mode 100644
> > index 000000000000..cc41747798e2
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/soc/renesas/renesas,rzv2m-sys.
> > +++ yaml
> 
> Filename should be based on the compatible. Pretty often some common parts of
> both are fine (e.g. when file contains multiple compatibles), but this very
> different then what I see below.

We plan to upstream another similar SoC, RZ/V2MA which has similar IP, so we may
need to add generic compatible rzv2m-sys. I am checking with HW people to get
more info about RZ/V2MA.

> 
> > @@ -0,0 +1,39 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2
> > +---
> > +$id:
> "
"
> > +$schema:
> "
"
> 
> Drop quotes from both.

OK.

> 
> > +
> > +title: Renesas RZ/V2M System Configuration (SYS)
> > +
> > +maintainers:
> > +  - Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> > +
> > +description:
> > +  The RZ/V2M System Configuration (SYS) performs system control of
> > +the LSI
> > +  and supports the following functions,
> > +  - LSI version
> > +  - 34-bit address space access function
> > +  - PCIe related settings
> > +  - WDT stop control
> > +  - Temperature sensor (TSU) monitor
> > +
> > +properties:
> > +  compatible:
> > +    const: renesas,r9a09g011-sys
> > +
> > +  reg:
> > +    maxItems: 1
> > +
> > +required:
> > +  - compatible
> > +  - reg
> > +
> > +additionalProperties: false
> > +
> > +examples:
> > +  - |
> > +    sysc: system-configuration@a3f03000 {
> 
> If I get properly the purpose of the device, usually this is called "system-
> controller". Use that as device node.

The hardware manual mentions the below. So want to consistent with HW manual.

Section 38 System Configuration (SYS)
This section describes the functions of the system configuration (SYS).

> 
> > +            compatible = "renesas,r9a09g011-sys";
> > +            reg = <0xa3f03000 0x400>;
> 
> Use 4 spaces for example indentation.

OK, Agreed.

Cheers,
Biju





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux