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
