Hi Biju, On Fri, 31 Jan 2025 at 12:25, Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > Enable eMMC on SDHI0 and SD on SDHI2 on RZ/G3E SMARC SoM. > > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > --- > v1->v2: > * Added missing header file gpio.h > * Used fixed regulator for eMMC on SD0 and dropped sd0-iovs pins for > eMMC. > * Sorted pinctrl nodes for sd2 > * Enabled internal regulator for SD2. Thanks for the update! > --- a/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts > +++ b/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts > @@ -7,6 +7,7 @@ > > /dts-v1/; > > +#include <dt-bindings/gpio/gpio.h> Not needed in this patch. > #include <dt-bindings/pinctrl/renesas,r9a09g047-pinctrl.h> > #include "r9a09g047e57.dtsi" > #include "rzg3e-smarc-som.dtsi" > --- a/arch/arm64/boot/dts/renesas/rzg3e-smarc-som.dtsi > +++ b/arch/arm64/boot/dts/renesas/rzg3e-smarc-som.dtsi > +&vqmmc_sdhi2 { > + regulator-name = "SD2_PVDD"; Why override the name? > + status = "okay"; > +}; > + > &wdt1 { > status = "okay"; > }; 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