| Message ID | 20250731-dt-fsi-cleanups-v1-6-e7b695a29fc3@kernel.org |
|---|---|
| State | New |
| Headers | show |
| Series | ASpeed FSI DT clean-ups | expand |
On 7/31/25 17:12, Rob Herring (Arm) wrote: > Convert the ASpeed Coldfire offloaded GPIO FSI master. > > Drop the "fsi-master" compatible as it has not be used consistently and > doesn't represent anything. Acked-by: Eddie James <eajames@linux.ibm.com> > > Signed-off-by: Rob Herring (Arm) <robh@kernel.org> > --- > .../bindings/fsi/aspeed,ast2400-cf-fsi-master.yaml | 81 ++++++++++++++++++++++ > .../devicetree/bindings/fsi/fsi-master-ast-cf.txt | 36 ---------- > 2 files changed, 81 insertions(+), 36 deletions(-) > > diff --git a/Documentation/devicetree/bindings/fsi/aspeed,ast2400-cf-fsi-master.yaml b/Documentation/devicetree/bindings/fsi/aspeed,ast2400-cf-fsi-master.yaml > new file mode 100644 > index 000000000000..690b6c936f18 > --- /dev/null > +++ b/Documentation/devicetree/bindings/fsi/aspeed,ast2400-cf-fsi-master.yaml > @@ -0,0 +1,81 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/fsi/aspeed,ast2400-cf-fsi-master.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: ASpeed ColdFire offloaded GPIO-based FSI master > + > +maintainers: > + - Eddie James <eajames@linux.ibm.com> > + > +allOf: > + - $ref: /schemas/fsi/fsi-controller.yaml# > + > +properties: > + compatible: > + enum: > + - aspeed,ast2400-cf-fsi-master > + - aspeed,ast2500-cf-fsi-master > + > + clock-gpios: > + maxItems: 1 > + description: GPIO for FSI clock > + > + data-gpios: > + maxItems: 1 > + description: GPIO for FSI data signal > + > + enable-gpios: > + maxItems: 1 > + description: GPIO for enable signal > + > + trans-gpios: > + maxItems: 1 > + description: GPIO for voltage translator enable > + > + mux-gpios: > + maxItems: 1 > + description: > + GPIO for pin multiplexing with other functions (eg, external FSI masters) > + > + memory-region: > + maxItems: 1 > + description: > + Reference to the reserved memory for the ColdFire. Must be 2M aligned on > + AST2400 and 1M aligned on AST2500. > + > + aspeed,cvic: > + description: Reference to the CVIC node. > + $ref: /schemas/types.yaml#/definitions/phandle > + > + aspeed,sram: > + description: Reference to the SRAM node. > + $ref: /schemas/types.yaml#/definitions/phandle > + > +required: > + - compatible > + - clock-gpios > + - data-gpios > + - enable-gpios > + - trans-gpios > + - mux-gpios > + - memory-region > + - aspeed,cvic > + - aspeed,sram > + > +unevaluatedProperties: false > + > +examples: > + - | > + fsi-master { > + compatible = "aspeed,ast2500-cf-fsi-master"; > + clock-gpios = <&gpio 0>; > + data-gpios = <&gpio 1>; > + enable-gpios = <&gpio 2>; > + trans-gpios = <&gpio 3>; > + mux-gpios = <&gpio 4>; > + memory-region = <&coldfire_memory>; > + aspeed,cvic = <&cvic>; > + aspeed,sram = <&sram>; > + }; > diff --git a/Documentation/devicetree/bindings/fsi/fsi-master-ast-cf.txt b/Documentation/devicetree/bindings/fsi/fsi-master-ast-cf.txt > deleted file mode 100644 > index 3dc752db748b..000000000000 > --- a/Documentation/devicetree/bindings/fsi/fsi-master-ast-cf.txt > +++ /dev/null > @@ -1,36 +0,0 @@ > -Device-tree bindings for ColdFire offloaded gpio-based FSI master driver > ------------------------------------------------------------------------- > - > -Required properties: > - - compatible = > - "aspeed,ast2400-cf-fsi-master" for an AST2400 based system > - or > - "aspeed,ast2500-cf-fsi-master" for an AST2500 based system > - > - - clock-gpios = <gpio-descriptor>; : GPIO for FSI clock > - - data-gpios = <gpio-descriptor>; : GPIO for FSI data signal > - - enable-gpios = <gpio-descriptor>; : GPIO for enable signal > - - trans-gpios = <gpio-descriptor>; : GPIO for voltage translator enable > - - mux-gpios = <gpio-descriptor>; : GPIO for pin multiplexing with other > - functions (eg, external FSI masters) > - - memory-region = <phandle>; : Reference to the reserved memory for > - the ColdFire. Must be 2M aligned on > - AST2400 and 1M aligned on AST2500 > - - aspeed,sram = <phandle>; : Reference to the SRAM node. > - - aspeed,cvic = <phandle>; : Reference to the CVIC node. > - > -Examples: > - > - fsi-master { > - compatible = "aspeed,ast2500-cf-fsi-master", "fsi-master"; > - > - clock-gpios = <&gpio 0>; > - data-gpios = <&gpio 1>; > - enable-gpios = <&gpio 2>; > - trans-gpios = <&gpio 3>; > - mux-gpios = <&gpio 4>; > - > - memory-region = <&coldfire_memory>; > - aspeed,sram = <&sram>; > - aspeed,cvic = <&cvic>; > - } >
diff --git a/Documentation/devicetree/bindings/fsi/aspeed,ast2400-cf-fsi-master.yaml b/Documentation/devicetree/bindings/fsi/aspeed,ast2400-cf-fsi-master.yaml new file mode 100644 index 000000000000..690b6c936f18 --- /dev/null +++ b/Documentation/devicetree/bindings/fsi/aspeed,ast2400-cf-fsi-master.yaml @@ -0,0 +1,81 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/fsi/aspeed,ast2400-cf-fsi-master.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: ASpeed ColdFire offloaded GPIO-based FSI master + +maintainers: + - Eddie James <eajames@linux.ibm.com> + +allOf: + - $ref: /schemas/fsi/fsi-controller.yaml# + +properties: + compatible: + enum: + - aspeed,ast2400-cf-fsi-master + - aspeed,ast2500-cf-fsi-master + + clock-gpios: + maxItems: 1 + description: GPIO for FSI clock + + data-gpios: + maxItems: 1 + description: GPIO for FSI data signal + + enable-gpios: + maxItems: 1 + description: GPIO for enable signal + + trans-gpios: + maxItems: 1 + description: GPIO for voltage translator enable + + mux-gpios: + maxItems: 1 + description: + GPIO for pin multiplexing with other functions (eg, external FSI masters) + + memory-region: + maxItems: 1 + description: + Reference to the reserved memory for the ColdFire. Must be 2M aligned on + AST2400 and 1M aligned on AST2500. + + aspeed,cvic: + description: Reference to the CVIC node. + $ref: /schemas/types.yaml#/definitions/phandle + + aspeed,sram: + description: Reference to the SRAM node. + $ref: /schemas/types.yaml#/definitions/phandle + +required: + - compatible + - clock-gpios + - data-gpios + - enable-gpios + - trans-gpios + - mux-gpios + - memory-region + - aspeed,cvic + - aspeed,sram + +unevaluatedProperties: false + +examples: + - | + fsi-master { + compatible = "aspeed,ast2500-cf-fsi-master"; + clock-gpios = <&gpio 0>; + data-gpios = <&gpio 1>; + enable-gpios = <&gpio 2>; + trans-gpios = <&gpio 3>; + mux-gpios = <&gpio 4>; + memory-region = <&coldfire_memory>; + aspeed,cvic = <&cvic>; + aspeed,sram = <&sram>; + }; diff --git a/Documentation/devicetree/bindings/fsi/fsi-master-ast-cf.txt b/Documentation/devicetree/bindings/fsi/fsi-master-ast-cf.txt deleted file mode 100644 index 3dc752db748b..000000000000 --- a/Documentation/devicetree/bindings/fsi/fsi-master-ast-cf.txt +++ /dev/null @@ -1,36 +0,0 @@ -Device-tree bindings for ColdFire offloaded gpio-based FSI master driver ------------------------------------------------------------------------- - -Required properties: - - compatible = - "aspeed,ast2400-cf-fsi-master" for an AST2400 based system - or - "aspeed,ast2500-cf-fsi-master" for an AST2500 based system - - - clock-gpios = <gpio-descriptor>; : GPIO for FSI clock - - data-gpios = <gpio-descriptor>; : GPIO for FSI data signal - - enable-gpios = <gpio-descriptor>; : GPIO for enable signal - - trans-gpios = <gpio-descriptor>; : GPIO for voltage translator enable - - mux-gpios = <gpio-descriptor>; : GPIO for pin multiplexing with other - functions (eg, external FSI masters) - - memory-region = <phandle>; : Reference to the reserved memory for - the ColdFire. Must be 2M aligned on - AST2400 and 1M aligned on AST2500 - - aspeed,sram = <phandle>; : Reference to the SRAM node. - - aspeed,cvic = <phandle>; : Reference to the CVIC node. - -Examples: - - fsi-master { - compatible = "aspeed,ast2500-cf-fsi-master", "fsi-master"; - - clock-gpios = <&gpio 0>; - data-gpios = <&gpio 1>; - enable-gpios = <&gpio 2>; - trans-gpios = <&gpio 3>; - mux-gpios = <&gpio 4>; - - memory-region = <&coldfire_memory>; - aspeed,sram = <&sram>; - aspeed,cvic = <&cvic>; - }
Convert the ASpeed Coldfire offloaded GPIO FSI master. Drop the "fsi-master" compatible as it has not be used consistently and doesn't represent anything. Signed-off-by: Rob Herring (Arm) <robh@kernel.org> --- .../bindings/fsi/aspeed,ast2400-cf-fsi-master.yaml | 81 ++++++++++++++++++++++ .../devicetree/bindings/fsi/fsi-master-ast-cf.txt | 36 ---------- 2 files changed, 81 insertions(+), 36 deletions(-)