Hi Luca, On Thu, Apr 21, 2022 at 08:46:42PM +0200, Luca Weiss wrote: > On Mittwoch, 6. April 2022 16:55:40 CEST Ulf Hansson wrote: > > To get this thoroughly tested, I have applied it to my next branch, for now. > > > > If it turns out that there are no regressions being reported, I think > > we should move the patch to the fixes branch (to get it included for > > v5.18) and then also tag it for stable. So, I will get back to this in > > a couple of weeks. > > Unfortunately this patch breaks internal storage on qcom-msm8974-fairphone-fp2 That is indeed unfortunate :( So we should definitely not pick it to fixes/stable, at least not yet. And if we can't come to a solution soon, maybe revert it entirely, or at least drop the HS200 portions of the change. (The systems that inspired this change are OK at HS400ES, FWIW, so the HS200 changes are just a bonus.) > With this patch (included in linux-next-20220421) it fails to initialize: > > [ 1.868608] mmc0: SDHCI controller on f9824900.sdhci [f9824900.sdhci] using > ADMA 64-bit > [ 1.925220] mmc0: mmc_select_hs200 failed, error -110 > [ 1.925285] mmc0: error -110 whilst initialising MMC card > > After reverting this patch, it works fine again. > > [ 1.908835] mmc0: SDHCI controller on f9824900.sdhci [f9824900.sdhci] using > ADMA 64-bit > [ 1.964700] mmc0: new HS200 MMC card at address 0001 > [ 1.965388] mmcblk0: mmc0:0001 BWBC3R 29.1 GiB > [ 1.975106] mmcblk0: p1 p2 p3 p4 p5 p6 p7 p8 p9 p10 p11 p12 p13 p14 p15 > p16 p17 p18 p19 p20 > [ 1.982545] mmcblk0boot0: mmc0:0001 BWBC3R 4.00 MiB > [ 1.988247] mmcblk0boot1: mmc0:0001 BWBC3R 4.00 MiB > [ 1.993287] mmcblk0rpmb: mmc0:0001 BWBC3R 4.00 MiB, chardev (242:0) As a bit of a (semi-educated) shot in the dark: can you try the appended patch? That's what my patch v1 did, but I changed it due to review comments. (Either way worked for my systems.) After re-reading the HS200-specific portions of the spec (JESD84-B51 page 45 / 6.6.2.2), it's possible setting all the way to 200 MHz this early was a bit overagressive, and we should be keeping a max of 52 MHz at this point. Thanks for testing and reporting. Brian --- a/drivers/mmc/core/mmc.c +++ b/drivers/mmc/core/mmc.c @@ -1491,7 +1491,7 @@ static int mmc_select_hs200(struct mmc_card *card) old_timing = host->ios.timing; old_clock = host->ios.clock; mmc_set_timing(host, MMC_TIMING_MMC_HS200); - mmc_set_bus_speed(card); + mmc_set_clock(card->host, card->ext_csd.hs_max_dtr); /* * For HS200, CRC errors are not a reliable way to know the
