RE: [PATCH] arm64: dts: renesas: rzt2h-n2h-evk: Configure eMMC/SDHI pins

[Fabrizio Castro]

Hi Geert,

Are you happy with this patch?

Without this patch fast SD cards/eMMCs may fail on the RZ/T2H and RZ/N2H.

If it helps, we may also add a couple of fixes tags to the patch, to fast
track it:
Fixes: 4d7624fc85a2 ("arm64: dts: renesas: rzt2h-rzn2h-evk: Enable eMMC")
Fixes: dba8ee27c5de ("arm64: dts: renesas: rzt2h-rzn2h-evk: Enable MicroSD card slot")

I haven't added them to the patch because I wanted to avoid being selected
for stable, as it depends on the below series:
https://lore.kernel.org/linux-renesas-soc/20260319141515.2053556-1-prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx/

Cheers,
Fab

> From: Fabrizio Castro <fabrizio.castro.jz@xxxxxxxxxxx>
> Sent: 31 March 2026 15:52
> To: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>; magnus.damm <magnus.damm@xxxxxxxxx>; Rob Herring
> <robh@xxxxxxxxxx>; Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>; Conor Dooley <conor+dt@xxxxxxxxxx>
> Cc: Fabrizio Castro <fabrizio.castro.jz@xxxxxxxxxxx>; linux-renesas-soc@xxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Biju Das <biju.das.jz@xxxxxxxxxxxxxx>;
> Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx>
> Subject: [PATCH] arm64: dts: renesas: rzt2h-n2h-evk: Configure eMMC/SDHI pins
> 
> The HW user manual for the Renesas RZ/T2H and the RZ/N2H state
> that for SDR104, SDR50, and HS200 to work properly the eMMC/SDHI
> interface pins have to be configured as specified below:
> * SDn_CLK pin - drive strength: Ultra High, slew rate: fast
> * Other SDn_* pins: drive strength: High, slew rate: fast,
>   Schmitt trigger: disabled (not applicable to SDn_RST pins).
> 
> Adjust the pin definitions accordingly.
> 
> Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@xxxxxxxxxxx>
> ---
>  .../dts/renesas/rzt2h-n2h-evk-common.dtsi     | 54 ++++++++++++++++---
>  1 file changed, 46 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/renesas/rzt2h-n2h-evk-common.dtsi b/arch/arm64/boot/dts/renesas/rzt2h-
> n2h-evk-common.dtsi
> index f87c2492f414..3fae950db603 100644
> --- a/arch/arm64/boot/dts/renesas/rzt2h-n2h-evk-common.dtsi
> +++ b/arch/arm64/boot/dts/renesas/rzt2h-n2h-evk-common.dtsi
> @@ -275,12 +275,28 @@ data-pins {
>  				 <RZT2H_PORT_PINMUX(12, 7, 0x29)>, /* SD0_DATA5 */
>  				 <RZT2H_PORT_PINMUX(13, 0, 0x29)>, /* SD0_DATA6 */
>  				 <RZT2H_PORT_PINMUX(13, 1, 0x29)>; /* SD0_DATA7 */
> +			drive-strength-microamp = <9000>;
> +			slew-rate = <1>;
> +			input-schmitt-disable;
>  		};
> 
> -		ctrl-pins {
> -			pinmux = <RZT2H_PORT_PINMUX(12, 0, 0x29)>, /* SD0_CLK */
> -				 <RZT2H_PORT_PINMUX(12, 1, 0x29)>, /* SD0_CMD */
> -				 <RZT2H_PORT_PINMUX(13, 2, 0x29)>; /* SD0_RST# */
> +		clk-pins {
> +			pinmux = <RZT2H_PORT_PINMUX(12, 0, 0x29)>; /* SD0_CLK */
> +			drive-strength-microamp = <11800>;
> +			slew-rate = <1>;
> +		};
> +
> +		cmd-pins {
> +			pinmux = <RZT2H_PORT_PINMUX(12, 1, 0x29)>; /* SD0_CMD */
> +			drive-strength-microamp = <9000>;
> +			slew-rate = <1>;
> +			input-schmitt-disable;
> +		};
> +
> +		rst-pins {
> +			pinmux = <RZT2H_PORT_PINMUX(13, 2, 0x29)>; /* SD0_RST# */
> +			drive-strength-microamp = <9000>;
> +			slew-rate = <1>;
>  		};
>  	};
> 
> @@ -299,12 +315,23 @@ data-pins {
>  				 <RZT2H_PORT_PINMUX(12, 3, 0x29)>, /* SD0_DATA1 */
>  				 <RZT2H_PORT_PINMUX(12, 4, 0x29)>, /* SD0_DATA2 */
>  				 <RZT2H_PORT_PINMUX(12, 5, 0x29)>; /* SD0_DATA3 */
> +			drive-strength-microamp = <9000>;
> +			slew-rate = <1>;
> +			input-schmitt-disable;
> +		};
> +
> +		clk-pins {
> +			pinmux = <RZT2H_PORT_PINMUX(12, 0, 0x29)>; /* SD0_CLK */
> +			drive-strength-microamp = <11800>;
> +			slew-rate = <1>;
>  		};
> 
>  		ctrl-pins {
> -			pinmux = <RZT2H_PORT_PINMUX(12, 0, 0x29)>, /* SD0_CLK */
> -				 <RZT2H_PORT_PINMUX(12, 1, 0x29)>, /* SD0_CMD */
> +			pinmux = <RZT2H_PORT_PINMUX(12, 1, 0x29)>, /* SD0_CMD */
>  				 <RZT2H_PORT_PINMUX(22, 5, 0x29)>; /* SD0_CD */
> +			drive-strength-microamp = <9000>;
> +			slew-rate = <1>;
> +			input-schmitt-disable;
>  		};
>  	};
> 
> @@ -323,12 +350,23 @@ data-pins {
>  				 <RZT2H_PORT_PINMUX(17, 0, 0x29)>, /* SD1_DATA1 */
>  				 <RZT2H_PORT_PINMUX(17, 1, 0x29)>, /* SD1_DATA2 */
>  				 <RZT2H_PORT_PINMUX(17, 2, 0x29)>; /* SD1_DATA3 */
> +			drive-strength-microamp = <9000>;
> +			slew-rate = <1>;
> +			input-schmitt-disable;
> +		};
> +
> +		clk-pins {
> +			pinmux = <RZT2H_PORT_PINMUX(16, 5, 0x29)>; /* SD1_CLK */
> +			drive-strength-microamp = <11800>;
> +			slew-rate = <1>;
>  		};
> 
>  		ctrl-pins {
> -			pinmux = <RZT2H_PORT_PINMUX(16, 5, 0x29)>, /* SD1_CLK */
> -				 <RZT2H_PORT_PINMUX(16, 6, 0x29)>, /* SD1_CMD */
> +			pinmux = <RZT2H_PORT_PINMUX(16, 6, 0x29)>, /* SD1_CMD */
>  				 <RZT2H_PORT_PINMUX(17, 4, 0x29)>; /* SD1_CD */
> +			drive-strength-microamp = <9000>;
> +			slew-rate = <1>;
> +			input-schmitt-disable;
>  		};
>  	};
>  };
> --
> 2.34.1