On Wed Oct 1, 2025 at 10:29 AM CEST, Konrad Dybcio wrote: > On 9/30/25 4:32 PM, Luca Weiss wrote: >> From: Luca Weiss <[email protected]> >> >> Enable the RGB LED connected to the PM7350C (PM8350C). >> >> Signed-off-by: Luca Weiss <[email protected]> >> Signed-off-by: Luca Weiss <[email protected]> >> --- >> arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts | 27 >> ++++++++++++++++++++++++ >> 1 file changed, 27 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts >> b/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts >> index >> 36cc67469615974c193ea3a9db1b2362869d56bb..b4c69dc50ed0f0b3cb45341a01c89210bd261993 >> 100644 >> --- a/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts >> +++ b/arch/arm64/boot/dts/qcom/qcm6490-shift-otter.dts >> @@ -632,6 +632,33 @@ led-0 { >> }; >> }; >> >> +&pm8350c_pwm { >> + status = "okay"; >> + >> + multi-led { >> + color = <LED_COLOR_ID_RGB>; >> + function = LED_FUNCTION_STATUS; >> + >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + led@1 { >> + reg = <1>; >> + color = <LED_COLOR_ID_RED>; >> + }; >> + >> + led@2 { >> + reg = <2>; >> + color = <LED_COLOR_ID_GREEN>; >> + }; >> + >> + led@3 { >> + reg = <3>; >> + color = <LED_COLOR_ID_BLUE>; >> + }; > > I assume you managed to confirm these mappings are correct.. sometimes > they are not what you'd expect
If you mean, that red == red, green == green and blue == blue, yes. The RGB configured from user space matches the color on the LED. Regards Luca > > Reviewed-by: Konrad Dybcio <[email protected]> > > Konrad
