On 21.02.2025 11:14 AM, Manikanta Mylavarapu wrote: > From: Devi Priya <quic_devipriy@xxxxxxxxxxx> > > Add NSSCC clock and reset definitions for ipq9574. > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > Signed-off-by: Devi Priya <quic_devipriy@xxxxxxxxxxx> > Signed-off-by: Manikanta Mylavarapu <quic_mmanikan@xxxxxxxxxxx> > --- [...] > + - | > + #include <dt-bindings/clock/qcom,ipq9574-gcc.h> > + #include <dt-bindings/clock/qcom,ipq-cmn-pll.h> > + clock-controller@39b00000 { > + compatible = "qcom,ipq9574-nsscc"; > + reg = <0x39b00000 0x80000>; > + clocks = <&xo_board_clk>, > + <&cmn_pll NSS_1200MHZ_CLK>, > + <&cmn_pll PPE_353MHZ_CLK>, > + <&gcc GPLL0_OUT_AUX>, > + <&uniphy 0>, > + <&uniphy 1>, > + <&uniphy 2>, > + <&uniphy 3>, > + <&uniphy 4>, > + <&uniphy 5>, > + <&gcc GCC_NSSCC_CLK>; > + clock-names = "xo", > + "nss_1200", > + "ppe_353", > + "gpll0_out", > + "uniphy0_rx", > + "uniphy0_tx", > + "uniphy1_rx", > + "uniphy1_tx", > + "uniphy2_rx", > + "uniphy2_tx", > + "nsscc"; I see that the input clock is named rather non-descriptively, but maybe we should call it something like "bus" so that it has more meaning to the reader Konrad