Arnd,

On Wed, May 14, 2014 at 5:18 AM, Arnd Bergmann <[email protected]> wrote:
> I just stumbled over this thread. Looking at the callers of the
> execute_tuning() function, I see that all three come from non-atomic
> context. How about an add-on patch to turn the mdelay(1) into an
> msleep(1)? That would be much friendlier to other tasks running
> concurrently.

Even better if someone could tell us why the mdelay() was there to
begin with.  As I understand it Nick was confident that it wasn't
needed for eMMC.  He didn't _think_ it was needed for SD cards either
but it was safer to leave it there for SD cards since the delay wasn't
causing any known problems on them.

-Doug
--
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

Reply via email to