Re: [PATCH v2 2/2] i2c: eg20t: use generic power management

From: Bjorn Helgaas
Date: Tue Aug 25 2020 - 11:17:43 EST


On Tue, Aug 25, 2020 at 11:53:42AM +0200, Jean Delvare wrote:
> Hi Bjorn, Vaibhav,
>
> On Fri, 07 Aug 2020 15:23:21 -0500, Bjorn Helgaas wrote:
> > Also, i801_suspend() looks suspicious because it writes SMBHSTCFG, but
> > I don't see anything corresponding in i801_resume().
>
> You're right, it's buggy. Volker Rümelin's patch at:
>
> https://patchwork.ozlabs.org/project/linux-i2c/patch/a2fc5a6d-a3bf-eaf0-bb75-1521be346333@xxxxxxxxxxxxxx/
>
> should fix it. I was supposed to review it but did not, shame on me.
> I'll do it today.

Always nice when the fix is already there :) Thanks for following up
on this!

Bjorn