Re: [PATCH V2 11/16] mmc: omap_hsmmc: ensure pbias configuration is always done

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

 



On Fri, May 6, 2011 at 2:44 PM, Adrian Hunter <adrian.hunter@xxxxxxxxx> wrote:
> Go through the driver's set_power() functions rather than
> calling regulator_enable/disable() directly because otherwise
> pbias configuration for MMC1 is not done.
Hi Chris,

Are you OK to queue this patch as bug fix. Rest of the patches of this
series is either
merged or not needed. Should I rebase and repost this alone ?

FWIW:
Acked-by: Balaji T K <balajitk@xxxxxx>

>
> Signed-off-by: Adrian Hunter <adrian.hunter@xxxxxxxxx>
> ---
>  drivers/mmc/host/omap_hsmmc.c |   17 ++++++++---------
>  1 files changed, 8 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c
> index 4f6e552..8aa9440 100644
> --- a/drivers/mmc/host/omap_hsmmc.c
> +++ b/drivers/mmc/host/omap_hsmmc.c
> @@ -445,15 +445,14 @@ static int omap_hsmmc_reg_get(struct omap_hsmmc_host *host)
>                * framework is fixed, we need a workaround like this
>                * (which is safe for MMC, but not in general).
>                */
> -               if (regulator_is_enabled(host->vcc) > 0) {
> -                       regulator_enable(host->vcc);
> -                       regulator_disable(host->vcc);
> -               }
> -               if (host->vcc_aux) {
> -                       if (regulator_is_enabled(reg) > 0) {
> -                               regulator_enable(reg);
> -                               regulator_disable(reg);
> -                       }
> +               if (regulator_is_enabled(host->vcc) > 0 ||
> +                   (host->vcc_aux && regulator_is_enabled(host->vcc_aux))) {
> +                       int vdd = ffs(mmc_slot(host).ocr_mask) - 1;
> +
> +                       mmc_slot(host).set_power(host->dev, host->slot_id,
> +                                                1, vdd);
> +                       mmc_slot(host).set_power(host->dev, host->slot_id,
> +                                                0, 0);
>                }
>        }
>
> --
> 1.7.0.4
>
> --
> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux