On 9.12.2023 22:55, Dmitry Baryshkov wrote: > The stats ram on sm8150 platform contains invalid data at the > DDR_DYNAMIC_OFFSET. Most likely this is because the platform didn't > support DDR sleep stats. Interesting. Can you read back DDR_DYNAMIC_OFFSET on 8350/8280 and see if 8150 has correct data in there? > However this platform uses generic > "qcom,rpmh-stats" compatible, which implies presense of the DDR data. > Add safety net to prevent old DTB files from crashing the > qcom,rpmh-stats driver. Yeah I'dve never thought there would be garbage in there.. I'd advocate for simply not doing anything wrt sleep stats if DDR stats are unavailable though. The QMP handle can stay, as there may (I don't know) be more data available that we want to export through this driver. Konrad
