[PATCH v7 0/4] leds: aw2013: Add AW2027 support
From: Paul Sajna
Date: Sat Nov 14 2026 - 17:55:53 EST
Refer to PATCH 4 for a full description of the similarities and
differences between AW2013 and AW2027.
I'm hoping the sashiko reviews about unbalanced regulators that I was
struggling to fix are covered by
https://lore.kernel.org/linux-leds/20261021123406.3163547-1-lee@xxxxxxxxxx/
and I can drop my failed attempts to fix them.
Let me know if I misunderstood.
I'll do my best to address all of Sashiko's comments, but it might take
me some time to circle back, don't bother reviewing until I've done
this.
This PATCH resulted out of discussions pertaining to
sdm660-mainline/linux, a close-to-mainline fork for Qualcomm Snapdragon 660
devices, specifically for the BlackBerry KEY2 Keyboard Backlight,
in the following GitHub PR: https://github.com/sdm660-mainline/linux/pull/218
Signed-off-by: Paul Sajna <sajattack@xxxxxxxxxxxxxxxx>
---
Changes in v7:
- Fix multiple imax setting/regulators per Sashiko
- Small description updates to AW20XX per Heidelberg
- Reorder dev_err_probe commit so that introducing new code using
dev_err_probe is cleaner
- Simplify dev_err_probe, prevent unneccessary assignment
- Revert early return / else suggested by Heidelberg, Travkin and I both
prefer else
- Lower Hex
- split c99 for-loops to separate commit
- simplify aw2027_set_local_imax and aw2027_set_global_imax
- Link to v6: https://patch.msgid.link/20261112-aw2013-aw20xx-rename-v6-0-251eca340abc@xxxxxxxxxxxxxxxx
Changes in v6:
- Fix the compile error introduced by my unchecked hasty rebase
- Some adjustments per David Heidelberg's comments
- Link to v5: https://patch.msgid.link/20261112-aw2013-aw20xx-rename-v5-0-b2c5e323a90a@xxxxxxxxxxxxxxxx
Changes in v5:
- Drop move of chip->num_leds
- Make aw2013_microamp_to_imax more readable
- rebase to next-20261112 for Lee's changes
- Link to v4: https://lore.kernel.org/linux-leds/20261108-aw2013-aw20xx-rename-v4-0-826529beeb52@xxxxxxxxxxxxxxxx/
Changes in v4:
- Drop sashiko fixes covered by https://lore.kernel.org/linux-leds/20261021123406.3163547-1-lee@xxxxxxxxxx/
- Rename the chip id register value to reflect that it covers more chips
(https://www.awinic.com/en/driver/AW2027DNR/640)
- Address preliminary review comments from Lee Jones
- Link to v3: https://lore.kernel.org/r/20261012-aw2013-aw20xx-rename-v3-0-cab1fdd050b6@xxxxxxxxxxxxxxxx
Changes in v3:
- Drop rename
- Capitalize commit subject
- Extra tweaks to cleanup patches (preventing more unbalanced regulator
stuff)
- Whitespace fiddling
- Link to v2: https://lore.kernel.org/r/20261005-aw2013-aw20xx-rename-v2-0-108ecbdf2775@xxxxxxxxxxxxxxxx
Changes in v2:
- Reduce renames to internal driver scope, to prevent breaking compat
- Address Sashiko Review Comments
- Split renames into a separate commit to make the actual changes easier
to identify
- Change from RFC to PATCH
- Link to v1: https://lore.kernel.org/r/20260907-aw2013-aw20xx-rename-v1-0-c9a7df12a701@xxxxxxxxxxxxxxxx
To: Lee Jones <lee@xxxxxxxxxx>
To: Pavel Machek <pavel@xxxxxxxxxx>
To: Griffin Kroah-Hartman <griffin@xxxxxxxxx>
To: Rob Herring <robh@xxxxxxxxxx>
To: Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>
To: Conor Dooley <conor+dt@xxxxxxxxxx>
To: Nikita Travkin <nikitos.tr@xxxxxxxxx>
Cc: linux-leds@xxxxxxxxxxxxxxx
Cc: devicetree@xxxxxxxxxxxxxxx
Cc: linux-kernel@xxxxxxxxxxxxxxx
---
Paul Sajna (4):
dt-bindings: leds: aw2013: Add awinic,aw2027 compatible
leds: aw2013: Use dev_err_probe
drivers: leds: aw2013: Leverage C99 for-loops
leds: aw2013: Add AW2027 support
.../devicetree/bindings/leds/leds-aw2013.yaml | 6 +-
drivers/leds/leds-aw2013.c | 240 ++++++++++++++++-----
2 files changed, 194 insertions(+), 52 deletions(-)
---
base-commit: 6d582d78fad9d828e575ca0cdcff8ea47a77220b
change-id: 20260907-aw2013-aw20xx-rename-df233c5921c6
Best regards,
--
Paul Sajna <sajattack@xxxxxxxxxxxxxxxx>