On 28 January 2014 11:30, Jaehoon Chung <jh80.chung@xxxxxxxxxxx> wrote: > 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. :) Hehehe. My bad, sorry for spamming. You have my ack, then! Kind regards Uffe > > 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 -- 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