devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Kumar, Udit" <u-kumar1@ti.com>
To: Judith Mendez <jm@ti.com>, Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: Nishanth Menon <nm@ti.com>, Vignesh Raghavendra <vigneshr@ti.com>,
	Tero Kristo <kristo@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	Bhavya Kapoor <b-kapoor@ti.com>,
	Dasnavis Sabiya <sabiya.d@ti.com>
Subject: Re: [PATCH 7/7] arm64: dts: ti: k3-j784s4-evm: Remove HS400 mode support for eMMC
Date: Fri, 12 Apr 2024 10:22:46 +0530	[thread overview]
Message-ID: <f59e7f17-b978-4ff3-86d4-a77191ca66ac@ti.com> (raw)
In-Reply-To: <20240411225257.383889-8-jm@ti.com>

Hi Judith

On 4/12/2024 4:22 AM, Judith Mendez wrote:
> From: Udit Kumar <u-kumar1@ti.com>
>
> The eMMC fails to enumerate intermittently on HS400 mode. Also
> observing multiple CQE recovery warnings.
>
> Update the sdhci0 node to disable HS400.
>
> Cc: Vignesh Raghavendra <vigneshr@ti.com>
> Cc: Bhavya Kapoor <b-kapoor@ti.com>
> Cc: Siddharth Vadapalli <s-vadapalli@ti.com>
> Signed-off-by: Udit Kumar <u-kumar1@ti.com>
> Signed-off-by: Vignesh Raghavendra <vigneshr@ti.com>
> [Judith: Reword commit]
> Signed-off-by: Judith Mendez <jm@ti.com>
> ---
>   arch/arm64/boot/dts/ti/k3-j784s4-evm.dts | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-evm.dts b/arch/arm64/boot/dts/ti/k3-j784s4-evm.dts
> index 81fd7afac8c57..04a241a91e6b8 100644
> --- a/arch/arm64/boot/dts/ti/k3-j784s4-evm.dts
> +++ b/arch/arm64/boot/dts/ti/k3-j784s4-evm.dts
> @@ -791,6 +791,7 @@ &main_sdhci0 {
>   	non-removable;
>   	ti,driver-strength-ohm = <50>;
>   	disable-wp;
> +	no-mmc-hs400;

This is just WA to get rid of CQE errors. I prefer to hold this patch 
under debug from hardware side is not complete.

Moreover, CQE errors are seen with 32GB variant not with 16GB, So we can 
not blame controller for this


>   };
>   
>   &main_sdhci1 {

  reply	other threads:[~2024-04-12  4:52 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-11 22:52 [PATCH 0/7] MMC updates for TI K3 platforms Judith Mendez
2024-04-11 22:52 ` [PATCH 1/7] arm64: dts: ti: k3-am65-main: Update sdhci properties Judith Mendez
2024-04-11 22:52 ` [PATCH 2/7] arm64: dts: ti: k3-am65-main: Remove unused properties in sdhci nodes Judith Mendez
2024-04-11 22:52 ` [PATCH 3/7] arm64: dts: ti: k3-am62a: Enable UHS mode support for SD cards Judith Mendez
2024-04-11 22:52 ` [PATCH 4/7] arm64: dts: ti: k3-j721s2-main: Enable support for SDR104 speed mode Judith Mendez
2024-04-11 22:52 ` [PATCH 5/7] arm64: dts: ti: k3-j784s4-main: Enable support for UHS mode Judith Mendez
2024-04-11 22:52 ` [PATCH 6/7] arm64: dts: ti: k3-am69-sk: Remove HS400 mode support for eMMC Judith Mendez
2024-04-12  4:53   ` Kumar, Udit
2024-04-12 22:14     ` Judith Mendez
2024-04-11 22:52 ` [PATCH 7/7] arm64: dts: ti: k3-j784s4-evm: " Judith Mendez
2024-04-12  4:52   ` Kumar, Udit [this message]
2024-04-12 22:14     ` Judith Mendez
2024-04-12 13:46 ` [PATCH 0/7] MMC updates for TI K3 platforms Rob Herring

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=f59e7f17-b978-4ff3-86d4-a77191ca66ac@ti.com \
    --to=u-kumar1@ti.com \
    --cc=b-kapoor@ti.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=jm@ti.com \
    --cc=kristo@kernel.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nm@ti.com \
    --cc=robh@kernel.org \
    --cc=sabiya.d@ti.com \
    --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;
as well as URLs for NNTP newsgroup(s).