[v2] dt-bindings: phy: ralink-usb-phy: convert to dtschema
Commit Message
Convert the ralink-usb-phy bindings to DT schema.
Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
Changes in v2:
- Add missing SPDX License Identifier
.../bindings/phy/ralink-usb-phy.txt | 23 ------
.../bindings/phy/ralink-usb-phy.yaml | 70 +++++++++++++++++++
2 files changed, 70 insertions(+), 23 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/phy/ralink-usb-phy.txt
create mode 100644 Documentation/devicetree/bindings/phy/ralink-usb-phy.yaml
Comments
On 22/10/2023 15:12, Sergio Paracuellos wrote:
> Convert the ralink-usb-phy bindings to DT schema.
>
> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> ---
> Changes in v2:
> - Add missing SPDX License Identifier
>
> .../bindings/phy/ralink-usb-phy.txt | 23 ------
> .../bindings/phy/ralink-usb-phy.yaml | 70 +++++++++++++++++++
> 2 files changed, 70 insertions(+), 23 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/phy/ralink-usb-phy.txt
> create mode 100644 Documentation/devicetree/bindings/phy/ralink-usb-phy.yaml
>
> diff --git a/Documentation/devicetree/bindings/phy/ralink-usb-phy.txt b/Documentation/devicetree/bindings/phy/ralink-usb-phy.txt
> deleted file mode 100644
> index 9d2868a437ab..000000000000
> --- a/Documentation/devicetree/bindings/phy/ralink-usb-phy.txt
> +++ /dev/null
> @@ -1,23 +0,0 @@
> -Mediatek/Ralink USB PHY
> -
> -Required properties:
> - - compatible: "ralink,rt3352-usbphy"
> - "mediatek,mt7620-usbphy"
> - "mediatek,mt7628-usbphy"
> - - reg: required for "mediatek,mt7628-usbphy", unused otherwise
> - - #phy-cells: should be 0
> - - ralink,sysctl: a phandle to a ralink syscon register region
> - - resets: the two reset controllers for host and device
> - - reset-names: the names of the 2 reset controllers
> -
> -Example:
> -
> -usbphy: phy {
> - compatible = "mediatek,mt7628-usbphy";
> - reg = <0x10120000 0x1000>;
> - #phy-cells = <0>;
> -
> - ralink,sysctl = <&sysc>;
> - resets = <&rstctrl 22 &rstctrl 25>;
> - reset-names = "host", "device";
> -};
> diff --git a/Documentation/devicetree/bindings/phy/ralink-usb-phy.yaml b/Documentation/devicetree/bindings/phy/ralink-usb-phy.yaml
> new file mode 100644
> index 000000000000..b05665f5b641
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/phy/ralink-usb-phy.yaml
Filename matching compatible. Missing vendor prefix, comma.
> @@ -0,0 +1,70 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/phy/ralink-usb-phy.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Mediatek/Ralink USB PHY
> +
> +maintainers:
> + - Sergio Paracuellos <sergio.paracuellos@gmail.com>
> +
> +properties:
> + compatible:
> + enum:
> + - ralink,rt3352-usbphy
> + - mediatek,mt7620-usbphy
> + - mediatek,mt7628-usbphy
Keep them ordered alphabetically, please.
Blank line here.
> + reg:
> + maxItems: 1
> +
> + "#phy-cells":
> + const: 0
> +
> + ralink,sysctl:
> + description:
> + phandle to a ralink syscon register region.
> + $ref: /schemas/types.yaml#/definitions/phandle
> +
> + resets:
> + items:
> + - description: USB Host reset controller
> + - description: USB Device reset controller
> +
> + reset-names:
> + items:
> + - const: host
> + - const: device
> +
> +allOf:
> + - if:
> + properties:
> + compatible:
> + contains:
> + const: mediatek,mt7628-usbphy
> + then:
> + required:
> + - reg
else:
is it even valid?
Anyway, please put allOf: block after required: block.
> +
> +required:
> + - compatible
> + - "#phy-cells"
> + - ralink,sysctl
> + - resets
> + - reset-names
> +
Best regards,
Krzysztof
Hi Krysztof,
On Sun, Oct 22, 2023 at 5:57 PM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> On 22/10/2023 15:12, Sergio Paracuellos wrote:
> > Convert the ralink-usb-phy bindings to DT schema.
> >
> > Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> > ---
> > Changes in v2:
> > - Add missing SPDX License Identifier
> >
> > .../bindings/phy/ralink-usb-phy.txt | 23 ------
> > .../bindings/phy/ralink-usb-phy.yaml | 70 +++++++++++++++++++
> > 2 files changed, 70 insertions(+), 23 deletions(-)
> > delete mode 100644 Documentation/devicetree/bindings/phy/ralink-usb-phy.txt
> > create mode 100644 Documentation/devicetree/bindings/phy/ralink-usb-phy.yaml
> >
> > diff --git a/Documentation/devicetree/bindings/phy/ralink-usb-phy.txt b/Documentation/devicetree/bindings/phy/ralink-usb-phy.txt
> > deleted file mode 100644
> > index 9d2868a437ab..000000000000
> > --- a/Documentation/devicetree/bindings/phy/ralink-usb-phy.txt
> > +++ /dev/null
> > @@ -1,23 +0,0 @@
> > -Mediatek/Ralink USB PHY
> > -
> > -Required properties:
> > - - compatible: "ralink,rt3352-usbphy"
> > - "mediatek,mt7620-usbphy"
> > - "mediatek,mt7628-usbphy"
> > - - reg: required for "mediatek,mt7628-usbphy", unused otherwise
> > - - #phy-cells: should be 0
> > - - ralink,sysctl: a phandle to a ralink syscon register region
> > - - resets: the two reset controllers for host and device
> > - - reset-names: the names of the 2 reset controllers
> > -
> > -Example:
> > -
> > -usbphy: phy {
> > - compatible = "mediatek,mt7628-usbphy";
> > - reg = <0x10120000 0x1000>;
> > - #phy-cells = <0>;
> > -
> > - ralink,sysctl = <&sysc>;
> > - resets = <&rstctrl 22 &rstctrl 25>;
> > - reset-names = "host", "device";
> > -};
> > diff --git a/Documentation/devicetree/bindings/phy/ralink-usb-phy.yaml b/Documentation/devicetree/bindings/phy/ralink-usb-phy.yaml
> > new file mode 100644
> > index 000000000000..b05665f5b641
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/phy/ralink-usb-phy.yaml
>
> Filename matching compatible. Missing vendor prefix, comma.
Sure, will use mediatek,mt7628-usbphy compatible, then.
>
>
> > @@ -0,0 +1,70 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> > +%YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/phy/ralink-usb-phy.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: Mediatek/Ralink USB PHY
> > +
> > +maintainers:
> > + - Sergio Paracuellos <sergio.paracuellos@gmail.com>
> > +
> > +properties:
> > + compatible:
> > + enum:
> > + - ralink,rt3352-usbphy
> > + - mediatek,mt7620-usbphy
> > + - mediatek,mt7628-usbphy
>
> Keep them ordered alphabetically, please.
Will do.
>
> Blank line here.
True, thanks.
>
>
> > + reg:
> > + maxItems: 1
> > +
> > + "#phy-cells":
> > + const: 0
> > +
> > + ralink,sysctl:
> > + description:
> > + phandle to a ralink syscon register region.
> > + $ref: /schemas/types.yaml#/definitions/phandle
> > +
> > + resets:
> > + items:
> > + - description: USB Host reset controller
> > + - description: USB Device reset controller
> > +
> > + reset-names:
> > + items:
> > + - const: host
> > + - const: device
> > +
> > +allOf:
> > + - if:
> > + properties:
> > + compatible:
> > + contains:
> > + const: mediatek,mt7628-usbphy
> > + then:
> > + required:
> > + - reg
>
> else:
> is it even valid?
Will add else with reg to false for the rest since looking into driver
code it looks like it is not using for other compatible at all.
>
> Anyway, please put allOf: block after required: block.
Will do.
>
> > +
> > +required:
> > + - compatible
> > + - "#phy-cells"
> > + - ralink,sysctl
> > + - resets
> > + - reset-names
> > +
>
>
> Best regards,
> Krzysztof
>
Thanks,
Sergio Paracuellos
deleted file mode 100644
@@ -1,23 +0,0 @@
-Mediatek/Ralink USB PHY
-
-Required properties:
- - compatible: "ralink,rt3352-usbphy"
- "mediatek,mt7620-usbphy"
- "mediatek,mt7628-usbphy"
- - reg: required for "mediatek,mt7628-usbphy", unused otherwise
- - #phy-cells: should be 0
- - ralink,sysctl: a phandle to a ralink syscon register region
- - resets: the two reset controllers for host and device
- - reset-names: the names of the 2 reset controllers
-
-Example:
-
-usbphy: phy {
- compatible = "mediatek,mt7628-usbphy";
- reg = <0x10120000 0x1000>;
- #phy-cells = <0>;
-
- ralink,sysctl = <&sysc>;
- resets = <&rstctrl 22 &rstctrl 25>;
- reset-names = "host", "device";
-};
new file mode 100644
@@ -0,0 +1,70 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/phy/ralink-usb-phy.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Mediatek/Ralink USB PHY
+
+maintainers:
+ - Sergio Paracuellos <sergio.paracuellos@gmail.com>
+
+properties:
+ compatible:
+ enum:
+ - ralink,rt3352-usbphy
+ - mediatek,mt7620-usbphy
+ - mediatek,mt7628-usbphy
+ reg:
+ maxItems: 1
+
+ "#phy-cells":
+ const: 0
+
+ ralink,sysctl:
+ description:
+ phandle to a ralink syscon register region.
+ $ref: /schemas/types.yaml#/definitions/phandle
+
+ resets:
+ items:
+ - description: USB Host reset controller
+ - description: USB Device reset controller
+
+ reset-names:
+ items:
+ - const: host
+ - const: device
+
+allOf:
+ - if:
+ properties:
+ compatible:
+ contains:
+ const: mediatek,mt7628-usbphy
+ then:
+ required:
+ - reg
+
+required:
+ - compatible
+ - "#phy-cells"
+ - ralink,sysctl
+ - resets
+ - reset-names
+
+additionalProperties: false
+
+examples:
+ - |
+ phy@10120000 {
+ compatible = "mediatek,mt7628-usbphy";
+ reg = <0x10120000 0x1000>;
+ #phy-cells = <0>;
+ ralink,sysctl = <&sysc>;
+ resets = <&rstctrl 22>,
+ <&rstctrl 25>;
+ reset-names = "host", "device";
+ };
+
+...