On Wed, Sep 09, 2020 at 08:39:50PM +0000, Chris Packham wrote: > > On 9/09/20 8:23 pm, Wolfram Sang wrote: > > On Wed, Sep 09, 2020 at 08:32:47AM +1200, Evan Nimmo wrote: > >> If something goes wrong (such as the SCL being stuck low) then we need > >> to reset the PCA chip. The issue with this is that on reset we lose all > >> config settings and the chip ends up in a disabled state which results > >> in a lock up/high CPU usage. We need to re-apply any configuration that > >> had previously been set and re-enable the chip. > >> > >> Signed-off-by: Evan Nimmo <evan.nimmo@xxxxxxxxxxxxxxxxxxx> > >> Reviewed-by: Chris Packham <chris.packham@xxxxxxxxxxxxxxxxxxx> > >> Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > >> Reviewed-by: Wolfram Sang <wsa@xxxxxxxxxx> > > Applied to for-current, thanks! > > > > For the record, were you able to test both, PCA9564 and PCA9665? > > > Our hardware platforms only have PCA9665 so that's all we can test. Okay, good to know. BTW, just after I sent out my pull request containing this patch, I noticed there is no Fixes: tag. So, if you want this patch to be backported, please send it to stable@ once my pull-request is in Linus' tree.
Attachment:
signature.asc
Description: PGP signature