Re: [PATCH] mmc: core: Allow sdio operations in other thread during sdio_add_func()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Feb 11, 2011 at 4:25 PM, Dmitry Shmidt <dimitrysh@xxxxxxxxxxx> wrote:
> commit 9cb71a1eb86a2acf0762d31af633984cf9e24d32
> Author: Dmitry Shmidt <dimitrysh@xxxxxxxxxx>
> Date:   Fri Feb 11 16:10:33 2011 -0800
>
>    mmc: core: Allow sdio operations in other thread during sdio_add_func()
>
>    Signed-off-by: Dmitry Shmidt <dimitrysh@xxxxxxxxxx>
>
> diff --git a/drivers/mmc/core/sdio.c b/drivers/mmc/core/sdio.c
> index dec4290..d4e6e6ef 100644
> --- a/drivers/mmc/core/sdio.c
> +++ b/drivers/mmc/core/sdio.c
> @@ -834,7 +834,6 @@ int mmc_attach_sdio(struct mmc_host *host)
>         */
>        mmc_release_host(host);
>        err = mmc_add_card(host->card);
> -       mmc_claim_host(host);
>        if (err)
>                goto remove_added;
>
> @@ -846,13 +845,12 @@ int mmc_attach_sdio(struct mmc_host *host)
>                if (err)
>                        goto remove_added;
>        }
> -
> +       mmc_claim_host(host);
>        return 0;
>
>
>  remove_added:
>        /* Remove without lock if the device has been added. */
> -       mmc_release_host(host);
>        mmc_sdio_remove(host);
>        mmc_claim_host(host);
>  remove:
>
>
> --
> 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
>

This is important when wlan driver is doing sdio operations from another
  thread during initialization.
This behavior was introduced as a side-effect in patch
  807e8e40673d9628fa7dcdd14423424b4ee5f43b
  mmc: Fix sd/sdio/mmc initialization frequency retries
--
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


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux