2024-04-11 22:53:19

by Judith Mendez

[permalink] [raw]
Subject: [PATCH 6/7] arm64: dts: ti: k3-am69-sk: Remove HS400 mode support for eMMC

From: Dasnavis Sabiya <[email protected]>

The eMMC fails to enumerate intermittently on HS400 mode. Also
observing multiple CQE recovery warnings.
Update the sdhci0 node to disable HS400 and set HS200 mode for
eMMC node.

Signed-off-by: Dasnavis Sabiya <[email protected]>
Signed-off-by: Vignesh Raghavendra <[email protected]>
---
arch/arm64/boot/dts/ti/k3-am69-sk.dts | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/ti/k3-am69-sk.dts b/arch/arm64/boot/dts/ti/k3-am69-sk.dts
index 50de2a448a3a6..a677bb9200a35 100644
--- a/arch/arm64/boot/dts/ti/k3-am69-sk.dts
+++ b/arch/arm64/boot/dts/ti/k3-am69-sk.dts
@@ -906,6 +906,7 @@ &main_sdhci0 {
non-removable;
ti,driver-strength-ohm = <50>;
disable-wp;
+ no-mmc-hs400;
};

&main_sdhci1 {
--
2.43.2



2024-04-12 04:54:02

by Kumar, Udit

[permalink] [raw]
Subject: Re: [PATCH 6/7] arm64: dts: ti: k3-am69-sk: Remove HS400 mode support for eMMC

Hi Judith

On 4/12/2024 4:22 AM, Judith Mendez wrote:
> From: Dasnavis Sabiya <[email protected]>
>
> The eMMC fails to enumerate intermittently on HS400 mode. Also
> observing multiple CQE recovery warnings.
> Update the sdhci0 node to disable HS400 and set HS200 mode for
> eMMC node.
>
> Signed-off-by: Dasnavis Sabiya <[email protected]>
> Signed-off-by: Vignesh Raghavendra <[email protected]>
> ---
> arch/arm64/boot/dts/ti/k3-am69-sk.dts | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am69-sk.dts b/arch/arm64/boot/dts/ti/k3-am69-sk.dts
> index 50de2a448a3a6..a677bb9200a35 100644
> --- a/arch/arm64/boot/dts/ti/k3-am69-sk.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am69-sk.dts
> @@ -906,6 +906,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.

> };
>
> &main_sdhci1 {

2024-04-12 22:14:41

by Judith Mendez

[permalink] [raw]
Subject: Re: [PATCH 6/7] arm64: dts: ti: k3-am69-sk: Remove HS400 mode support for eMMC

Hi,

On 4/11/24 11:53 PM, Kumar, Udit wrote:
> Hi Judith
>
> On 4/12/2024 4:22 AM, Judith Mendez wrote:
>> From: Dasnavis Sabiya <[email protected]>
>>
>> The eMMC fails to enumerate intermittently on HS400 mode. Also
>> observing multiple CQE recovery warnings.
>> Update the sdhci0 node to disable HS400 and set HS200 mode for
>> eMMC node.
>>
>> Signed-off-by: Dasnavis Sabiya <[email protected]>
>> Signed-off-by: Vignesh Raghavendra <[email protected]>
>> ---
>>   arch/arm64/boot/dts/ti/k3-am69-sk.dts | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-am69-sk.dts
>> b/arch/arm64/boot/dts/ti/k3-am69-sk.dts
>> index 50de2a448a3a6..a677bb9200a35 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am69-sk.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-am69-sk.dts
>> @@ -906,6 +906,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.

ok, will drop this patch.

~ Judith

>
>>   };
>>   &main_sdhci1 {