| Message ID | 20250416-02-k1-pinctrl-clk-v2-1-2b5fcbd4183c@gentoo.org |
|---|---|
| State | New |
| Headers | show |
| Series | [v2,1/2] dt-bindings: pinctrl: spacemit: add clock and reset property | expand |
On 4/15/25 7:15 PM, Yixun Lan wrote: > SpacemiT K1 SoC's pinctrl controller requires two clocks in order > to work properly, also has one reset line from hardware perspective. You mention the reset line here but that isn't implemented by this patch (or the next). I assume the reset series will be merged soon, and if that's the case, maybe you can add two more patches to incorporate that in the next version of this series. Other than the description, this looks good to me. You'll want one of the DT maintainers to sign off, though. -Alex > > Signed-off-by: Yixun Lan <dlan@gentoo.org> > --- > .../bindings/pinctrl/spacemit,k1-pinctrl.yaml | 18 ++++++++++++++++++ > 1 file changed, 18 insertions(+) > > diff --git a/Documentation/devicetree/bindings/pinctrl/spacemit,k1-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/spacemit,k1-pinctrl.yaml > index b01ecd83b71b5e9f47256adde23bc8b183bef05e..d80e88aa07b45f4a3d90e8217caf93d32655927d 100644 > --- a/Documentation/devicetree/bindings/pinctrl/spacemit,k1-pinctrl.yaml > +++ b/Documentation/devicetree/bindings/pinctrl/spacemit,k1-pinctrl.yaml > @@ -17,6 +17,19 @@ properties: > items: > - description: pinctrl io memory base > > + clocks: > + items: > + - description: Functional Clock > + - description: Bus Clock > + > + clock-names: > + items: > + - const: func > + - const: bus > + > + resets: > + maxItems: 1 > + > patternProperties: > '-cfg$': > type: object > @@ -94,6 +107,8 @@ patternProperties: > required: > - compatible > - reg > + - clocks > + - clock-names > > additionalProperties: false > > @@ -108,6 +123,9 @@ examples: > pinctrl@d401e000 { > compatible = "spacemit,k1-pinctrl"; > reg = <0x0 0xd401e000 0x0 0x400>; > + clocks = <&syscon_apbc 42>, > + <&syscon_apbc 94>; > + clock-names = "func", "bus"; > > uart0_2_cfg: uart0-2-cfg { > uart0-2-pins { >
On Wed, 16 Apr 2025 08:15:27 +0800, Yixun Lan wrote: > SpacemiT K1 SoC's pinctrl controller requires two clocks in order > to work properly, also has one reset line from hardware perspective. > > Signed-off-by: Yixun Lan <dlan@gentoo.org> > --- > .../bindings/pinctrl/spacemit,k1-pinctrl.yaml | 18 ++++++++++++++++++ > 1 file changed, 18 insertions(+) > Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
diff --git a/Documentation/devicetree/bindings/pinctrl/spacemit,k1-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/spacemit,k1-pinctrl.yaml index b01ecd83b71b5e9f47256adde23bc8b183bef05e..d80e88aa07b45f4a3d90e8217caf93d32655927d 100644 --- a/Documentation/devicetree/bindings/pinctrl/spacemit,k1-pinctrl.yaml +++ b/Documentation/devicetree/bindings/pinctrl/spacemit,k1-pinctrl.yaml @@ -17,6 +17,19 @@ properties: items: - description: pinctrl io memory base + clocks: + items: + - description: Functional Clock + - description: Bus Clock + + clock-names: + items: + - const: func + - const: bus + + resets: + maxItems: 1 + patternProperties: '-cfg$': type: object @@ -94,6 +107,8 @@ patternProperties: required: - compatible - reg + - clocks + - clock-names additionalProperties: false @@ -108,6 +123,9 @@ examples: pinctrl@d401e000 { compatible = "spacemit,k1-pinctrl"; reg = <0x0 0xd401e000 0x0 0x400>; + clocks = <&syscon_apbc 42>, + <&syscon_apbc 94>; + clock-names = "func", "bus"; uart0_2_cfg: uart0-2-cfg { uart0-2-pins {
SpacemiT K1 SoC's pinctrl controller requires two clocks in order to work properly, also has one reset line from hardware perspective. Signed-off-by: Yixun Lan <dlan@gentoo.org> --- .../bindings/pinctrl/spacemit,k1-pinctrl.yaml | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+)