| Message ID | 20220124121853.23600-5-michael.srba@seznam.cz |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [v3,1/5] dt-bindings: clock: gcc-msm8998: Add definitions of SSC-related clocks | expand |
On Mon 24 Jan 06:18 CST 2022, michael.srba@seznam.cz wrote: > From: Michael Srba <michael.srba@seznam.cz> > Something is off with your $subject, perhaps the entire commit message was treated as the subject? > Signed-off-by: Michael Srba <Michael.Srba@seznam.cz> > --- > CHANGES: > - v2: add this patch > - v3: fix missing Signed-off-by > --- > arch/arm64/boot/dts/qcom/msm8998.dtsi | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/msm8998.dtsi b/arch/arm64/boot/dts/qcom/msm8998.dtsi > index f273bc1ff629..cff83af8c12e 100644 > --- a/arch/arm64/boot/dts/qcom/msm8998.dtsi > +++ b/arch/arm64/boot/dts/qcom/msm8998.dtsi > @@ -863,6 +863,12 @@ gcc: clock-controller@100000 { > > clock-names = "xo", "sleep_clk"; > clocks = <&xo>, <&sleep_clk>; > + > + // be conservative by default, the board dts > + // can overwrite this list By next week we've forgotten why these clocks are listed here and then it's not really going to help to know that it's a conservative list. Please spell out why these clocks are listed here, and please use /* */ Thanks, Bjorn > + protected-clocks = <AGGRE2_SNOC_NORTH_AXI>, > + <SSC_XO>, > + <SSC_CNOC_AHBS_CLK>; > }; > > rpm_msg_ram: sram@778000 { > -- > 2.34.1 >
diff --git a/arch/arm64/boot/dts/qcom/msm8998.dtsi b/arch/arm64/boot/dts/qcom/msm8998.dtsi index f273bc1ff629..cff83af8c12e 100644 --- a/arch/arm64/boot/dts/qcom/msm8998.dtsi +++ b/arch/arm64/boot/dts/qcom/msm8998.dtsi @@ -863,6 +863,12 @@ gcc: clock-controller@100000 { clock-names = "xo", "sleep_clk"; clocks = <&xo>, <&sleep_clk>; + + // be conservative by default, the board dts + // can overwrite this list + protected-clocks = <AGGRE2_SNOC_NORTH_AXI>, + <SSC_XO>, + <SSC_CNOC_AHBS_CLK>; }; rpm_msg_ram: sram@778000 {