[PATCH 5/8] dt-bindings: clock: qcom: Add NSS clock controller for IPQ5424 SoC
Luo Jie
quic_luoj at quicinc.com
Mon Jun 23 06:19:29 PDT 2025
On 6/21/2025 6:09 PM, Konrad Dybcio wrote:
>>>> compatible:
>>>> - const: qcom,ipq9574-nsscc
>>>> + enum:
>>>> + - qcom,ipq5424-nsscc
>>>> + - qcom,ipq9574-nsscc
>>>> clocks:
>>>> items:
>>>> - description: Board XO source
>>>> - - description: CMN_PLL NSS 1200MHz (Bias PLL cc) clock source
>>>> - - description: CMN_PLL PPE 353MHz (Bias PLL ubi nc) clock source
>>>> + - description: CMN_PLL NSS 1200 MHz or 300 MHZ (Bias PLL cc) clock source
>>>> + - description: CMN_PLL PPE 353 MHz or 375 MHZ (Bias PLL ubi nc) clock source
>>> This change means devices are different. Just ocme with your own schema.
>> The NSS clock controller hardware block on the IPQ5424 SoC is identical
>> in design to that of the IPQ9574 SoC. The main difference is in the
>> clock rates for its two parent clocks sourced from the CMN PLL block.
>>
>> Given this, would it be acceptable to update the clock name and its
>> description to use a more generic clock name, such as "nss" and "ppe"
>> instead of the current "nss_1200" and "ppe_353"?
> Because you used those clock_names in the existing ipq9574, you can't
> change them now. You could introduce a separate set of clock_names
> for the new ipq5424 though, but I think it could be useful to drop the
> rate suffix for new additions
>
> Konrad
OK, Understand, I will add the new separate clock names "nss" and "ppe"
for supporting IPQ5424 SoC and further SoCs with similar design.
Thanks for confirmation.
More information about the linux-arm-kernel
mailing list