On 24 October 2016 at 09:49, Linus Walleij <linus.walleij@xxxxxxxxxx> wrote: > On Thu, Oct 20, 2016 at 10:19 AM, Ulf Hansson <ulf.hansson@xxxxxxxxxx> wrote: > >> When polling for busy after sending a MMC_SWITCH command, both the optional >> ->card_busy() callback and CMD13 are being used in conjunction. >> >> This doesn't make sense. Instead it's more reasonable to rely solely on the >> ->card_busy() callback when it exists. Let's change that and instead use >> the CMD13 as a fall-back. In this way we avoid sending CMD13, unless it's >> really needed. >> >> Within this context, let's also take the opportunity to make some >> additional clean-ups and clarifications to the related code. >> >> Signed-off-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx> > > Tried these four patches but it doesn't make my root mount before > 20 minutes on the Dragonboard, but I don't know if that was the > intention even? Just cleanup? Sorry if I got it wrong... Apologize if this was confusing, these changes was not intended to fix your reported problem. They are clean-ups and the last change improves the polling loop for switch commands a bit. I will post a patch for the reported regression asap and will keep you on cc. Appreciate if you could help in testing! Kind regards Uffe -- 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
