* 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).