[U-Boot] [BISECTED] [BUG]: MMC initialization hang at Zynq Z-turn board

[Matwey V. Kornilov]

чт, 1 авг. 2019 г. в 10:40, Faiz Abbas <faiz_abbas at ti.com>:
>
> Hi Matwey,
>
> On 01/08/19 1:03 PM, Matwey V. Kornilov wrote:
> > Hi all,
> >
> > чт, 1 авг. 2019 г. в 09:48, Faiz Abbas <faiz_abbas at ti.com>:
> >>
> >> Hi,
> >>
> >> On 01/08/19 7:51 AM, Peng Fan wrote:
> >>>> Subject: [BISECTED] [BUG]: MMC initialization hang at Zynq Z-turn board
> >>>>
> >>>> Hello,
> >>>>
> >>>> I am running Zynq Z-turn board and I face the following issue with MMC
> >>>> initialization in SPL.
> >>>> With u-boot master, I see the message similar to the following:
> >>>>
> >>>> U-Boot SPL 2019.07-00352-gb5f3eb3393 (Jul 31 2019 - 20:03:42 +0300) mmc
> >>>> boot Trying to boot from MMC1
> >>>>
> >>>> Then, the u-boot waits forever. I've tried to add debug prints and found that
> >>>> execution is stalled somewhere inside mmc_init().
> >>>> Using bisect I've found that the following broken commit is the following:
> >>>
> >>> https://gitlab.denx.de/u-boot/custodians/u-boot-mmc/commit/41a9fab8dac841afb70a059668abaf14d47cdf59
> >>>
> >>> Would this patch help?
> >>>
> >>
> >> Yeah, this mostly looks like a NULL pointer access issue.
> >
> > Unfortunately, 41a9fab8 doesn't help me.... :-(
> > Behavior remains the same.
> >
>
> You need to implement the same thing with your driver (I assume you are
> using zynq_sdhci.c?). sdhci_setup_cfg() requires a valid host->mmc->dev.
> Move that assignment to above the sdhci_set_cfg() call in your driver.
>

Now it works. Thank you a lot! I'll prepare fixing patch for
zynq_sdhci.c shortly.

> Thanks,
> Faiz
>


-- 
With best regards,
Matwey V. Kornilov