| Message ID | 20240209-msm8996-fix-ufs-v1-3-107b52e57420@linaro.org (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | scsi: ufs: qcom: fix UFSDHCD support on MSM8996 platform | expand |
On 9.02.2024 22:50, Dmitry Baryshkov wrote: > Since the commit b4e13e1ae95e ("scsi: ufs: qcom: Add multiple frequency > support for MAX_CORE_CLK_1US_CYCLES") the Qualcomm UFS driver uses > core_clk_unipro values from frequency table to calculate cycles_in_1us. > The DT file for MSM8996 passed 0 HZ frequencies there, resulting in > broken UFS support on that platform. Fix the corresponding clock values > in the frequency table. > > Suggested-by: Nitin Rawat <quic_nitirawa@quicinc.com> > Fixes: 57fc67ef0d35 ("arm64: dts: qcom: msm8996: Add ufs related nodes") > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > --- I'd reword the commit title but okay Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org> Konrad
diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi index f6b6fdc12f44..80d83e01bb4d 100644 --- a/arch/arm64/boot/dts/qcom/msm8996.dtsi +++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi @@ -2077,7 +2077,7 @@ ufshc: ufshc@624000 { <0 0>, <0 0>, <150000000 300000000>, - <0 0>, + <75000000 150000000>, <0 0>, <0 0>, <0 0>,
Since the commit b4e13e1ae95e ("scsi: ufs: qcom: Add multiple frequency support for MAX_CORE_CLK_1US_CYCLES") the Qualcomm UFS driver uses core_clk_unipro values from frequency table to calculate cycles_in_1us. The DT file for MSM8996 passed 0 HZ frequencies there, resulting in broken UFS support on that platform. Fix the corresponding clock values in the frequency table. Suggested-by: Nitin Rawat <quic_nitirawa@quicinc.com> Fixes: 57fc67ef0d35 ("arm64: dts: qcom: msm8996: Add ufs related nodes") Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> --- arch/arm64/boot/dts/qcom/msm8996.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)