diff mbox series

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

Message ID 20240411225257.383889-7-jm@ti.com (mailing list archive)
State New, archived
Headers show
Series MMC updates for TI K3 platforms | expand

Commit Message

Judith Mendez April 11, 2024, 10:52 p.m. UTC 
From: Dasnavis Sabiya <sabiya.d@ti.com>

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 <sabiya.d@ti.com>
Signed-off-by: Vignesh Raghavendra <vigneshr@ti.com>
---
 arch/arm64/boot/dts/ti/k3-am69-sk.dts | 1 +
 1 file changed, 1 insertion(+)

Comments

Udit Kumar April 12, 2024, 4:53 a.m. UTC | #1 
Hi Judith

On 4/12/2024 4:22 AM, Judith Mendez wrote:
> From: Dasnavis Sabiya <sabiya.d@ti.com>
>
> 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 <sabiya.d@ti.com>
> Signed-off-by: Vignesh Raghavendra <vigneshr@ti.com>
> ---
>   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 {
Judith Mendez April 12, 2024, 10:14 p.m. UTC | #2 
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 <sabiya.d@ti.com>
>>
>> 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 <sabiya.d@ti.com>
>> Signed-off-by: Vignesh Raghavendra <vigneshr@ti.com>
>> ---
>>   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 {
diff mbox series

Patch

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 {