On Mon, Jan 11, 2016 at 1:22 PM, Ulf Hansson <ulf.hansson@xxxxxxxxxx> wrote: > On 4 January 2016 at 22:15, Carlo Caione <carlo@xxxxxxxxxx> wrote: >> From: Carlo Caione <carlo@xxxxxxxxxxxx> >> >> While in sdhci_execute_tuning() the choice whether or not to enable the >> tuning is done on the actual timing, in the mmc_sdio_init_uhs_card() the >> check is done on the capability of the card. >> >> This difference is causing some issues with some SDIO cards in DDR50 >> mode where the CDM19 is wrongly issued. > > I wonder if this statement is really correct, because for SDIO cards > in DDR50 the tuning command wasn't being issued. Right!? Not quite. In my case the issue of the tuning command was the problem. This patch is basically what was suggested by Yang here http://permalink.gmane.org/gmane.linux.kernel.mmc/35055 >> With this patch we modify the check in both >> mmc_(sd|sdio)_init_uhs_card() functions to take the proper decision >> only according to the actual timing specification. > > Certainly there's a bug in the way how we initiate SDIO cards from the > mmc core perspective, but how is the SD case wrong? The SD case is not wrong. I changed it also to be more aligned with the SDIO case > Anyway, I do like the change as such as the code gets more aligned to > check the "ios.timing" parameter. Agree > Please add "stable" tag, unless we can find a reasonable commit it > fixes and add a fixes tag instead. Ok Thanks, -- Carlo Caione -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
