[PATCH] arm64: dts: ti: k3-am62a7-sk: Disable mmc Schmitt Trigger
Judith Mendez
jm at ti.com
Thu Feb 19 16:42:37 PST 2026
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
More information about the linux-arm-kernel
mailing list