[PATCH] Revert "mmc: disable UHS modes if Vcc cannot be switched on and off"

Faiz Abbas faiz_abbas at ti.com
Tue Aug 11 18:03:19 CEST 2020


Pali,

On 11/08/20 1:19 pm, Pali Rohár wrote:
> On Tuesday 11 August 2020 08:39:24 Faiz Abbas wrote:
>> Pali, Peng,
>>
>> On 10/08/20 6:25 am, Peng Fan wrote:
>>> Faiz, Jean
>>>
>>>> Subject: [PATCH] Revert "mmc: disable UHS modes if Vcc cannot be switched
>>>> on and off"
>>>
>>> Got time to take a look?
>>>
>>
>> When this issue was reported in the last thread, Pali said that he was unable to get
>> prints because of the constant reboot loops? Shouldn't it be easy to put while(1)
>> at different points in this execution and see which step causes the reboot loop?
> 
> In May in that last thread I wrote details which I was able to gather:
> 
>     Month ago I was able to detect that problem is somewhere in function
>     mmc_set_ios() from mmc.c file. I put long debug line prior and also
>     after mmc_set_ios() call. And it was printed only prior. Not after.
>     So I think NULL pointer dereference is in mmc_set_ios() function.
> 
> I could try with that while(1) loop to print detailed log message and
> read/capture it. But what information for debugging you need? Or what do
> you want to print which could help you?
> 

You can continue to bisect into omap_hsmmc_set_ios() to see at what point the NULL pointer
gets triggered and what the NULL pointer is. Also can you point to your config file?

Thanks,
Faiz


More information about the U-Boot mailing list