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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html