Re: [PATCH] leds: rgb: leds-qcom-lpg: Only enable TRILED when LPG is used by LED

[Fenglin Wu <fenglin.wu@oss.qualcomm.com>]

On 11/14/2025 12:58 PM, Bjorn Andersson wrote:
> If chan->in_use, then the channel is exposed as a LED and
> lpg_pwm_request() should have returned -EBUSY, so we should never reach
> lpg_pwm_apply()?

Yes, I agree.

Change is trying to ignore enabling TRILED channel when the LPG channel 
mapping to TRILED is not used for controlling the LED (not defining the 
LED child nodes).

So the fix should be just removing this line instead of adding the if check.

I will update it in patch v2.

>
> Why do you check chan->triled_mask? I guess we will still read/write the
> triled regiter, but don't make any changes if this is 0?
>
> Or is this the actual issue that you're fixing, that we read/write the
> registers when we shouldn't? If so this should be clarified in the
> commit message.

Yes, there was a case that a LPG channel mapping to TRILED is repurposed 
to control a fan, and it was seen that the BOB1 (supplies to TRILED) 
voltage bumped to higher voltage when the PWM channel was enabled.

>
> Regards,
> Bjorn