Hi Claudiu, On Mon, Nov 20, 2023 at 8:03 AM Claudiu <claudiu.beznea@xxxxxxxxx> wrote: > From: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx> > > Add switch OFF/OFF description to values of SW_SD0_DEV_SEL for > better understanding. > > Signed-off-by: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx> Thanks for your patch! > --- a/arch/arm64/boot/dts/renesas/rzg3s-smarc-som.dtsi > +++ b/arch/arm64/boot/dts/renesas/rzg3s-smarc-som.dtsi > @@ -11,8 +11,8 @@ > /* > * Signals of SW_CONFIG switches: > * @SW_SD0_DEV_SEL: > - * 0 - SD0 is connected to eMMC > - * 1 - SD0 is connected to uSD0 card > + * 0 - (switch OFF) SD0 is connected to eMMC > + * 1 - (switch ON) SD0 is connected to uSD0 card > * @SW_SD2_EN: > * 0 - (switch OFF) SD2 is connected to SoC > * 1 - (switch ON) SCIF1, SSI0, IRQ0, IRQ1 connected to SoC I guess this makes sense 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