Drop required-opps from required list on SM8650 videocc, camcc

[PATCH V4 0/2] Drop required-opps from required list on SM8650 videocc, camcc

Posted by Jagadeesh Kona 1 year, 6 months ago

On SM8650, the minimum voltage corner supported on MMCX from cmd-db is
sufficient for clock controllers to operate and there is no need to specify
the required-opps. Hence remove the required-opps property from the list of
required properties for SM8650 camcc and videocc bindings.

Changes in V4:
 - Dropped change to disallow required-opps on variants that don't need it
 - Split the change into separate commits for videocc and camcc
 - Link to V3: https://lore.kernel.org/all/20240730034552.31271-1-quic_jkona@quicinc.com/
Changes in V3:
 - Made only required-opps property conditional and added it based on the variant
 - Link to V2: https://lore.kernel.org/all/20240720052818.26441-1-quic_jkona@quicinc.com/
Changes in V2:
 - Made required: conditional and dropped required-opps from it only for SM8650 platform
 - Dropped Krzysztof Acked-by tag due to above changes
 - Link to V1: https://lore.kernel.org/all/20240708130836.19273-1-quic_jkona@quicinc.com/

Jagadeesh Kona (2):
  dt-bindings: clock: qcom: Drop required-opps in required on sm8650
    videocc
  dt-bindings: clock: qcom: Drop required-opps in required on SM8650
    camcc

 .../bindings/clock/qcom,sm8450-camcc.yaml     | 19 +++++++++++++++----
 .../bindings/clock/qcom,sm8450-videocc.yaml   | 11 ++++++++++-
 2 files changed, 25 insertions(+), 5 deletions(-)

-- 
2.45.2

Re: [PATCH V4 0/2] Drop required-opps from required list on SM8650 videocc, camcc

Posted by Bjorn Andersson 1 year, 5 months ago

On Thu, 01 Aug 2024 12:14:46 +0530, Jagadeesh Kona wrote:
> On SM8650, the minimum voltage corner supported on MMCX from cmd-db is
> sufficient for clock controllers to operate and there is no need to specify
> the required-opps. Hence remove the required-opps property from the list of
> required properties for SM8650 camcc and videocc bindings.
> 
> Changes in V4:
>  - Dropped change to disallow required-opps on variants that don't need it
>  - Split the change into separate commits for videocc and camcc
>  - Link to V3: https://lore.kernel.org/all/20240730034552.31271-1-quic_jkona@quicinc.com/
> Changes in V3:
>  - Made only required-opps property conditional and added it based on the variant
>  - Link to V2: https://lore.kernel.org/all/20240720052818.26441-1-quic_jkona@quicinc.com/
> Changes in V2:
>  - Made required: conditional and dropped required-opps from it only for SM8650 platform
>  - Dropped Krzysztof Acked-by tag due to above changes
>  - Link to V1: https://lore.kernel.org/all/20240708130836.19273-1-quic_jkona@quicinc.com/
> 
> [...]

Applied, thanks!

[1/2] dt-bindings: clock: qcom: Drop required-opps in required on sm8650 videocc
      commit: 6720e8dbcb1b6b3b43e38998b522088814ae1268
[2/2] dt-bindings: clock: qcom: Drop required-opps in required on SM8650 camcc
      commit: db30c1160ca5f115476cd8c8008f67572acb2c08

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>