* [PATCH 1/7] mmc: core add the property for hs200
@ 2014-01-21 10:16 Jaehoon Chung
2014-01-21 10:30 ` Arnd Bergmann
0 siblings, 1 reply; 5+ messages in thread
From: Jaehoon Chung @ 2014-01-21 10:16 UTC (permalink / raw)
To: linux-mmc@vger.kernel.org
Cc: Chris Ball, Ulf Hansson, Seungwon Jeon, Guennadi Liakhovetski
caps2-mmc-hs200-1.8v and caps2-mmc-hs200-1.2v can be used to the general capability.
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
---
drivers/mmc/core/host.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/mmc/core/host.c b/drivers/mmc/core/host.c
index 49bc403..96ac4c1 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, "caps2-mmc-hs200-1_8v", NULL))
+ host->caps2 |= MMC_CAP2_HS200_1_8V_SDR;
+ if (of_find_property(np, "caps2-mmc-hs200-1_2v", NULL))
+ host->caps2 |= MMC_CAP2_HS200_1_2V_SDR;
return 0;
--
1.7.9.5
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/7] mmc: core add the property for hs200
2014-01-21 10:16 [PATCH 1/7] mmc: core add the property for hs200 Jaehoon Chung
@ 2014-01-21 10:30 ` Arnd Bergmann
2014-01-21 12:12 ` Ulf Hansson
0 siblings, 1 reply; 5+ messages in thread
From: Arnd Bergmann @ 2014-01-21 10:30 UTC (permalink / raw)
To: Jaehoon Chung
Cc: linux-mmc@vger.kernel.org, Chris Ball, Ulf Hansson, Seungwon Jeon,
Guennadi Liakhovetski
On Tuesday 21 January 2014 19:16:54 Jaehoon Chung wrote:
> caps2-mmc-hs200-1.8v and caps2-mmc-hs200-1.2v can be used to the general capability.
>
> Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
> ---
> drivers/mmc/core/host.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/mmc/core/host.c b/drivers/mmc/core/host.c
> index 49bc403..96ac4c1 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, "caps2-mmc-hs200-1_8v", NULL))
> + host->caps2 |= MMC_CAP2_HS200_1_8V_SDR;
> + if (of_find_property(np, "caps2-mmc-hs200-1_2v", NULL))
> + host->caps2 |= MMC_CAP2_HS200_1_2V_SDR;
>
> return 0;
If we do this, the strings also need to be documented in
bindings/mmc/mmc.txt.
Unfortunately, the names are not particularly intuitive (the caps2
part in particular), but it's probably better to stick with that
now.
Arnd
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/7] mmc: core add the property for hs200
2014-01-21 10:30 ` Arnd Bergmann
@ 2014-01-21 12:12 ` Ulf Hansson
2014-01-21 12:20 ` Jaehoon Chung
0 siblings, 1 reply; 5+ messages in thread
From: Ulf Hansson @ 2014-01-21 12:12 UTC (permalink / raw)
To: Arnd Bergmann, Jaehoon Chung
Cc: linux-mmc@vger.kernel.org, Chris Ball, Seungwon Jeon,
Guennadi Liakhovetski
On 21 January 2014 11:30, Arnd Bergmann <arnd@arndb.de> wrote:
> On Tuesday 21 January 2014 19:16:54 Jaehoon Chung wrote:
>> caps2-mmc-hs200-1.8v and caps2-mmc-hs200-1.2v can be used to the general capability.
>>
>> Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
>> ---
>> drivers/mmc/core/host.c | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/drivers/mmc/core/host.c b/drivers/mmc/core/host.c
>> index 49bc403..96ac4c1 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, "caps2-mmc-hs200-1_8v", NULL))
>> + host->caps2 |= MMC_CAP2_HS200_1_8V_SDR;
>> + if (of_find_property(np, "caps2-mmc-hs200-1_2v", NULL))
>> + host->caps2 |= MMC_CAP2_HS200_1_2V_SDR;
>>
>> return 0;
>
> If we do this, the strings also need to be documented in
> bindings/mmc/mmc.txt.
>
> Unfortunately, the names are not particularly intuitive (the caps2
> part in particular), but it's probably better to stick with that
> now.
I would prefer to remove "caps2" from the property name.
Going forward and cleaning up all the non needed caps flags, could
mean we may want to move caps2 flags into caps flags, I guess. This
would not be suitable if we decide to name the property as above.
Kind regards
Uffe
>
> Arnd
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/7] mmc: core add the property for hs200
2014-01-21 12:12 ` Ulf Hansson
@ 2014-01-21 12:20 ` Jaehoon Chung
2014-01-21 13:36 ` Arnd Bergmann
0 siblings, 1 reply; 5+ messages in thread
From: Jaehoon Chung @ 2014-01-21 12:20 UTC (permalink / raw)
To: Ulf Hansson, Arnd Bergmann, Jaehoon Chung
Cc: linux-mmc@vger.kernel.org, Chris Ball, Seungwon Jeon,
Guennadi Liakhovetski
On 01/21/2014 09:12 PM, Ulf Hansson wrote:
> On 21 January 2014 11:30, Arnd Bergmann <arnd@arndb.de> wrote:
>> On Tuesday 21 January 2014 19:16:54 Jaehoon Chung wrote:
>>> caps2-mmc-hs200-1.8v and caps2-mmc-hs200-1.2v can be used to the general capability.
>>>
>>> Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
>>> ---
>>> drivers/mmc/core/host.c | 4 ++++
>>> 1 file changed, 4 insertions(+)
>>>
>>> diff --git a/drivers/mmc/core/host.c b/drivers/mmc/core/host.c
>>> index 49bc403..96ac4c1 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, "caps2-mmc-hs200-1_8v", NULL))
>>> + host->caps2 |= MMC_CAP2_HS200_1_8V_SDR;
>>> + if (of_find_property(np, "caps2-mmc-hs200-1_2v", NULL))
>>> + host->caps2 |= MMC_CAP2_HS200_1_2V_SDR;
>>>
>>> return 0;
>>
>> If we do this, the strings also need to be documented in
>> bindings/mmc/mmc.txt.
>>
>> Unfortunately, the names are not particularly intuitive (the caps2
>> part in particular), but it's probably better to stick with that
>> now.
>
> I would prefer to remove "caps2" from the property name.
>
> Going forward and cleaning up all the non needed caps flags, could
> mean we may want to move caps2 flags into caps flags, I guess. This
> would not be suitable if we decide to name the property as above.
Agreed,caps2 flags can be changed in future.
I will remove the "caps2" from name.
And i will add the document in binding/mmc/mmc.txt at next patch.
Best Regards,
Jaehoon Chung
>
> Kind regards
> Uffe
>
>>
>> Arnd
> --
> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/7] mmc: core add the property for hs200
2014-01-21 12:20 ` Jaehoon Chung
@ 2014-01-21 13:36 ` Arnd Bergmann
0 siblings, 0 replies; 5+ messages in thread
From: Arnd Bergmann @ 2014-01-21 13:36 UTC (permalink / raw)
To: Jaehoon Chung
Cc: Ulf Hansson, linux-mmc@vger.kernel.org, Chris Ball, Seungwon Jeon,
Guennadi Liakhovetski
On Tuesday 21 January 2014, Jaehoon Chung wrote:
> On 01/21/2014 09:12 PM, Ulf Hansson wrote:
> > On 21 January 2014 11:30, Arnd Bergmann <arnd@arndb.de> wrote:
> >> On Tuesday 21 January 2014 19:16:54 Jaehoon Chung wrote:
> >>
> >> Unfortunately, the names are not particularly intuitive (the caps2
> >> part in particular), but it's probably better to stick with that
> >> now.
> >
> > I would prefer to remove "caps2" from the property name.
> >
> > Going forward and cleaning up all the non needed caps flags, could
> > mean we may want to move caps2 flags into caps flags, I guess. This
> > would not be suitable if we decide to name the property as above.
>
> Agreed,caps2 flags can be changed in future.
> I will remove the "caps2" from name.
> And i will add the document in binding/mmc/mmc.txt at next patch.
>
In general, we have to be very careful changing these things, so we don't
break existing dts files booting with newer kernels.
In this particular case, I think we're fine, since the hs200 properties
are fairly new and haven't made it into any dts files so far.
Arnd
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-01-21 13:36 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-21 10:16 [PATCH 1/7] mmc: core add the property for hs200 Jaehoon Chung
2014-01-21 10:30 ` Arnd Bergmann
2014-01-21 12:12 ` Ulf Hansson
2014-01-21 12:20 ` Jaehoon Chung
2014-01-21 13:36 ` Arnd Bergmann
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).