On Mon, Jul 6, 2020 at 2:21 PM Bartosz Golaszewski <bgolaszew...@baylibre.com> wrote:> > On Sun, Jul 5, 2020 at 3:31 PM Sungbo Eo <man...@gorani.run> wrote: > > > > NXP PCA9570 is a 4-bit I2C GPO expander without interrupt functionality. > > Its ports are controlled only by a data byte without register address. > >
> > Datasheet: https://www.nxp.com/docs/en/data-sheet/PCA9570.pdf > > No blank line here. > > Signed-off-by: Sungbo Eo <man...@gorani.run> > This driver looks nice now but why did you remove the mutex in v3? I > think when Andy commented on that, he meant not understanding why the > error check is protected, not the i2c operations. Right. > Are you sure you don't need this lock? It's a good point! -- With Best Regards, Andy Shevchenko
