| Message ID | 20230113090739.45805-2-dmitry.baryshkov@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | clk: qcom: msm8996: add APCS clock driver | expand |
On 13/01/2023 10:07, Dmitry Baryshkov wrote: > Only three platforms require `pll' and `aux' clocks: msm8916, msm8939 > and qcs404. Correct the list of platforms in the corresponding clause. > > Fixes: 0d17014e9189 ("dt-bindings: mailbox: Add binding for SDX55 APCS") > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > --- > .../bindings/mailbox/qcom,apcs-kpss-global.yaml | 9 +-------- > 1 file changed, 1 insertion(+), 8 deletions(-) > > diff --git a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml > index 943f9472ae10..b8a44ef0540f 100644 > --- a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml > +++ b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml > @@ -71,15 +71,8 @@ allOf: > compatible: > enum: > - qcom,msm8916-apcs-kpss-global > - - qcom,msm8994-apcs-kpss-global > - - qcom,msm8996-apcs-hmss-global > - - qcom,msm8998-apcs-hmss-global > + - qcom,msm8939-apcs-kpss-global > - qcom,qcs404-apcs-apps-global > - - qcom,sc7180-apss-shared > - - qcom,sdm660-apcs-hmss-global > - - qcom,sdm845-apss-shared > - - qcom,sm6125-apcs-hmss-global > - - qcom,sm8150-apss-shared And what in other case? Any clocks? They should be moved to their own if forbidding the clocks and clock-names. Best regards, Krzysztof
On 17/01/2023 13:34, Krzysztof Kozlowski wrote: > On 13/01/2023 10:07, Dmitry Baryshkov wrote: >> Only three platforms require `pll' and `aux' clocks: msm8916, msm8939 >> and qcs404. Correct the list of platforms in the corresponding clause. >> >> Fixes: 0d17014e9189 ("dt-bindings: mailbox: Add binding for SDX55 APCS") >> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> >> --- >> .../bindings/mailbox/qcom,apcs-kpss-global.yaml | 9 +-------- >> 1 file changed, 1 insertion(+), 8 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml >> index 943f9472ae10..b8a44ef0540f 100644 >> --- a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml >> +++ b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml >> @@ -71,15 +71,8 @@ allOf: >> compatible: >> enum: >> - qcom,msm8916-apcs-kpss-global >> - - qcom,msm8994-apcs-kpss-global >> - - qcom,msm8996-apcs-hmss-global >> - - qcom,msm8998-apcs-hmss-global >> + - qcom,msm8939-apcs-kpss-global >> - qcom,qcs404-apcs-apps-global >> - - qcom,sc7180-apss-shared >> - - qcom,sdm660-apcs-hmss-global >> - - qcom,sdm845-apss-shared >> - - qcom,sm6125-apcs-hmss-global >> - - qcom,sm8150-apss-shared > > And what in other case? Any clocks? They should be moved to their own if > forbidding the clocks and clock-names. Ack
diff --git a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml index 943f9472ae10..b8a44ef0540f 100644 --- a/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml +++ b/Documentation/devicetree/bindings/mailbox/qcom,apcs-kpss-global.yaml @@ -71,15 +71,8 @@ allOf: compatible: enum: - qcom,msm8916-apcs-kpss-global - - qcom,msm8994-apcs-kpss-global - - qcom,msm8996-apcs-hmss-global - - qcom,msm8998-apcs-hmss-global + - qcom,msm8939-apcs-kpss-global - qcom,qcs404-apcs-apps-global - - qcom,sc7180-apss-shared - - qcom,sdm660-apcs-hmss-global - - qcom,sdm845-apss-shared - - qcom,sm6125-apcs-hmss-global - - qcom,sm8150-apss-shared then: properties: clocks:
Only three platforms require `pll' and `aux' clocks: msm8916, msm8939 and qcs404. Correct the list of platforms in the corresponding clause. Fixes: 0d17014e9189 ("dt-bindings: mailbox: Add binding for SDX55 APCS") Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> --- .../bindings/mailbox/qcom,apcs-kpss-global.yaml | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-)