Re: [PATCH 1/2] Revert "mmc: s5p_sdhci: unset the SDHCI_QUIRK_BROKEN_R1B"

Henrik Grimler
2023-02-13T07:28:32Z

Hi Jaehoon,

On Fri, 2023-02-10 at 09:00 +0900, Jaehoon Chung wrote:
> Hi,
>
> > -----Original Message-----
> > From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Henrik
> > Grimler
> > Sent: Thursday, February 9, 2023 4:04 AM
> > To: joost@diskos.nl; jh80.chung@gmail.com; andy.wu@sony.com;
> > sjg@chromium.org;
> > m.szyprowski@samsung.com; u-boot@lists.denx.de;
> > ~postmarketos/upstreaming@lists.sr.ht
> > Cc: Henrik Grimler <henrik@grimler.se>
> > Subject: [PATCH 1/2] Revert "mmc: s5p_sdhci: unset the
> > SDHCI_QUIRK_BROKEN_R1B"
> >
> > This reverts commit a034ec06ff1d558bbe11d5ee05edbb4de3ee2215.
> >
> > Commit 4a3ea75de4c5 ("Revert "mmc: sdhci: set to INT_DATA_END when
> > there are data"") reverted the alternative fix that was added for
> > Exynos 4 devices, causing an error when trying to boot from an
> > sdcard:
> >
> >     <...>
> >     Loading Environment from MMC... sdhci_send_command: Timeout for
> > status update!
> >     mmc fail to send stop cmd
> >     <...>
>
> Thanks for sharing issue.
>
> I will check this on Exynos Board. Frankly, I hope not to re-add
> QUIRK.
> Because it was verified that it was working fine without
> SDHCI_QUIKR_BROKEN_RIB.

Thanks for looking into it! It could be that issue happens only on
odroid-u2 devices, since both me and Joost have that model.

> Best Regards,
> Jaehoon Chung

Best regards,
Henrik Grimler