On 29.12.2023 18:14, Manivannan Sadhasivam wrote: > On Wed, Dec 27, 2023 at 11:28:28PM +0100, Konrad Dybcio wrote: >> The CLKREQ pin should not be muxed to its active function when the RC >> is asleep. Add the missing pin sleep states to resolve that. >> > > I don't understand why it should not be muxed and wondering what is the actual > issue you are seeing that lead to this conclusion. According to my knowledge, demuxing this pin prevents the RC from getting (possibly erronous) reference clock request signals when running the RC shutdown sequence. Reading this again, the fixes tags don't really fit this commit. What's perhaps more interesting is that on msm8996, Qualcomm vendor kernels used to remove the pull-up on the CLKREQ pin in its sleep state, while on msm8998 through sm8550 they keep it there, always.. Perhaps an oversight? Konrad
