[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20230313084953.24088-1-johan+linaro@kernel.org>
Date: Mon, 13 Mar 2023 09:49:51 +0100
From: Johan Hovold <johan+linaro@...nel.org>
To: Georgi Djakov <djakov@...nel.org>
Cc: Bjorn Andersson <andersson@...nel.org>,
Andy Gross <agross@...nel.org>,
Konrad Dybcio <konrad.dybcio@...aro.org>,
Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
Yassine Oudjana <y.oudjana@...tonmail.com>,
Christophe JAILLET <christophe.jaillet@...adoo.fr>,
linux-arm-msm@...r.kernel.org, linux-pm@...r.kernel.org,
linux-kernel@...r.kernel.org,
Johan Hovold <johan+linaro@...nel.org>
Subject: [PATCH 0/2] interconnect: qcom: rpm: fix msm8996 interconnect registration
Christophe noticed that a recent patch adding a missing clock disable to
one of the Qualcomm rpm driver error paths was broken. This would lead
to the provider not being registered on msm8996 which is the only
platform using this code path.
Turns out, however, that the power domain attach during probe is bogus
and would always succeed as any power domain would already have been
attached by the platform bus code.
Georgi, the offending commit is currently in your icc-next (and local
icc-fixes) branch. Perhaps you can fold in the fixup unless you prefer
applying it on top.
The bogus PM domain lookup is redundant and confusing but should
otherwise be benign so the removal could be applied to either branch.
Johan
Johan Hovold (2):
interconnect: qcom: rpm: fix msm8996 interconnect registration
interconnect: qcom: rpm: drop bogus pm domain attach
drivers/interconnect/qcom/icc-rpm.c | 5 -----
drivers/interconnect/qcom/icc-rpm.h | 1 -
drivers/interconnect/qcom/msm8996.c | 1 -
3 files changed, 7 deletions(-)
--
2.39.2
Powered by blists - more mailing lists