Re: [PATCH] arm64: dts: ti: k3-am62a7-sk: Disable mmc Schmitt Trigger

[Judith Mendez <jm@ti.com>]

On 2/19/26 6:10 PM, Judith Mendez wrote:
> Hi Alexander,
> 
> On 2/17/26 7:23 AM, Sverdlin, Alexander wrote:
>> On Tue, 2026-02-17 at 13:57 +0100, Alexander Sverdlin wrote:
>>>>>>>> Can you please run a quick tap sweep on MMC1 and MMC0 interfaces 
>>>>>>>> like
>>>>>>>> so? https://gist.github.com/jmenti/f4a73a8323e44bf717c6d2c528c499ca
>>>>>>>>
>>>>>>>> This will give me an idea if whether we should be talking about
>>>>>>>> revisiting characterization with ST_ENA=1.
>>>
>>> I wanted to apply your patch and test on our HW, but I have some 
>>> doubts, if
>>> the patch maybe missing something:
>>>
>>> - am654_sdhci_write_otapdly() turns out to be unused in any upstream 
>>> U-Boot version
>>> - new "omap" variable in am654_sdhci_execute_tuning() is in fact 
>>> unused as well
>>
>> and
>>
>> | /home/sverdlin/u-boot/drivers/mmc/am654_sdhci.c: In function 
>> 'j721e_4bit_sdhci_set_ios_post':
>> | /home/sverdlin/u-boot/drivers/mmc/am654_sdhci.c:598:9: error: 
>> 'itap_del_sel' is used uninitialized [-Werror=uninitialized]
>> |   598 |         printf("j721e_4bit_sdhci_set_ios_post, mode=%d, 
>> otap=%d, itap=%d\n", mode, otap_del_sel, itap_del_sel);
>> |       |         
>> ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> | /home/sverdlin/u-boot/drivers/mmc/am654_sdhci.c:594:13: note: 
>> 'itap_del_sel' was declared here
>> |   594 |         u32 itap_del_sel;
>> |       |             ^~~~~~~~~~~~
>>
>>
>> so I'm not sure regarding the previous test results at all any longer..
> 
> Sorry for the late reply, somehow I missed this email.
> 
> Oh my, I am not sure what happend but seems like I did not copy the
> whole patch correctly. Were you able to bypass or do you need me
> to rebase the patch and copy correctly?
> 

Your response actually explains why Marcus's log did not make
sense at all, thanks. Here is the updated patch in case you need
it: https://gist.github.com/f4a73a8323e44bf717c6d2c528c499ca.git

Markus, can you try with this patch instead?

~ Judith