Revert "interconnect: qcom: icc-rpmh: Add BCMs to commit list in pre_aggregate"
authorGeorgi Djakov <djakov@kernel.org>
Thu, 12 Aug 2021 06:24:39 +0000 (09:24 +0300)
committerGeorgi Djakov <djakov@kernel.org>
Thu, 12 Aug 2021 06:24:39 +0000 (09:24 +0300)
commitf753067494c2726d7a09ffca351d9b2599fee876
tree9a0644f25c5da5e9f15fcdcdaab05f0dd3314ed3
parent36a21d51725af2ce0700c6ebcb6b9594aac658a6
Revert "interconnect: qcom: icc-rpmh: Add BCMs to commit list in pre_aggregate"

This reverts commit f84f5b6f72e68bbaeb850b58ac167e4a3a47532a, which is
causing regressions on some platforms, preventing them to boot or do a
clean reboot. This is because the above commit is sending also all the
zero bandwidth requests to turn off any resources that might be enabled
unnecessarily, but currently this may turn off interconnects that are
enabled by default, but with no consumer to keep them on.

Let's revert this for now as some platforms are not ready for such
change yet. In the future we can introduce some _ignore_unused option
that could keep also the unused resources on platforms that have only
partial interconnect support and also add .shutdown callbacks to deal
with disabling the resources in the right order.

Reported-by: Stephen Boyd <swboyd@chromium.org>
Reported-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Link: https://lore.kernel.org/r/CAE-0n52iVgX0JjjnYi=NDg49xP961p=+W5R2bmO+2xwRceFhfA@mail.gmail.com
Signed-off-by: Georgi Djakov <djakov@kernel.org>
drivers/interconnect/qcom/icc-rpmh.c