On Mon, 7 Nov 2022 at 06:55, Amit Pundir <amit.pundir@xxxxxxxxxx> wrote: > > On Mon, 7 Nov 2022 at 08:43, Bjorn Andersson <andersson@xxxxxxxxxx> wrote: > > > > On Thu, 27 Oct 2022 13:57:45 +0200, Ulf Hansson wrote: > > > To support the deeper cluster idle state for sm8250 platforms, some > > > additional synchronization is needed between the rpmh-rsc device and the > > > CPU cluster PM domain. Until that is supported, let's disable the cluster > > > idle state. > > > > > > This fixes a problem that has been reported for the Qcom RB5 platform (see > > > below), but most likely other sm8250 platforms suffers from similar issues, > > > so let's make the fix generic for sm8250. > > > > > > [...] > > > > Applied, thanks! > > > > [1/1] arm64: dts: qcom: sm8250: Disable the not yet supported cluster idle state > > commit: 5c7fa5de12a31c1425cc87ba2ef27e6ae0a1788c > > Hi Bjorn, > > There seem to be some error while applying the patch "arm64: dts: > qcom: sm8250: Disable the not yet supported cluster idle state". > This patch is already applied in your arm64-fixes-for-6.1 tree > (commit: cadaa773bcf161184fa428180516bae33a7bc667) > > The new commit: 5c7fa5de12a31c1425cc87ba2ef27e6ae0a1788c, > however, disables the Big cpu idle state instead. I'm not sure if that > is intentional though. It's a mistake. There have been a lot of various patches/discussions around this issue at LKML, not entirely easy to follow. Anyway to make it clear, we shouldn't need to disable the BIG_CPU_SLEEP_0 state, but only the CLUSTER_SLEEP_0. Bjorn, can you please have a look and drop/revert commit 5c7fa5de12a31c1425cc87ba2ef27e6ae0a1788c ? Kind regards Uffe
