diff mbox

mmc: omap_hsmmc: support more DT properties

Message ID 1392636993-15864-1-git-send-email-zonque@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Daniel Mack Feb. 17, 2014, 11:36 a.m. UTC
This should probably be done implicitly through mmc_of_parse(), but that
doesn't play well along with the multi-slot model the hsmmc driver
features. Hence, for now, do it manually. The properties are already
documented in Documentation/devicetree/bindings/mmc/mmc.txt.

Signed-off-by: Daniel Mack <zonque@gmail.com>
---
 drivers/mmc/host/omap_hsmmc.c | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Balaji T K Feb. 18, 2014, 2:36 p.m. UTC | #1
On Monday 17 February 2014 05:06 PM, Daniel Mack wrote:
> This should probably be done implicitly through mmc_of_parse(), but that
> doesn't play well along with the multi-slot model the hsmmc driver
> features. Hence, for now, do it manually. The properties are already
> documented in Documentation/devicetree/bindings/mmc/mmc.txt.
>
> Signed-off-by: Daniel Mack <zonque@gmail.com>

looks good to me
Acked-by: Balaji T K <balajitk@ti.com>

> ---
>   drivers/mmc/host/omap_hsmmc.c | 6 ++++++
>   1 file changed, 6 insertions(+)
>
> diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c
> index 2815de6..a5a38cc 100644
> --- a/drivers/mmc/host/omap_hsmmc.c
> +++ b/drivers/mmc/host/omap_hsmmc.c
> @@ -1765,6 +1765,12 @@ static struct omap_mmc_platform_data *of_get_hsmmc_pdata(struct device *dev)
>   	if (of_find_property(np, "ti,needs-special-hs-handling", NULL))
>   		pdata->slots[0].features |= HSMMC_HAS_HSPE_SUPPORT;
>
> +	if (of_find_property(np, "keep-power-in-suspend", NULL))
> +		pdata->slots[0].pm_caps |= MMC_PM_KEEP_POWER;
> +
> +	if (of_find_property(np, "enable-sdio-wakeup", NULL))
> +		pdata->slots[0].pm_caps |= MMC_PM_WAKE_SDIO_IRQ;
> +
>   	return pdata;
>   }
>   #else
>

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Daniel Mack Feb. 24, 2014, 3:31 p.m. UTC | #2
On 02/18/2014 03:36 PM, Balaji T K wrote:
> On Monday 17 February 2014 05:06 PM, Daniel Mack wrote:
>> This should probably be done implicitly through mmc_of_parse(), but that
>> doesn't play well along with the multi-slot model the hsmmc driver
>> features. Hence, for now, do it manually. The properties are already
>> documented in Documentation/devicetree/bindings/mmc/mmc.txt.
>>
>> Signed-off-by: Daniel Mack <zonque@gmail.com>
> 
> looks good to me
> Acked-by: Balaji T K <balajitk@ti.com>

Thanks for the review! Did it land in any tree yet?


Best regards,
Daniel

>> ---
>>   drivers/mmc/host/omap_hsmmc.c | 6 ++++++
>>   1 file changed, 6 insertions(+)
>>
>> diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c
>> index 2815de6..a5a38cc 100644
>> --- a/drivers/mmc/host/omap_hsmmc.c
>> +++ b/drivers/mmc/host/omap_hsmmc.c
>> @@ -1765,6 +1765,12 @@ static struct omap_mmc_platform_data *of_get_hsmmc_pdata(struct device *dev)
>>   	if (of_find_property(np, "ti,needs-special-hs-handling", NULL))
>>   		pdata->slots[0].features |= HSMMC_HAS_HSPE_SUPPORT;
>>
>> +	if (of_find_property(np, "keep-power-in-suspend", NULL))
>> +		pdata->slots[0].pm_caps |= MMC_PM_KEEP_POWER;
>> +
>> +	if (of_find_property(np, "enable-sdio-wakeup", NULL))
>> +		pdata->slots[0].pm_caps |= MMC_PM_WAKE_SDIO_IRQ;
>> +
>>   	return pdata;
>>   }
>>   #else
>>
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c
index 2815de6..a5a38cc 100644
--- a/drivers/mmc/host/omap_hsmmc.c
+++ b/drivers/mmc/host/omap_hsmmc.c
@@ -1765,6 +1765,12 @@  static struct omap_mmc_platform_data *of_get_hsmmc_pdata(struct device *dev)
 	if (of_find_property(np, "ti,needs-special-hs-handling", NULL))
 		pdata->slots[0].features |= HSMMC_HAS_HSPE_SUPPORT;
 
+	if (of_find_property(np, "keep-power-in-suspend", NULL))
+		pdata->slots[0].pm_caps |= MMC_PM_KEEP_POWER;
+
+	if (of_find_property(np, "enable-sdio-wakeup", NULL))
+		pdata->slots[0].pm_caps |= MMC_PM_WAKE_SDIO_IRQ;
+
 	return pdata;
 }
 #else