| Message ID | 20221224214404.18280-1-krzysztof.kozlowski@linaro.org |
|---|---|
| State | Accepted |
| Commit | f6f10b4ae5e5c986f521cc2282721f3c1b158df4 |
| Headers | show |
| Series | dt-bindings: clock: qcom,camcc-sm8250: extend clocks and power domains | expand |
Quoting Krzysztof Kozlowski (2022-12-24 13:44:04) > Add clocks and properties (power-domains, required-opps) already used in > SM8250 DTS: > > sm8250-hdk.dtb: clock-controller@ad00000: clocks: [[46, 10], [44, 0], [44, 1], [45]] is too long > sm8250-hdk.dtb: clock-controller@ad00000: clock-names:0: 'bi_tcxo' was expected > sm8250-hdk.dtb: clock-controller@ad00000: 'power-domains', 'required-opps' do not match any of the regexes: 'pinctrl-[0-9]+' > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- Reviewed-by: Stephen Boyd <sboyd@kernel.org>
On Sat, 24 Dec 2022 22:44:04 +0100, Krzysztof Kozlowski wrote: > Add clocks and properties (power-domains, required-opps) already used in > SM8250 DTS: > > sm8250-hdk.dtb: clock-controller@ad00000: clocks: [[46, 10], [44, 0], [44, 1], [45]] is too long > sm8250-hdk.dtb: clock-controller@ad00000: clock-names:0: 'bi_tcxo' was expected > sm8250-hdk.dtb: clock-controller@ad00000: 'power-domains', 'required-opps' do not match any of the regexes: 'pinctrl-[0-9]+' > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../bindings/clock/qcom,camcc-sm8250.yaml | 20 +++++++++++++++++-- > 1 file changed, 18 insertions(+), 2 deletions(-) > Acked-by: Rob Herring <robh@kernel.org>
On Sat, 24 Dec 2022 22:44:04 +0100, Krzysztof Kozlowski wrote: > Add clocks and properties (power-domains, required-opps) already used in > SM8250 DTS: > > sm8250-hdk.dtb: clock-controller@ad00000: clocks: [[46, 10], [44, 0], [44, 1], [45]] is too long > sm8250-hdk.dtb: clock-controller@ad00000: clock-names:0: 'bi_tcxo' was expected > sm8250-hdk.dtb: clock-controller@ad00000: 'power-domains', 'required-opps' do not match any of the regexes: 'pinctrl-[0-9]+' > > [...] Applied, thanks! [1/1] dt-bindings: clock: qcom,camcc-sm8250: extend clocks and power domains commit: f6f10b4ae5e5c986f521cc2282721f3c1b158df4 Best regards,
diff --git a/Documentation/devicetree/bindings/clock/qcom,camcc-sm8250.yaml b/Documentation/devicetree/bindings/clock/qcom,camcc-sm8250.yaml index 93ec1f598e6e..426335a2841c 100644 --- a/Documentation/devicetree/bindings/clock/qcom,camcc-sm8250.yaml +++ b/Documentation/devicetree/bindings/clock/qcom,camcc-sm8250.yaml @@ -21,12 +21,16 @@ properties: clocks: items: + - description: AHB - description: Board XO source + - description: Board active XO source - description: Sleep clock source clock-names: items: + - const: iface - const: bi_tcxo + - const: bi_tcxo_ao - const: sleep_clk '#clock-cells': @@ -38,9 +42,18 @@ properties: '#power-domain-cells': const: 1 + power-domains: + items: + - description: MMCX power domain + reg: maxItems: 1 + required-opps: + maxItems: 1 + description: + OPP node describing required MMCX performance point. + required: - compatible - reg @@ -54,13 +67,16 @@ additionalProperties: false examples: - | + #include <dt-bindings/clock/qcom,gcc-sm8250.h> #include <dt-bindings/clock/qcom,rpmh.h> clock-controller@ad00000 { compatible = "qcom,sm8250-camcc"; reg = <0x0ad00000 0x10000>; - clocks = <&rpmhcc RPMH_CXO_CLK>, + clocks = <&gcc GCC_CAMERA_AHB_CLK>, + <&rpmhcc RPMH_CXO_CLK>, + <&rpmhcc RPMH_CXO_CLK_A>, <&sleep_clk>; - clock-names = "bi_tcxo", "sleep_clk"; + clock-names = "iface", "bi_tcxo", "bi_tcxo_ao", "sleep_clk"; #clock-cells = <1>; #reset-cells = <1>; #power-domain-cells = <1>;
Add clocks and properties (power-domains, required-opps) already used in SM8250 DTS: sm8250-hdk.dtb: clock-controller@ad00000: clocks: [[46, 10], [44, 0], [44, 1], [45]] is too long sm8250-hdk.dtb: clock-controller@ad00000: clock-names:0: 'bi_tcxo' was expected sm8250-hdk.dtb: clock-controller@ad00000: 'power-domains', 'required-opps' do not match any of the regexes: 'pinctrl-[0-9]+' Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- .../bindings/clock/qcom,camcc-sm8250.yaml | 20 +++++++++++++++++-- 1 file changed, 18 insertions(+), 2 deletions(-)