On 2026-05-07 10:15, Ahmad Fatoum wrote: > Hi, > > On 5/7/26 9:02 AM, Sascha Hauer wrote: > > "officially" is meant in the way that even with the > > SDHCI_QUIRK2_BROKEN_HS200 > > flag set the driver already does HS200 when the device tree mandates it. > > Shouldn't we change mci-core.c to evaluate MMC_CAP2_HS200 then?
It already does. The problem is that when the device tree has the "mmc-hs200-1_8v" property than it's authorative. The SDHCI_QUIRK2_BROKEN_HS200 only prevents the MMC_CAP2_HS200 from being set based on other caps, but it doesn't clear MMC_CAP2_HS200 when SDHCI_QUIRK2_BROKEN_HS200 is set. The behaviour is the same as in Linux, so I hesitated from changing this. Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
