| Message ID | 20240926102005.15475-6-wsa+renesas@sang-engineering.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
| Series | ARM: dts: renesas: genmai: enable SDHI and MMCIF | expand |
Hi Wolfram, On Thu, Sep 26, 2024 at 12:20 PM Wolfram Sang <wsa+renesas@sang-engineering.com> wrote: > For this to work, User LEDs must be disabled because they share their > pins with SD data lines. > > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> Thanks for your patch! > --- a/arch/arm/boot/dts/renesas/r7s72100-genmai.dts > +++ b/arch/arm/boot/dts/renesas/r7s72100-genmai.dts > @@ -76,7 +76,8 @@ key-1 { > }; > > leds { > - status = "okay"; > + /* Needs SDHI0 to be disabled */ > + status = "disabled"; > compatible = "gpio-leds"; Pity... > > led1 { Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> i.e. will queue in renesas-devel for v6.13. Gr{oetje,eeting}s, Geert
On Fri, Oct 4, 2024 at 3:23 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote: > On Thu, Sep 26, 2024 at 12:20 PM Wolfram Sang > <wsa+renesas@sang-engineering.com> wrote: > > For this to work, User LEDs must be disabled because they share their > > pins with SD data lines. > > > > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > i.e. will queue in renesas-devel for v6.13. Oops, obsolete version. Please ignore. Gr{oetje,eeting}s, Geert
diff --git a/arch/arm/boot/dts/renesas/r7s72100-genmai.dts b/arch/arm/boot/dts/renesas/r7s72100-genmai.dts index 2d72daa4fac2..e93f444b2442 100644 --- a/arch/arm/boot/dts/renesas/r7s72100-genmai.dts +++ b/arch/arm/boot/dts/renesas/r7s72100-genmai.dts @@ -76,7 +76,8 @@ key-1 { }; leds { - status = "okay"; + /* Needs SDHI0 to be disabled */ + status = "disabled"; compatible = "gpio-leds"; led1 { @@ -227,6 +228,18 @@ scif2_pins: serial2 { /* P3_0 as TxD2; P3_2 as RxD2 */ pinmux = <RZA1_PINMUX(3, 0, 6)>, <RZA1_PINMUX(3, 2, 4)>; }; + + sdhi0_pins: sdhi0 { + /* SDHI0: P4_8 up to P4_15 */ + pinmux = <RZA1_PINMUX(4, 8, 3)>, /* SD_CD_0 */ + <RZA1_PINMUX(4, 9, 3)>, /* SD_WP_0 */ + <RZA1_PINMUX(4, 10, 3)>, /* SD_D1_0 */ + <RZA1_PINMUX(4, 11, 3)>, /* SD_D0_0 */ + <RZA1_PINMUX(4, 12, 3)>, /* SD_CLK_0 */ + <RZA1_PINMUX(4, 13, 3)>, /* SD_CMD_0 */ + <RZA1_PINMUX(4, 14, 3)>, /* SD_D3_0 */ + <RZA1_PINMUX(4, 15, 3)>; /* SD_D2_0 */ + }; }; &rtc_x1_clk { @@ -244,6 +257,14 @@ &scif2 { status = "okay"; }; +&sdhi0 { + pinctrl-names = "default"; + pinctrl-0 = <&sdhi0_pins>; + + bus-width = <4>; + status = "okay"; +}; + &spi4 { status = "okay";
For this to work, User LEDs must be disabled because they share their pins with SD data lines. Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> --- arch/arm/boot/dts/renesas/r7s72100-genmai.dts | 23 ++++++++++++++++++- 1 file changed, 22 insertions(+), 1 deletion(-)