Re: [PATCH 1/2] i2c: qcom-cci: Reset the controller when it resumes

From: Birk Skyum

Date: Wed Oct 21 2026 - 15:38:24 EST


On Wed, Oct 21, 2026 at 9:20 AM Loic Poulain <loic.poulain@xxxxxxxxxxxxxxxx> wrote:
> > + ret = cci_init(cci);
> > + if (!ret)
> > + ret = cci_reset(cci);
>
> If we don't know the state, it's probably better to just
> unconditionally cci_reset().

Right. I kept cci_init() first so the reset-done interrupt is unmasked
after a power collapse, but cci_reset() alone works there too. On 7.3
with only cci_reset() on resume, all 30 camera restarts after 2 s
pauses started, with the controller suspended in between and no reset
timeout. v2 calls just cci_reset() on resume.