The LPG channel can be used for only outputting PWM signal without the
need to enable TRILED. Update the DT binding document to explain that
usage and remove the TRILED enabling register write in the driver for
that use case.
Signed-off-by: Fenglin Wu <fenglin.wu@oss.qualcomm.com>
---
Changes in v3:
- Add a DT binding change to clarify the standalone PWM usage according
to the suggestion from Bjorn.
- Link to v2: https://lore.kernel.org/r/20251118-lpg_triled_fix-v2-1-6d1e5831333f@oss.qualcomm.com
Changes in v2:
- Check "chan->in_use" flag in lpg_pwm_apply() is not correct, as it
indicates the channel is being used as a LED and this PWM API would
never get called. Instead, remove the code line which enables TRILED
in lpg_pwm_apply() and update the commit text to explain it clearly.
- Link to v1: https://lore.kernel.org/r/20251114-lpg_triled_fix-v1-1-9b239832c53c@oss.qualcomm.com
---
Fenglin Wu (2):
dt-bindings: leds: qcom-lpg: Explain standalone PWM usage
leds: rgb: leds-qcom-lpg: Don't enable TRILED when configuring PWM
Documentation/devicetree/bindings/leds/leds-qcom-lpg.yaml | 5 +++++
drivers/leds/rgb/leds-qcom-lpg.c | 4 +---
2 files changed, 6 insertions(+), 3 deletions(-)
---
base-commit: ea1c4c7e648d1ca91577071fc42fdc219521098c
change-id: 20251114-lpg_triled_fix-44491b49b340
Best regards,
--
Fenglin Wu <fenglin.wu@oss.qualcomm.com>