On 08/07/2025 08:20, Shawn Guo wrote:
>> +
>> +    backlight: backlight {
>> +            compatible = "pwm-backlight";
>> +            pwms = <&pwm1 0 500000 0>;
>> +            power-supply = <&reg_5v>;
>> +            brightness-levels = <0 100>;
>> +            num-interpolated-steps = <100>;
>> +            default-brightness-level = <100>;
>> +            status = "okay";
> 
> We usually use "okay" status to enable "disabled" devices.  It doesn't
> seem to be needed here.

I already asked for this in v2. Changelog claims it was done, but I
believe nothing here improved and they just sent the same patchset
wasting review effort.

Best regards,
Krzysztof

Reply via email to