leds: pca963x: Fix open-drain initialization
authorZahari Petkov <zahari@balena.io>
Mon, 18 Nov 2019 21:02:55 +0000 (23:02 +0200)
committerPhil Elwell <phil@raspberrypi.org>
Tue, 7 Jan 2020 17:11:04 +0000 (17:11 +0000)
commitc8e1eab18bf4cb273431d6d7076355e6d251c5ca
treeb37b9467b5e19aa1fb23e2306ef9810b7ec92156
parent9ee95326a1811c97f505f00aa7328e81dfdbd01b
leds: pca963x: Fix open-drain initialization

commit 697529091ac7a0a90ca349b914bb30641c13c753 upstream.

Before commit bb29b9cccd95 ("leds: pca963x: Add bindings to invert
polarity") Mode register 2 was initialized directly with either 0x01
or 0x05 for open-drain or totem pole (push-pull) configuration.

Afterwards, MODE2 initialization started using bitwise operations on
top of the default MODE2 register value (0x05). Using bitwise OR for
setting OUTDRV with 0x01 and 0x05 does not produce correct results.
When open-drain is used, instead of setting OUTDRV to 0, the driver
keeps it as 1:

Open-drain: 0x05 | 0x01 -> 0x05 (0b101 - incorrect)
Totem pole: 0x05 | 0x05 -> 0x05 (0b101 - correct but still wrong)

Now OUTDRV setting uses correct bitwise operations for initialization:

Open-drain: 0x05 & ~0x04 -> 0x01 (0b001 - correct)
Totem pole: 0x05 | 0x04 -> 0x05 (0b101 - correct)

Additional MODE2 register definitions are introduced now as well.

Fixes: bb29b9cccd95 ("leds: pca963x: Add bindings to invert polarity")
Signed-off-by: Zahari Petkov <zahari@balena.io>
Signed-off-by: Pavel Machek <pavel@ucw.cz>
drivers/leds/leds-pca963x.c