Dear, Ulf. On 01/28/2014 07:17 PM, Ulf Hansson wrote: > On 28 January 2014 09:24, Jaehoon Chung <jh80.chung@xxxxxxxxxxx> wrote: >> mmc-hs200-1.8v and mmc-hs200-1.2v can be used to the general capability. >> >> Signed-off-by: Jaehoon Chung <jh80.chung@xxxxxxxxxxx> >> --- >> Changelog V2: >> - Rename the mmc-hs200-1_8v/1_2v instead of "_caps2_" >> - Add the properties into Documentation/devicetree/bindings/mmc/mmc.txt >> >> Documentation/devicetree/bindings/mmc/mmc.txt | 2 ++ >> drivers/mmc/core/host.c | 4 ++++ >> 2 files changed, 6 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/mmc/mmc.txt b/Documentation/devicetree/bindings/mmc/mmc.txt >> index 458b57f..4bb3894 100644 >> --- a/Documentation/devicetree/bindings/mmc/mmc.txt >> +++ b/Documentation/devicetree/bindings/mmc/mmc.txt >> @@ -29,6 +29,8 @@ Optional properties: >> - cap-power-off-card: powering off the card is safe >> - cap-sdio-irq: enable SDIO IRQ signalling on this interface >> - full-pwr-cycle: full power cycle of the card is supported > > Why remove above? I guess a typo? :-) It's not removed. Just symbol in front of sentence. :) Best Regards, Jaehoon Chung > >> +- mmc-hs200-1_8v: HS200 mode(1.8V I/O) is supported >> +- mmc-hs200-1_2v: HS200 mode(1.2V I/O) is supported >> >> *NOTE* on CD and WP polarity. To use common for all SD/MMC host controllers line >> polarity properties, we have to fix the meaning of the "normal" and "inverted" >> diff --git a/drivers/mmc/core/host.c b/drivers/mmc/core/host.c >> index 49bc403..5a82e81 100644 >> --- a/drivers/mmc/core/host.c >> +++ b/drivers/mmc/core/host.c >> @@ -429,6 +429,10 @@ int mmc_of_parse(struct mmc_host *host) >> host->pm_caps |= MMC_PM_KEEP_POWER; >> if (of_find_property(np, "enable-sdio-wakeup", &len)) >> host->pm_caps |= MMC_PM_WAKE_SDIO_IRQ; >> + if (of_find_property(np, "mmc-hs200-1_8v", NULL)) >> + host->caps2 |= MMC_CAP2_HS200_1_8V_SDR; >> + if (of_find_property(np, "mmc-hs200-1_2v", NULL)) >> + host->caps2 |= MMC_CAP2_HS200_1_2V_SDR; >> >> return 0; >> >> -- >> 1.7.9.5 > > Once you fixed the above; you have my ack. > > Kind regards > Uffe > -- > 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-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
