| Message ID | 20240518-dt-bindings-ti-soc-mfd-v1-1-b3952f104c9a@linaro.org |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | ti: fix dtbs_check for some syscon bindings and DTS node | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
On 18/05/2024 23:07, Krzysztof Kozlowski wrote: > Add dedicated binding for AM62 and AM62A wakeup system controller > registers, already used in the DTS to properly describe their children. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../bindings/soc/ti/ti,am62-system-controller.yaml | 77 ++++++++++++++++++++++ > 1 file changed, 77 insertions(+) > > diff --git a/Documentation/devicetree/bindings/soc/ti/ti,am62-system-controller.yaml b/Documentation/devicetree/bindings/soc/ti/ti,am62-system-controller.yaml > new file mode 100644 > index 000000000000..d3bd67717999 > --- /dev/null > +++ b/Documentation/devicetree/bindings/soc/ti/ti,am62-system-controller.yaml > @@ -0,0 +1,77 @@ > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/soc/ti/ti,am62-system-controller.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: TI AM62 System Controller Registers R/W > + > +maintainers: > + - Kishon Vijay Abraham I <kishon@ti.com> Above email might be invalid. Please use this instead Kishon Vijay Abraham I <kishon@kernel.org> > + - Roger Quadros <rogerq@kernel.org> > + > +properties: > + compatible: > + items: > + - enum: > + - ti,am62-system-controller > + - ti,am62a-system-controller > + - const: syscon > + - const: simple-mfd > + > + reg: > + maxItems: 1 > + > + "#address-cells": > + const: 1 > + > + "#size-cells": > + const: 1 > + > + ranges: true > + > +patternProperties: > + "^chipid@[0-9a-f]+$": > + type: object > + $ref: /schemas/hwinfo/ti,k3-socinfo.yaml# > + > + "^syscon@[0-9a-f]+$": > + type: object > + $ref: /schemas/mfd/syscon.yaml# > + unevaluatedProperties: false > + properties: > + compatible: > + items: > + - const: ti,am62-usb-phy-ctrl > + - const: syscon > + > +required: > + - compatible > + - reg > + - "#address-cells" > + - "#size-cells" > + - ranges > + > +additionalProperties: false > + > +examples: > + - | > + syscon@43000000 { > + compatible = "ti,am62-system-controller", "syscon", "simple-mfd"; > + reg = <0x43000000 0x20000>; > + bootph-all; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0x0 0x43000000 0x20000>; > + > + chipid@14 { > + compatible = "ti,am654-chipid"; > + reg = <0x14 0x4>; > + bootph-all; > + }; > + > + syscon@4008 { > + compatible = "ti,am62-usb-phy-ctrl", "syscon"; > + reg = <0x4008 0x4>; > + }; > + }; >
On 20/05/2024 13:35, Roger Quadros wrote: > > > On 18/05/2024 23:07, Krzysztof Kozlowski wrote: >> Add dedicated binding for AM62 and AM62A wakeup system controller >> registers, already used in the DTS to properly describe their children. >> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >> --- >> .../bindings/soc/ti/ti,am62-system-controller.yaml | 77 ++++++++++++++++++++++ >> 1 file changed, 77 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/soc/ti/ti,am62-system-controller.yaml b/Documentation/devicetree/bindings/soc/ti/ti,am62-system-controller.yaml >> new file mode 100644 >> index 000000000000..d3bd67717999 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/soc/ti/ti,am62-system-controller.yaml >> @@ -0,0 +1,77 @@ >> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/soc/ti/ti,am62-system-controller.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: TI AM62 System Controller Registers R/W >> + >> +maintainers: >> + - Kishon Vijay Abraham I <kishon@ti.com> > > Above email might be invalid. Please use this instead > > Kishon Vijay Abraham I <kishon@kernel.org> > Thanks. I took old email from bindings, so it would be nice if Kishon fixes his email everywhere (not only mailmap). Best regards, Krzysztof
On Sat, May 18, 2024 at 10:07:16PM +0200, Krzysztof Kozlowski wrote: > Add dedicated binding for AM62 and AM62A wakeup system controller > registers, already used in the DTS to properly describe their children. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Reviewed-by: Conor Dooley <conor.dooley@microchip.com> Cheers, Conor.
diff --git a/Documentation/devicetree/bindings/soc/ti/ti,am62-system-controller.yaml b/Documentation/devicetree/bindings/soc/ti/ti,am62-system-controller.yaml new file mode 100644 index 000000000000..d3bd67717999 --- /dev/null +++ b/Documentation/devicetree/bindings/soc/ti/ti,am62-system-controller.yaml @@ -0,0 +1,77 @@ +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/soc/ti/ti,am62-system-controller.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: TI AM62 System Controller Registers R/W + +maintainers: + - Kishon Vijay Abraham I <kishon@ti.com> + - Roger Quadros <rogerq@kernel.org> + +properties: + compatible: + items: + - enum: + - ti,am62-system-controller + - ti,am62a-system-controller + - const: syscon + - const: simple-mfd + + reg: + maxItems: 1 + + "#address-cells": + const: 1 + + "#size-cells": + const: 1 + + ranges: true + +patternProperties: + "^chipid@[0-9a-f]+$": + type: object + $ref: /schemas/hwinfo/ti,k3-socinfo.yaml# + + "^syscon@[0-9a-f]+$": + type: object + $ref: /schemas/mfd/syscon.yaml# + unevaluatedProperties: false + properties: + compatible: + items: + - const: ti,am62-usb-phy-ctrl + - const: syscon + +required: + - compatible + - reg + - "#address-cells" + - "#size-cells" + - ranges + +additionalProperties: false + +examples: + - | + syscon@43000000 { + compatible = "ti,am62-system-controller", "syscon", "simple-mfd"; + reg = <0x43000000 0x20000>; + bootph-all; + #address-cells = <1>; + #size-cells = <1>; + ranges = <0x0 0x43000000 0x20000>; + + chipid@14 { + compatible = "ti,am654-chipid"; + reg = <0x14 0x4>; + bootph-all; + }; + + syscon@4008 { + compatible = "ti,am62-usb-phy-ctrl", "syscon"; + reg = <0x4008 0x4>; + }; + };
Add dedicated binding for AM62 and AM62A wakeup system controller registers, already used in the DTS to properly describe their children. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- .../bindings/soc/ti/ti,am62-system-controller.yaml | 77 ++++++++++++++++++++++ 1 file changed, 77 insertions(+)