* Re: [PATCH] ARM: dts: Update vdd_arm regulator
[not found] <1370516572-313-1-git-send-email-tushar.behera@linaro.org>
@ 2013-06-08 11:52 ` Tomasz Figa
2013-06-10 3:44 ` Tushar Behera
0 siblings, 1 reply; 3+ messages in thread
From: Tomasz Figa @ 2013-06-08 11:52 UTC (permalink / raw)
To: Tushar Behera
Cc: linux-samsung-soc, kgene.kim, patches, broonie, swarren,
grant.likely, rob.herring, l.majewski, s.nawrocki,
devicetree-discuss
Hi Tushar,
On Thursday 06 of June 2013 16:32:52 Tushar Behera wrote:
> Cpufreq driver for EXYNOS4210 is not a platform driver, hence it is not
> possible to provide the regulator supply name through DT bindings.
> Since the cpufreq driver requires the regulator to be named as
> 'vdd_arm', the related regulator name should be kept same.
>
> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
> ---
> arch/arm/boot/dts/exynos4210-origen.dts | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/exynos4210-origen.dts
> b/arch/arm/boot/dts/exynos4210-origen.dts index bcf8079..bd5f589 100644
> --- a/arch/arm/boot/dts/exynos4210-origen.dts
> +++ b/arch/arm/boot/dts/exynos4210-origen.dts
> @@ -192,7 +192,7 @@
> };
>
> buck1_reg: BUCK1 {
> - regulator-name = "VDD_ARM_1.2V";
> + regulator-name = "vdd_arm";
Yes, this is the hack I mentioned in my review of
[PATCH 0/2] Clock update for EXYNOS4210-CPUFREQ driver
Best regards,
Tomasz
> regulator-min-microvolt =
<950000>;
> regulator-max-microvolt =
<1350000>;
> regulator-always-on;
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ARM: dts: Update vdd_arm regulator
2013-06-08 11:52 ` [PATCH] ARM: dts: Update vdd_arm regulator Tomasz Figa
@ 2013-06-10 3:44 ` Tushar Behera
2013-06-20 11:10 ` Tushar Behera
0 siblings, 1 reply; 3+ messages in thread
From: Tushar Behera @ 2013-06-10 3:44 UTC (permalink / raw)
To: Tomasz Figa
Cc: linux-samsung-soc, kgene.kim, patches, broonie, swarren,
grant.likely, rob.herring, l.majewski, s.nawrocki,
devicetree-discuss
On 06/08/2013 05:22 PM, Tomasz Figa wrote:
> Hi Tushar,
>
> On Thursday 06 of June 2013 16:32:52 Tushar Behera wrote:
>> Cpufreq driver for EXYNOS4210 is not a platform driver, hence it is not
>> possible to provide the regulator supply name through DT bindings.
>> Since the cpufreq driver requires the regulator to be named as
>> 'vdd_arm', the related regulator name should be kept same.
>>
>> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
>> ---
>> arch/arm/boot/dts/exynos4210-origen.dts | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/exynos4210-origen.dts
>> b/arch/arm/boot/dts/exynos4210-origen.dts index bcf8079..bd5f589 100644
>> --- a/arch/arm/boot/dts/exynos4210-origen.dts
>> +++ b/arch/arm/boot/dts/exynos4210-origen.dts
>> @@ -192,7 +192,7 @@
>> };
>>
>> buck1_reg: BUCK1 {
>> - regulator-name = "VDD_ARM_1.2V";
>> + regulator-name = "vdd_arm";
>
> Yes, this is the hack I mentioned in my review of
> [PATCH 0/2] Clock update for EXYNOS4210-CPUFREQ driver
>
We can hold this patch till we get to a conclusion for the above
mentioned patch set.
> Best regards,
> Tomasz
>
>> regulator-min-microvolt =
> <950000>;
>> regulator-max-microvolt =
> <1350000>;
>> regulator-always-on;
--
Tushar Behera
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ARM: dts: Update vdd_arm regulator
2013-06-10 3:44 ` Tushar Behera
@ 2013-06-20 11:10 ` Tushar Behera
0 siblings, 0 replies; 3+ messages in thread
From: Tushar Behera @ 2013-06-20 11:10 UTC (permalink / raw)
To: kgene.kim
Cc: Tomasz Figa, linux-samsung-soc, patches, broonie, swarren,
grant.likely, rob.herring, l.majewski, s.nawrocki,
devicetree-discuss
On 06/10/2013 09:14 AM, Tushar Behera wrote:
> On 06/08/2013 05:22 PM, Tomasz Figa wrote:
>> Hi Tushar,
>>
>> On Thursday 06 of June 2013 16:32:52 Tushar Behera wrote:
>>> Cpufreq driver for EXYNOS4210 is not a platform driver, hence it is not
>>> possible to provide the regulator supply name through DT bindings.
>>> Since the cpufreq driver requires the regulator to be named as
>>> 'vdd_arm', the related regulator name should be kept same.
>>>
>>> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
>>> ---
>>> arch/arm/boot/dts/exynos4210-origen.dts | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/arch/arm/boot/dts/exynos4210-origen.dts
>>> b/arch/arm/boot/dts/exynos4210-origen.dts index bcf8079..bd5f589 100644
>>> --- a/arch/arm/boot/dts/exynos4210-origen.dts
>>> +++ b/arch/arm/boot/dts/exynos4210-origen.dts
>>> @@ -192,7 +192,7 @@
>>> };
>>>
>>> buck1_reg: BUCK1 {
>>> - regulator-name = "VDD_ARM_1.2V";
>>> + regulator-name = "vdd_arm";
>>
>> Yes, this is the hack I mentioned in my review of
>> [PATCH 0/2] Clock update for EXYNOS4210-CPUFREQ driver
>>
>
> We can hold this patch till we get to a conclusion for the above
> mentioned patch set.
>
Kukjin,
We have agreed to go ahead with this approach till the cpufreq driver
has not been modified.
Can you please take this patch now?
--
Tushar Behera
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-06-20 11:10 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1370516572-313-1-git-send-email-tushar.behera@linaro.org>
2013-06-08 11:52 ` [PATCH] ARM: dts: Update vdd_arm regulator Tomasz Figa
2013-06-10 3:44 ` Tushar Behera
2013-06-20 11:10 ` Tushar Behera
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).