From: Judith Mendez <jm@ti.com>
To: Nishanth Menon <nm@ti.com>
Cc: Vignesh Raghavendra <vigneshr@ti.com>,
Tero Kristo <kristo@kernel.org>, Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
<linux-arm-kernel@lists.infradead.org>,
<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] mmc: dts: am654: Add 650ms power off delay for proper MMC power cycle
Date: Tue, 8 Sep 2026 12:29:26 -0500 [thread overview]
Message-ID: <e8abc328-578e-48cb-a469-1e3edec6780a@ti.com> (raw)
In-Reply-To: <20260908171613.bpd6eo53cjtlrztr@macaw>
Hi Nishanth,
On 9/8/26 12:16, Nishanth Menon wrote:
> On 12:06-20260908, Judith Mendez wrote:
>> 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 stages. When U-Boot 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 causing
>> a boot failure.
>>
>> 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>
>
> Fixes ?
Fix is really for u-boot, don't think I can point to a u-boot
commit here can I?
~ Judith
>
>> ---
>> 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
>>
>
next prev parent reply other threads:[~2026-09-08 17:30 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-08 17:06 [PATCH] mmc: dts: am654: Add 650ms power off delay for proper MMC power cycle Judith Mendez
2026-09-08 17:15 ` sashiko-bot
2026-09-08 17:16 ` Nishanth Menon
2026-09-08 17:29 ` Judith Mendez [this message]
2026-09-08 17:49 ` Nishanth Menon
2026-09-08 18:13 ` Judith Mendez
2026-09-09 10:36 ` Nishanth Menon
2026-09-09 15:36 ` Judith Mendez
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=e8abc328-578e-48cb-a469-1e3edec6780a@ti.com \
--to=jm@ti.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=kristo@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=nm@ti.com \
--cc=robh@kernel.org \
--cc=vigneshr@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox