Revert "soc: qcom: stats: Add DDR sleep stats"
authorKonrad Dybcio <konrad.dybcio@linaro.org>
Thu, 14 Dec 2023 12:25:15 +0000 (13:25 +0100)
committerBjorn Andersson <andersson@kernel.org>
Sat, 16 Dec 2023 04:22:36 +0000 (22:22 -0600)
commita7dc6343519752eb6d86bfa78378a8af5da1f475
tree6d3098f110e8d053d8e05620867a911cc89374ba
parent56fdc35ef067c8dffee22038dd3a84bb3fa6d2a4
Revert "soc: qcom: stats: Add DDR sleep stats"

After recent reports ([1], [2]) of older platforms (particularly 8150 and
7180) breaking after DDR sleep stats introduction, revert the following:

Commit 73380e2573c3 ("soc: qcom: stats: fix 64-bit division")
Commit e84e61bdb97c ("soc: qcom: stats: Add DDR sleep stats")

The feature itself is rather useful for debugging DRAM power management,
however it looks like the shared RPMh stats data structures differ on
previous SoCs.

Revert its addition for now to un-break booting on these earlier SoCs,
while I try to come up with a better way to enable it conditionally.

[1] https://lore.kernel.org/linux-arm-msm/20231209215601.3543895-2-dmitry.baryshkov@linaro.org/
[2] https://lore.kernel.org/linux-arm-msm/CAD=FV=XX4wLg1NNVL15RK4D4tLvuSzZyUv=k_tS4bSb3=7QJzQ@mail.gmail.com/

Reported-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reported-by: Doug Anderson <dianders@chromium.org>
Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Tested-by: Douglas Anderson <dianders@chromium.org>
Link: https://lore.kernel.org/r/20231214-topic-undo_ddr_stats-v1-1-1fe32c258e56@linaro.org
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
drivers/soc/qcom/qcom_stats.c