Hi Jaehoon, On Thu, Feb 24 2011, Jaehoon Chung wrote: > This patch is applied the power control with regulator. > > Signed-off-by: Jaehoon Chung <jh80.chung@xxxxxxxxxxx> > Signed-off-by: kyungmin Park <kyungmin.park@xxxxxxxxxxx> > --- > drivers/mmc/host/dw_mmc.c | 25 +++++++++++++++++++++++++ > include/linux/mmc/dw_mmc.h | 2 ++ > 2 files changed, 27 insertions(+), 0 deletions(-) > > diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c > index 46e5a89..338fedc 100644 > --- a/drivers/mmc/host/dw_mmc.c > +++ b/drivers/mmc/host/dw_mmc.c > @@ -32,6 +32,7 @@ > #include <linux/mmc/mmc.h> > #include <linux/mmc/dw_mmc.h> > #include <linux/bitops.h> > +#include <linux/regulator/consumer.h> > > #include "dw_mmc.h" > > @@ -1438,6 +1439,13 @@ static int __init dw_mci_init_slot(struct dw_mci *host, unsigned int id) > } > #endif /* CONFIG_MMC_DW_IDMAC */ > > + host->vmmc = regulator_get(mmc_dev(mmc), "vmmc"); > + if (IS_ERR(host->vmmc)) { > + printk(KERN_INFO "%s: no vmmc regulator found\n", mmc_hostname(mmc)); > + host->vmmc = NULL; > + } else > + regulator_enable(host->vmmc); > + > if (dw_mci_get_cd(mmc)) > set_bit(DW_MMC_CARD_PRESENT, &slot->flags); > else I think you want an "#ifdef CONFIG_REGULATOR" around this hunk. Please try building the driver with CONFIG_REGULATOR=n. The other hunks in the patch shouldn't need it, because they test "host->vcc" before making any regulator calls. -- Chris Ball <cjb@xxxxxxxxxx> <http://printf.net/> One Laptop Per Child -- 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