Hi Delphine, On Fri, Sep 6, 2024 at 8:28 AM Delphine CC Chiu <Delphine_CC_Chiu@xxxxxxxxxx> wrote: > Set fan led config in yosemite4 DTS. > > Signed-off-by: Delphine CC Chiu <Delphine_CC_Chiu@xxxxxxxxxx> Thanks for your patch! > @@ -75,6 +75,154 @@ tpmdev@0 { > reg = <0>; > }; > }; > + > + leds { > + compatible = "gpio-leds"; > + > + fan0_blue { Please no underscores in node names. LED node names should follow the pattern in Documentation/devicetree/bindings/leds/leds-gpio.yaml. LED color should be described using the "color" property, cfr. Documentation/devicetree/bindings/leds/common.yaml. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds