| Message ID | 20230805135318.6102-1-fr0st61te@gmail.com |
|---|---|
| State | Accepted, archived |
| Headers | show |
| Series | [v4] dt-bindings: net: ftgmac100: convert to yaml version from txt | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | warning | total: 0 errors, 2 warnings, 102 lines checked |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On Sat, 2023-08-05 at 16:53 +0300, Ivan Mikhaylov wrote: > Conversion from ftgmac100.txt to yaml format version. > > Signed-off-by: Ivan Mikhaylov <fr0st61te@gmail.com> > --- > .../bindings/net/faraday,ftgmac100.yaml | 102 > ++++++++++++++++++ > .../devicetree/bindings/net/ftgmac100.txt | 67 ------------ > 2 files changed, 102 insertions(+), 67 deletions(-) > create mode 100644 > Documentation/devicetree/bindings/net/faraday,ftgmac100.yaml > delete mode 100644 > Documentation/devicetree/bindings/net/ftgmac100.txt > > diff --git > a/Documentation/devicetree/bindings/net/faraday,ftgmac100.yaml > b/Documentation/devicetree/bindings/net/faraday,ftgmac100.yaml > new file mode 100644 > index 000000000000..9bcbacb6640d > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/faraday,ftgmac100.yaml > @@ -0,0 +1,102 @@ > +# SPDX-License-Identifier: GPL-2.0 > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/net/faraday,ftgmac100.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Faraday Technology FTGMAC100 gigabit ethernet controller > + > +allOf: > + - $ref: ethernet-controller.yaml# > + > +maintainers: > + - Po-Yu Chuang <ratbert@faraday-tech.com> > + > +properties: > + compatible: > + oneOf: > + - const: faraday,ftgmac100 > + - items: > + - enum: > + - aspeed,ast2400-mac > + - aspeed,ast2500-mac > + - aspeed,ast2600-mac > + - const: faraday,ftgmac100 > + > + reg: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + clocks: > + minItems: 1 > + items: > + - description: MAC IP clock > + - description: RMII RCLK gate for AST2500/2600 > + > + clock-names: > + minItems: 1 > + items: > + - const: MACCLK > + - const: RCLK > + > + phy-mode: > + enum: > + - rgmii > + - rmii > + > + phy-handle: true > + > + use-ncsi: > + description: > + Use the NC-SI stack instead of an MDIO PHY. Currently assumes > + rmii (100bT) but kept as a separate property in case NC-SI > grows support > + for a gigabit link. > + type: boolean > + > + no-hw-checksum: > + description: > + Used to disable HW checksum support. Here for backward > + compatibility as the driver now should have correct defaults > based on > + the SoC. > + type: boolean > + deprecated: true > + > + mdio: > + $ref: /schemas/net/mdio.yaml# > + > +required: > + - compatible > + - reg > + - interrupts > + > +unevaluatedProperties: false > + > +examples: > + - | > + ethernet@1e660000 { > + compatible = "aspeed,ast2500-mac", "faraday,ftgmac100"; > + reg = <0x1e660000 0x180>; > + interrupts = <2>; > + use-ncsi; > + }; > + > + ethernet@1e680000 { > + compatible = "aspeed,ast2500-mac", "faraday,ftgmac100"; > + reg = <0x1e680000 0x180>; > + interrupts = <2>; > + > + phy-handle = <&phy>; > + phy-mode = "rgmii"; > + > + mdio { > + #address-cells = <1>; > + #size-cells = <0>; > + > + phy: ethernet-phy@1 { > + compatible = "ethernet-phy-ieee802.3-c22"; > + reg = <1>; > + }; > + }; > + }; > diff --git a/Documentation/devicetree/bindings/net/ftgmac100.txt > b/Documentation/devicetree/bindings/net/ftgmac100.txt > deleted file mode 100644 > index 29234021f601..000000000000 > --- a/Documentation/devicetree/bindings/net/ftgmac100.txt > +++ /dev/null > @@ -1,67 +0,0 @@ > -* Faraday Technology FTGMAC100 gigabit ethernet controller > - > -Required properties: > -- compatible: "faraday,ftgmac100" > - > - Must also contain one of these if used as part of an Aspeed > AST2400 > - or 2500 family SoC as they have some subtle tweaks to the > - implementation: > - > - - "aspeed,ast2400-mac" > - - "aspeed,ast2500-mac" > - - "aspeed,ast2600-mac" > - > -- reg: Address and length of the register set for the device > -- interrupts: Should contain ethernet controller interrupt > - > -Optional properties: > -- phy-handle: See ethernet.txt file in the same directory. > -- phy-mode: See ethernet.txt file in the same directory. If the > property is > - absent, "rgmii" is assumed. Supported values are "rgmii*" and > "rmii" for > - aspeed parts. Other (unknown) parts will accept any value. > -- use-ncsi: Use the NC-SI stack instead of an MDIO PHY. Currently > assumes > - rmii (100bT) but kept as a separate property in case NC-SI grows > support > - for a gigabit link. > -- no-hw-checksum: Used to disable HW checksum support. Here for > backward > - compatibility as the driver now should have correct defaults based > on > - the SoC. > -- clocks: In accordance with the generic clock bindings. Must > describe the MAC > - IP clock, and optionally an RMII RCLK gate for the > AST2500/AST2600. The > - required MAC clock must be the first cell. > -- clock-names: > - > - - "MACCLK": The MAC IP clock > - - "RCLK": Clock gate for the RMII RCLK > - > -Optional subnodes: > -- mdio: See mdio.txt file in the same directory. > - > -Example: > - > - mac0: ethernet@1e660000 { > - compatible = "aspeed,ast2500-mac", > "faraday,ftgmac100"; > - reg = <0x1e660000 0x180>; > - interrupts = <2>; > - use-ncsi; > - }; > - > -Example with phy-handle: > - > - mac1: ethernet@1e680000 { > - compatible = "aspeed,ast2500-mac", > "faraday,ftgmac100"; > - reg = <0x1e680000 0x180>; > - interrupts = <2>; > - > - phy-handle = <&phy>; > - phy-mode = "rgmii"; > - > - mdio { > - #address-cells = <1>; > - #size-cells = <0>; > - > - phy: ethernet-phy@1 { > - compatible = "ethernet-phy-ieee802.3- > c22"; > - reg = <1>; > - }; > - }; > - }; Gentle ping, I think I resolved all comments related, anything else? Thanks.
On 05/08/2023 15:53, Ivan Mikhaylov wrote: > Conversion from ftgmac100.txt to yaml format version. > > Signed-off-by: Ivan Mikhaylov <fr0st61te@gmail.com> > --- > .../bindings/net/faraday,ftgmac100.yaml | 102 ++++++++++++++++++ Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On Sat, 05 Aug 2023 16:53:18 +0300, Ivan Mikhaylov wrote: > Conversion from ftgmac100.txt to yaml format version. > > Signed-off-by: Ivan Mikhaylov <fr0st61te@gmail.com> > --- > .../bindings/net/faraday,ftgmac100.yaml | 102 ++++++++++++++++++ > .../devicetree/bindings/net/ftgmac100.txt | 67 ------------ > 2 files changed, 102 insertions(+), 67 deletions(-) > create mode 100644 Documentation/devicetree/bindings/net/faraday,ftgmac100.yaml > delete mode 100644 Documentation/devicetree/bindings/net/ftgmac100.txt > Applied, thanks!
diff --git a/Documentation/devicetree/bindings/net/faraday,ftgmac100.yaml b/Documentation/devicetree/bindings/net/faraday,ftgmac100.yaml new file mode 100644 index 000000000000..9bcbacb6640d --- /dev/null +++ b/Documentation/devicetree/bindings/net/faraday,ftgmac100.yaml @@ -0,0 +1,102 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/net/faraday,ftgmac100.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Faraday Technology FTGMAC100 gigabit ethernet controller + +allOf: + - $ref: ethernet-controller.yaml# + +maintainers: + - Po-Yu Chuang <ratbert@faraday-tech.com> + +properties: + compatible: + oneOf: + - const: faraday,ftgmac100 + - items: + - enum: + - aspeed,ast2400-mac + - aspeed,ast2500-mac + - aspeed,ast2600-mac + - const: faraday,ftgmac100 + + reg: + maxItems: 1 + + interrupts: + maxItems: 1 + + clocks: + minItems: 1 + items: + - description: MAC IP clock + - description: RMII RCLK gate for AST2500/2600 + + clock-names: + minItems: 1 + items: + - const: MACCLK + - const: RCLK + + phy-mode: + enum: + - rgmii + - rmii + + phy-handle: true + + use-ncsi: + description: + Use the NC-SI stack instead of an MDIO PHY. Currently assumes + rmii (100bT) but kept as a separate property in case NC-SI grows support + for a gigabit link. + type: boolean + + no-hw-checksum: + description: + Used to disable HW checksum support. Here for backward + compatibility as the driver now should have correct defaults based on + the SoC. + type: boolean + deprecated: true + + mdio: + $ref: /schemas/net/mdio.yaml# + +required: + - compatible + - reg + - interrupts + +unevaluatedProperties: false + +examples: + - | + ethernet@1e660000 { + compatible = "aspeed,ast2500-mac", "faraday,ftgmac100"; + reg = <0x1e660000 0x180>; + interrupts = <2>; + use-ncsi; + }; + + ethernet@1e680000 { + compatible = "aspeed,ast2500-mac", "faraday,ftgmac100"; + reg = <0x1e680000 0x180>; + interrupts = <2>; + + phy-handle = <&phy>; + phy-mode = "rgmii"; + + mdio { + #address-cells = <1>; + #size-cells = <0>; + + phy: ethernet-phy@1 { + compatible = "ethernet-phy-ieee802.3-c22"; + reg = <1>; + }; + }; + }; diff --git a/Documentation/devicetree/bindings/net/ftgmac100.txt b/Documentation/devicetree/bindings/net/ftgmac100.txt deleted file mode 100644 index 29234021f601..000000000000 --- a/Documentation/devicetree/bindings/net/ftgmac100.txt +++ /dev/null @@ -1,67 +0,0 @@ -* Faraday Technology FTGMAC100 gigabit ethernet controller - -Required properties: -- compatible: "faraday,ftgmac100" - - Must also contain one of these if used as part of an Aspeed AST2400 - or 2500 family SoC as they have some subtle tweaks to the - implementation: - - - "aspeed,ast2400-mac" - - "aspeed,ast2500-mac" - - "aspeed,ast2600-mac" - -- reg: Address and length of the register set for the device -- interrupts: Should contain ethernet controller interrupt - -Optional properties: -- phy-handle: See ethernet.txt file in the same directory. -- phy-mode: See ethernet.txt file in the same directory. If the property is - absent, "rgmii" is assumed. Supported values are "rgmii*" and "rmii" for - aspeed parts. Other (unknown) parts will accept any value. -- use-ncsi: Use the NC-SI stack instead of an MDIO PHY. Currently assumes - rmii (100bT) but kept as a separate property in case NC-SI grows support - for a gigabit link. -- no-hw-checksum: Used to disable HW checksum support. Here for backward - compatibility as the driver now should have correct defaults based on - the SoC. -- clocks: In accordance with the generic clock bindings. Must describe the MAC - IP clock, and optionally an RMII RCLK gate for the AST2500/AST2600. The - required MAC clock must be the first cell. -- clock-names: - - - "MACCLK": The MAC IP clock - - "RCLK": Clock gate for the RMII RCLK - -Optional subnodes: -- mdio: See mdio.txt file in the same directory. - -Example: - - mac0: ethernet@1e660000 { - compatible = "aspeed,ast2500-mac", "faraday,ftgmac100"; - reg = <0x1e660000 0x180>; - interrupts = <2>; - use-ncsi; - }; - -Example with phy-handle: - - mac1: ethernet@1e680000 { - compatible = "aspeed,ast2500-mac", "faraday,ftgmac100"; - reg = <0x1e680000 0x180>; - interrupts = <2>; - - phy-handle = <&phy>; - phy-mode = "rgmii"; - - mdio { - #address-cells = <1>; - #size-cells = <0>; - - phy: ethernet-phy@1 { - compatible = "ethernet-phy-ieee802.3-c22"; - reg = <1>; - }; - }; - };
Conversion from ftgmac100.txt to yaml format version. Signed-off-by: Ivan Mikhaylov <fr0st61te@gmail.com> --- .../bindings/net/faraday,ftgmac100.yaml | 102 ++++++++++++++++++ .../devicetree/bindings/net/ftgmac100.txt | 67 ------------ 2 files changed, 102 insertions(+), 67 deletions(-) create mode 100644 Documentation/devicetree/bindings/net/faraday,ftgmac100.yaml delete mode 100644 Documentation/devicetree/bindings/net/ftgmac100.txt