Re: [PATCHv1 0/6] leds: pca9653x: support inverted outputs and cleanups

[Ricardo Ribalda Delgado]

Hi again

On Tue, Apr 19, 2016 at 3:27 PM, Olliver Schinagl <oliver@xxxxxxxxxxx> wrote:
> Hey Ricardo,

> Without actually looking at the code right now, but the driver does a
> read/modify/write on the register, and a register is shared among several
> leds. So in that regard, it makes sense and I don't think it's very
> expensive to move the lock, since we have to lock for the write a few lines
> down anyway.

Actually, the code is only making sure that PCA963X_MODE2_DMBLNK is
on. It is never cleared afterwards.

It will be great if you could set that bit on probe and remove those
two lines and verify that it works on real hardware.


The move of the lock can be a bit expensive. i2c writes can take a
while to be performed, this is why only ledout was protected
initially.

Best regards