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]

 




> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> Sent: 11 November 2022 10:52
> 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 11/11/2022 10:06, Biju Das wrote:
> > 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.ya
> >>> ml
> >>> b/Documentation/devicetree/bindings/soc/renesas/renesas,rzv2m-sys.ya
> >>> ml
> >>> 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.
> 
> If the hardware manual said this is called "foo-whatever-name" or
> "rz85736dfnx2", you would use it as well?
> 
> Node names should be generic.

What about system-configuration-controller to make it generic or
You still prefer system-controller like [1]

[1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm64/boot/dts/renesas/r9a07g054.dtsi?h=v6.1-rc4#n635

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