[PATCH v2 0/1] AM65x MMC power-off delay fix

[PATCH v2 0/1] AM65x MMC power-off delay fix

[Judith Mendez]

This series adds power-off-delay-us in AM65x EVM DT to ensure proper MMC
card power cycling during boot.

AM65x EVM VDD rail discharges slowly (~632.97ms) through RC circuit.
Without sufficient power-off delay, SD card boot fails in bootloader stage.

AM65x SD boot failure was initially exposed by U-Boot MMC driver patch [0],
which was correct but revealed EVM hardware limitation. There is no SD card
boot failure in Linux yet, but proper power cycling is required regardless.
Additionally, equivalent Linux kernel discussions for similar fix [0] are
ongoing. Linux MMC driver now supports delay after MMC power-off [1] and
this patch adds power-off-delay-us=650ms to AM65x EVM DT.

Links:
[0] U-Boot patch & debug discussion:
https://lore.kernel.org/u-boot/d17a810c-e2db-4a24-a37d-2d7e02466ce9@ti.com/

[1] Merged power_off_delay_us support in Linux MMC:
https://lore.kernel.org/linux-mmc/20260806162202.657512-1-jm@ti.com/

V1: https://lore.kernel.org/linux-devicetree/20260908170635.238775-1-jm@ti.com/

Changes since v1:
- Fix commit subject and description
- Add cover letter to provide full context of implementation & history

Judith Mendez (1):
  arm64: dts: ti: k3-am654-base-board: Add power-off delay for proper
    MMC power cycle

 arch/arm64/boot/dts/ti/k3-am654-base-board.dts | 1 +
 1 file changed, 1 insertion(+)

-- 
2.34.1

[PATCH v2 1/1] arm64: dts: ti: k3-am654-base-board: Add power-off delay for proper MMC power cycle

[Judith Mendez]

The AM65x EVM MMC1 VDD rail discharges slowly through RC circuit.
SD spec (section 6.4.1.3) requires VDD < 0.5V for minimum 1ms during
power cycle. Measured discharge time on this board is ~632.97ms to
reach < 0.5V threshold.

During boot, MMC power cycles between boot stages. Without power off
delay, VDD doesn't fully discharge. Card retains 1.8V signaling state
from previous boot stages. When MMC driver later initializes, card no
longer asserts S18A in ACMD41 response (already switched to 1.8V). Host
misinterprets card state and forces 1.8V anyway, falling into a bad
state and causes MMC init failure with "unable to select a mode"
signature.

Add power-off-delay-us = 650000 (650ms) to ensure proper VDD discharge
during power cycles, allowing card to fully reset state between boot
stages.

Signed-off-by: Judith Mendez <jm@ti.com>
---
Changes since v1:
- Fixup commit header and description
---
 arch/arm64/boot/dts/ti/k3-am654-base-board.dts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
index ca1300e130936..edcf039ccfbd0 100644
--- a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
@@ -460,6 +460,7 @@ &sdhci0 {
 &sdhci1 {
 	status = "okay";
 	vmmc-supply = <&vdd_mmc1_sd>;
+	power-off-delay-us = <650000>;
 	pinctrl-names = "default";
 	pinctrl-0 = <&main_mmc1_pins_default>;
 	ti,driver-strength-ohm = <50>;
-- 
2.34.1

Re: [PATCH v2 1/1] arm64: dts: ti: k3-am654-base-board: Add power-off delay for proper MMC power cycle

[sashiko-bot]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [High] The `power-off-delay-us` property is placed directly in the SDHCI host node where it is silently ignored by the kernel, rendering the intended fix ineffective.
--

commit ad27b5f7c84e8187c60f3d6ec61cfd8ac2996675
Author: Judith Mendez <jm@ti.com>

arm64: dts: ti: k3-am654-base-board: Add power-off delay for proper MMC power cycle

This commit adds a power-off-delay-us property to the SDHCI1 node in the
AM654 base board Devicetree. It intends to ensure the MMC1 VDD rail fully
discharges during power cycles between boot stages.

> diff --git a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
> index ca1300e130936..edcf039ccfbd0 100644
> --- a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
> @@ -460,6 +460,7 @@ &sdhci0 {
>  &sdhci1 {
>  	status = "okay";
>  	vmmc-supply = <&vdd_mmc1_sd>;
> +	power-off-delay-us = <650000>;

[Severity: High]
Will this property be parsed by the kernel when placed directly in the host
node?

The power-off-delay-us property is specific to the mmc-pwrseq-simple binding
and is parsed exclusively by drivers/mmc/core/pwrseq_simple.c. Neither the
core MMC code (drivers/mmc/core/host.c) nor the SDHCI host drivers parse
this property directly from the host controller node.

Because it is placed directly under &sdhci1 rather than inside an
mmc-pwrseq-simple node, the kernel will silently ignore it. Does this leave
the MMC initialization failure unresolved?

>  	pinctrl-names = "default";
>  	pinctrl-0 = <&main_mmc1_pins_default>;
>  	ti,driver-strength-ohm = <50>;

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260909173833.764103-2-jm@ti.com?part=1