On Wed, 14 Feb 2024 at 23:24, Konrad Dybcio <konrad.dybcio@xxxxxxxxxx> wrote: > > On 13.02.2024 12:22, Dmitry Baryshkov wrote: > > Follow the example of other platforms and specify core_clk frequencies > > in the frequency table in addition to the core_clk_src frequencies. The > > driver should be setting the leaf frequency instead of some interim > > clock freq. > > > > Suggested-by: Nitin Rawat <quic_nitirawa@xxxxxxxxxxx> > > Fixes: 57fc67ef0d35 ("arm64: dts: qcom: msm8996: Add ufs related nodes") > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > > --- > > arch/arm64/boot/dts/qcom/msm8996.dtsi | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi > > index 80d83e01bb4d..401c6cce9fec 100644 > > --- a/arch/arm64/boot/dts/qcom/msm8996.dtsi > > +++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi > > @@ -2072,7 +2072,7 @@ ufshc: ufshc@624000 { > > <&gcc GCC_UFS_RX_SYMBOL_0_CLK>; > > freq-table-hz = > > <100000000 200000000>, > > - <0 0>, > > + <100000000 200000000>, > > That's bus_clk, no? No, it's a core_clk. The "core_clk_src" is removed in one of the next patches. > > Konrad > -- With best wishes Dmitry