From mboxrd@z Thu Jan 1 00:00:00 1970 From: vaibhav.hiremath@linaro.org (Vaibhav Hiremath) Date: Tue, 8 Sep 2015 15:05:51 +0530 Subject: [PATCH-v2 5/7] mmc: sdhci-pxav3: Fix HS200 mode support In-Reply-To: <20150908145348.6ce640b1@xhacker> References: <1441624721-15612-1-git-send-email-vaibhav.hiremath@linaro.org> <1441624721-15612-6-git-send-email-vaibhav.hiremath@linaro.org> <20150908145348.6ce640b1@xhacker> Message-ID: <55EEABF7.5010407@linaro.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tuesday 08 September 2015 12:23 PM, Jisheng Zhang wrote: > On Mon, 7 Sep 2015 16:48:39 +0530 > Vaibhav Hiremath wrote: > >> From: Kevin Liu >> >> IN case of MMC HS200 mode, current code does not enable >> SD_CE_ATA_2.MMC_HS200 & SD_CE_ATA_2.MMC_CARD bit configurations. >> >> So this patch updates the above bit fields correctly. >> >> Signed-off-by: Tim Wang >> Signed-off-by: Kevin Liu >> Signed-off-by: Vaibhav Hiremath >> --- >> Note: Unfortunately I do not have access to any other datasheets >> which uses sdhci-pxav3 driver, so quite not sure whether this would >> break any existing platform, probably NOT, as I do not see any >> references for this change. >> If anyone can confirm that would be really great. >> >> drivers/mmc/host/sdhci-pxav3.c | 17 +++++++++++++++++ >> 1 file changed, 17 insertions(+) >> >> diff --git a/drivers/mmc/host/sdhci-pxav3.c b/drivers/mmc/host/sdhci-pxav3.c >> index d933f75..6978810 100644 >> --- a/drivers/mmc/host/sdhci-pxav3.c >> +++ b/drivers/mmc/host/sdhci-pxav3.c >> @@ -57,6 +57,8 @@ >> #define SD_CE_ATA_1 0x10C >> >> #define SD_CE_ATA_2 0x10E >> +#define SD_CE_ATA2_HS200_EN BIT(10) >> +#define SD_CE_ATA2_MMC_MODE BIT(12) >> #define SDCE_MISC_INT BIT(2) >> #define SDCE_MISC_INT_EN BIT(1) >> >> @@ -330,6 +332,17 @@ static int pxav3_select_pinstate(struct sdhci_host *host, unsigned int uhs) >> return pinctrl_select_state(pxa->pinctrl, pinctrl); >> } >> >> +static int pxav3_select_hs200(struct sdhci_host *host) > > I didn't see why we need the return value, make it void would be better? > Fair enough. Will fix it in next version. Thanks, Vaibhav