Hi Prabhakar, On Tue, Aug 25, 2020 at 6:28 PM Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> wrote: > Add support for LED trigger on SD2 interface. > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > Reviewed-by: Chris Paterson <Chris.Paterson2@xxxxxxxxxxx> > --- a/arch/arm/boot/dts/r8a7742-iwg21d-q7.dts > +++ b/arch/arm/boot/dts/r8a7742-iwg21d-q7.dts > @@ -63,6 +63,16 @@ > enable-gpios = <&gpio3 11 GPIO_ACTIVE_HIGH>; > }; > > + leds { > + compatible = "gpio-leds"; > + > + sdhi2_led { > + label = "sdio-led"; > + gpios = <&gpio5 22 GPIO_ACTIVE_LOW>; GPIO_ACTIVE_HIGH? The LED is driven by an NPN transistor, with the LED between 3.3V and the transistor's collector. > + linux,default-trigger = "mmc1"; > + }; > + }; > + > lvds-receiver { > compatible = "ti,ds90cf384a", "lvds-decoder"; > vcc-supply = <&vcc_3v3_tft1>; With the above fixed: Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> 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