Hi Girish, Right, it be mentioned about the open-drain mode in synopsys spec. But if didn't work on other board, there is some problem. We can consider the location of set_busmode() . Best Regards, Jaehoon Chung 2012/7/24 Girish K S <girish.shivananjappa@xxxxxxxxxx>: > On 24 July 2012 17:06, Girish K S <girish.shivananjappa@xxxxxxxxxx> wrote: >> On 24 July 2012 16:39, Jaehoon Chung <jh80.chung@xxxxxxxxxxx> wrote: >>> Hi Girish, >>> >>> Well..just tested on my board. >>> this patch didn't work.(eMMC card didn't initialize) >>> meaning of ios->bus_mode and synosys's bus_mode is same? > Yes it is same. synopsys spec says this is specific to MMC > initialization. when the card core sets this mode during > initialization. the host has to set this bit >>> How did you test this patch? >> Tested it on exynos5 board using dt patches of thomas (with some >> modification to his patch) >>> >>> Best Regards, >>> Jaehoon Chung >>> >>> On 07/23/2012 08:16 PM, Girish K S wrote: >>>> Synopsis Designware host controller has suppport for open >>>> drain mode selection. During the mmc card initialization the >>>> host controller can select the open-drain bit to allow the device >>>> initialization in the open-drain mode. Once the device enters >>>> the standby mode this bit can be reset to enter push-pull mode. >>>> >>>> Signed-off-by: Girish K S <girish.shivananjappa@xxxxxxxxxx> >>>> --- >>>> drivers/mmc/host/dw_mmc.c | 14 ++++++++++++++ >>>> drivers/mmc/host/dw_mmc.h | 1 + >>>> 2 files changed, 15 insertions(+), 0 deletions(-) >>>> >>>> diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c >>>> index 72dc3cd..da11e5a 100644 >>>> --- a/drivers/mmc/host/dw_mmc.c >>>> +++ b/drivers/mmc/host/dw_mmc.c >>>> @@ -773,6 +773,18 @@ static void dw_mci_request(struct mmc_host *mmc, struct mmc_request *mrq) >>>> spin_unlock_bh(&host->lock); >>>> } >>>> >>>> +static void dw_mci_set_busmode(struct dw_mci_slot *slot) >>>> +{ >>>> + struct mmc_ios *ios = &slot->mmc->ios; >>>> + u32 reg; >>>> + >>>> + reg = mci_readl(slot->host, CTRL); >>>> + if (ios->bus_mode == MMC_BUSMODE_OPENDRAIN) >>>> + mci_writel(slot->host, CTRL, reg | SDMMC_CTRL_ENABLE_OD); >>>> + else >>>> + mci_writel(slot->host, CTRL, reg & ~SDMMC_CTRL_ENABLE_OD); >>>> +} >>>> + >>>> static void dw_mci_set_ios(struct mmc_host *mmc, struct mmc_ios *ios) >>>> { >>>> struct dw_mci_slot *slot = mmc_priv(mmc); >>>> @@ -818,6 +830,8 @@ static void dw_mci_set_ios(struct mmc_host *mmc, struct mmc_ios *ios) >>>> default: >>>> break; >>>> } >>>> + >>>> + dw_mci_set_busmode(slot); >>>> } >>>> >>>> static int dw_mci_get_ro(struct mmc_host *mmc) >>>> diff --git a/drivers/mmc/host/dw_mmc.h b/drivers/mmc/host/dw_mmc.h >>>> index 15c27e1..1a53fb2 100644 >>>> --- a/drivers/mmc/host/dw_mmc.h >>>> +++ b/drivers/mmc/host/dw_mmc.h >>>> @@ -67,6 +67,7 @@ >>>> >>>> /* Control register defines */ >>>> #define SDMMC_CTRL_USE_IDMAC BIT(25) >>>> +#define SDMMC_CTRL_ENABLE_OD BIT(24) >>>> #define SDMMC_CTRL_CEATA_INT_EN BIT(11) >>>> #define SDMMC_CTRL_SEND_AS_CCSD BIT(10) >>>> #define SDMMC_CTRL_SEND_CCSD BIT(9) >>>> >>> >>> > -- > 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-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html