Re: [PATCH 6/7] arm64: dts: qcom: agatti: Do not mark MPM as power domain
Sneh Mankad <[email protected]> Mon, 3 Aug 2026 16:07:51 +0530
| Newsgroups | org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
On 22-Jul-26 6:10 PM, Dmitry Baryshkov wrote: > On Wed, Jul 22, 2026 at 10:28:44AM +0200, Konrad Dybcio wrote: >> On 7/22/26 12:22 AM, Dmitry Baryshkov wrote: >>> On Wed, Jul 15, 2026 at 12:01:42PM +0200, Konrad Dybcio wrote: >>>> On 7/13/26 12:25 PM, Sneh Mankad wrote: >>>>> Do not mark MPM device as power domain since it leads to idle-states init >>>>> failure because of probe dependencies. >>>>> >>>>> CPU cluster power domain node is kept disabled and hence CPU cluster will >>>>> never power collapse. Do not register MPM under it in this case. >>>> >>>> This is only the case on the RB1. How about we fix the issue instead? >>> >>> I guess it will be fixed by the MPM timer being programmed correctly. >> >> That's patch 4. Can we test it out? > > Yes, with the patches applied, cluster_pd reinstantiated and set as a > parent domain of the MPM device, the board seems to work correctly. I > was even able to revert the I2C-gpio workaround, the device works > correctly and can reprogram LT9611UXC firmware. > > @Sneh, please post the series, including the fixed Agatti DT (and > deletion of cluster_pd + related domains from RB1), I'll respond with > the Tested-by tag. That's great to hear Dmitry! Thanks a lot for testing this. I will re-enable the cluster states on RB1 and post in v2. Thanks, Sneh