mbox series

[v2,0/2] dt-bindings: clock: update qcom,mmcc bindings for MSM8996

Message ID 20220705112734.1323355-1-dmitry.baryshkov@linaro.org
Headers show
Series dt-bindings: clock: update qcom,mmcc bindings for MSM8996 | expand

Message

Dmitry Baryshkov July 5, 2022, 11:27 a.m. UTC 
This is a split of binding patches from [1]. When sending the original
series I missed adding DT maintainers to the CC list, so while all the
DTS changes were accepted by Bjorn, the dt-bindings were not reviewed
(and were not taken in).

Changes since v1:
 - Moved the first condition block (clocks/clock-names requirement)
   under the allOf (requested by Krzysztof).

[1] https://lore.kernel.org/linux-arm-msm/20220617122922.769562-1-dmitry.baryshkov@linaro.org/

Dmitry Baryshkov (2):
  dt-bindings: clock: qcom,mmcc: fix clocks/clock-names definitions
  dt-bindings: clock: qcom,mmcc: define clocks/clock-names for MSM8996

 .../devicetree/bindings/clock/qcom,mmcc.yaml  | 178 ++++++++++++++----
 1 file changed, 146 insertions(+), 32 deletions(-)

Comments

Krzysztof Kozlowski July 5, 2022, 11:28 a.m. UTC | #1 
On 05/07/2022 13:27, Dmitry Baryshkov wrote:
> Rather than defining (incorrect) global clocks and clock-names lists,
> define them per platform using conditionals. Also, while we are at it,
> mark these properties as required for all platforms for which DT files
> contained clocks/clock-names for the MMCC nodes from the beginning (in
> addition to existing MSM8998 this adds MSM8994, SDM630 and SDM660).
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>


Best regards,
Krzysztof
Bjorn Andersson Aug. 29, 2022, 11:45 p.m. UTC | #2 
On Tue, 5 Jul 2022 14:27:32 +0300, Dmitry Baryshkov wrote:
> This is a split of binding patches from [1]. When sending the original
> series I missed adding DT maintainers to the CC list, so while all the
> DTS changes were accepted by Bjorn, the dt-bindings were not reviewed
> (and were not taken in).
> 
> Changes since v1:
>  - Moved the first condition block (clocks/clock-names requirement)
>    under the allOf (requested by Krzysztof).
> 
> [...]

Applied, thanks!

[1/2] dt-bindings: clock: qcom,mmcc: fix clocks/clock-names definitions
      commit: 73e66ddfd9984594c1f28fb756e68010924798ed
[2/2] dt-bindings: clock: qcom,mmcc: define clocks/clock-names for MSM8996
      commit: 51b0a5e044a4c49bcaf96c00123d26119da105d7

Best regards,