On Wed, Sep 7, 2011 at 10:03 PM, Daniel Drake <[email protected]> wrote: >> Adding random sleeps without understanding what exactly do they solve >> generally feels wrong... > > Well, if you remove the existing sleeps from mmc_power_up do things > keep working for you?
I wouldn't call them random - they're pretty much explained. OTOH, the delay you're suggesting to add isn't: we don't know what is it solving exactly and why is it needed. IMHO mainline kernel code should be well explained, otherwise it'd become impossible to maintain. -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
