| Message ID | 20240617143910.154546-1-bastien.curutchet@bootlin.com |
|---|---|
| Headers | show |
| Series | leds: pca9532: Use hardware for blinking LEDs | expand |
On Mon, 17 Jun 2024 16:39:06 +0200, Bastien Curutchet wrote: > This series aims to use hardware more often to blink LEDs. > > The pca9532_set_blink() rejects asymmetric delays. So the core's software > fallback is almost always used when we want to blink a LED. Removing > this restriction revealed some conflicts between setting brightness and > blinking as the same PWM (PWM0) configuration is used by all LEDs for > both brightness and blinking. > > [...] Applied, thanks! [1/4] leds: pca9532: Use defines to select PWM instance commit: 8df8c664cd6f1a58201f03849faf06115deaa928 [2/4] leds: pca9532: Use PWM1 for hardware blinking commit: 8a8f75fbfde181fbba6d67ba8606fcc4ef7fe5fa [3/4] leds: pca9532: Explicitly disable hardware blink when PWM1 is unavailable commit: 5122d8e29cb64cd8c49bdae61236f68f265b6bb4 [4/4] leds: pca9532: Change default blinking frequency to 1Hz commit: cfe05eae52d4d60a53a97ce9d5e6b29bef6bfd60 -- Lee Jones [李琼斯]
Hi all, This series aims to use hardware more often to blink LEDs. The pca9532_set_blink() rejects asymmetric delays. So the core's software fallback is almost always used when we want to blink a LED. Removing this restriction revealed some conflicts between setting brightness and blinking as the same PWM (PWM0) configuration is used by all LEDs for both brightness and blinking. Make use of the second available PWM (PWM1) to blink LEDs. This PWM1 was reserved for beepers so hardware blinking is explicitly disabled if at least one LED is used to drive a beeper to avoid conflicts. Tested with PCA9532 Changes in v2: * Add defines to get rid of magic numbers * Replace every 'led' by 'LED' * Use dev_err() when returning errors * Remove unused struct pca9532_data from patch 2 to add it on patch 3 where it's actually used [v1] : https://lore.kernel.org/all/20240527125940.155260-1-bastien.curutchet@bootlin.com/ Bastien Curutchet (4): leds: pca9532: Use defines to select PWM instance leds: pca9532: Use PWM1 for hardware blinking leds: pca9532: Explicitly disable hardware blink when PWM1 is unavailable leds: pca9532: Change default blinking frequency to 1Hz drivers/leds/leds-pca9532.c | 81 ++++++++++++++++++++++++++++--------- 1 file changed, 63 insertions(+), 18 deletions(-)