* [PATCH 07/22] ARM: exynos: Remove '0x's from Exynos4110 DTSI file
[not found] <1374490361-25769-1-git-send-email-lee.jones@linaro.org>
@ 2013-07-22 10:52 ` Lee Jones
2013-07-22 19:25 ` Tomasz Figa
` (2 more replies)
2013-07-22 10:52 ` [PATCH 08/22] ARM: dts: Remove '0x's from Exynos4x12 " Lee Jones
` (3 subsequent siblings)
4 siblings, 3 replies; 17+ messages in thread
From: Lee Jones @ 2013-07-22 10:52 UTC (permalink / raw)
To: linux-arm-kernel, linux-kernel
Cc: arnd, linus.walleij, Lee Jones, Kukjin Kim, linux-samsung-soc
Cc: Kukjin Kim <kgene.kim@samsung.com>
Cc: linux-samsung-soc@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
arch/arm/boot/dts/exynos4210.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
index b7f358a..53e2527 100644
--- a/arch/arm/boot/dts/exynos4210.dtsi
+++ b/arch/arm/boot/dts/exynos4210.dtsi
@@ -72,7 +72,7 @@
};
};
- clock: clock-controller@0x10030000 {
+ clock: clock-controller@10030000 {
compatible = "samsung,exynos4210-clock";
reg = <0x10030000 0x20000>;
#clock-cells = <1>;
--
1.8.1.2
^ permalink raw reply related [flat|nested] 17+ messages in thread* Re: [PATCH 07/22] ARM: exynos: Remove '0x's from Exynos4110 DTSI file
2013-07-22 10:52 ` [PATCH 07/22] ARM: exynos: Remove '0x's from Exynos4110 DTSI file Lee Jones
@ 2013-07-22 19:25 ` Tomasz Figa
2013-07-23 8:55 ` Lee Jones
2013-07-24 12:39 ` [PATCH v2 07/22] ARM: Exynos4210: " Lee Jones
2013-07-24 15:09 ` [PATCH v3 07/22] ARM: dts: Remove '0x's from Exynos4210 " Lee Jones
2 siblings, 1 reply; 17+ messages in thread
From: Tomasz Figa @ 2013-07-22 19:25 UTC (permalink / raw)
To: Lee Jones
Cc: linux-arm-kernel, linux-kernel, arnd, linus.walleij, Kukjin Kim,
linux-samsung-soc
Hi Lee,
On Monday 22 of July 2013 11:52:26 Lee Jones wrote:
> Cc: Kukjin Kim <kgene.kim@samsung.com>
> Cc: linux-samsung-soc@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> ---
> arch/arm/boot/dts/exynos4210.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/exynos4210.dtsi
> b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
> --- a/arch/arm/boot/dts/exynos4210.dtsi
> +++ b/arch/arm/boot/dts/exynos4210.dtsi
> @@ -72,7 +72,7 @@
> };
> };
>
> - clock: clock-controller@0x10030000 {
> + clock: clock-controller@10030000 {
> compatible = "samsung,exynos4210-clock";
> reg = <0x10030000 0x20000>;
> #clock-cells = <1>;
This looks fine, but please fix commit message - it should be Exynos4210.
Also some explanation why this change is needed would be good, even if
it's obvious.
Best regards,
Tomasz
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH 07/22] ARM: exynos: Remove '0x's from Exynos4110 DTSI file
2013-07-22 19:25 ` Tomasz Figa
@ 2013-07-23 8:55 ` Lee Jones
2013-07-23 9:32 ` Tomasz Figa
0 siblings, 1 reply; 17+ messages in thread
From: Lee Jones @ 2013-07-23 8:55 UTC (permalink / raw)
To: Tomasz Figa
Cc: linux-arm-kernel, linux-kernel, arnd, linus.walleij, Kukjin Kim,
linux-samsung-soc
On Mon, 22 Jul 2013, Tomasz Figa wrote:
> Hi Lee,
>
> On Monday 22 of July 2013 11:52:26 Lee Jones wrote:
> > Cc: Kukjin Kim <kgene.kim@samsung.com>
> > Cc: linux-samsung-soc@vger.kernel.org
> > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> > ---
> > arch/arm/boot/dts/exynos4210.dtsi | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/boot/dts/exynos4210.dtsi
> > b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
> > --- a/arch/arm/boot/dts/exynos4210.dtsi
> > +++ b/arch/arm/boot/dts/exynos4210.dtsi
> > @@ -72,7 +72,7 @@
> > };
> > };
> >
> > - clock: clock-controller@0x10030000 {
> > + clock: clock-controller@10030000 {
> > compatible = "samsung,exynos4210-clock";
> > reg = <0x10030000 0x20000>;
> > #clock-cells = <1>;
>
> This looks fine, but please fix commit message - it should be Exynos4210.
> Also some explanation why this change is needed would be good, even if
> it's obvious.
Hi Tomasz,
I'm happy to fixup the $SUBJECT line, but do we really have to enter
an explanation if it's obvious? Seems a little belt and braces.
Kind regards,
Lee
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH 07/22] ARM: exynos: Remove '0x's from Exynos4110 DTSI file
2013-07-23 8:55 ` Lee Jones
@ 2013-07-23 9:32 ` Tomasz Figa
0 siblings, 0 replies; 17+ messages in thread
From: Tomasz Figa @ 2013-07-23 9:32 UTC (permalink / raw)
To: Lee Jones
Cc: Tomasz Figa, linux-arm-kernel, linux-kernel, arnd, linus.walleij,
Kukjin Kim, linux-samsung-soc
On Tuesday 23 of July 2013 09:55:52 Lee Jones wrote:
> On Mon, 22 Jul 2013, Tomasz Figa wrote:
> > Hi Lee,
> >
> > On Monday 22 of July 2013 11:52:26 Lee Jones wrote:
> > > Cc: Kukjin Kim <kgene.kim@samsung.com>
> > > Cc: linux-samsung-soc@vger.kernel.org
> > > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> > > ---
> > >
> > > arch/arm/boot/dts/exynos4210.dtsi | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/arch/arm/boot/dts/exynos4210.dtsi
> > > b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
> > > --- a/arch/arm/boot/dts/exynos4210.dtsi
> > > +++ b/arch/arm/boot/dts/exynos4210.dtsi
> > > @@ -72,7 +72,7 @@
> > >
> > > };
> > >
> > > };
> > >
> > > - clock: clock-controller@0x10030000 {
> > > + clock: clock-controller@10030000 {
> > >
> > > compatible = "samsung,exynos4210-clock";
> > > reg = <0x10030000 0x20000>;
> > > #clock-cells = <1>;
> >
> > This looks fine, but please fix commit message - it should be
> > Exynos4210. Also some explanation why this change is needed would be
> > good, even if it's obvious.
>
> Hi Tomasz,
>
> I'm happy to fixup the $SUBJECT line, but do we really have to enter
> an explanation if it's obvious? Seems a little belt and braces.
It's obvious for us, people working with device tree, but for people that
usually don't it might not be.
Something among following lines would be fine:
This patch removes "0x" prefix from addresses in DT nodes for the sake of
consistency with other nodes and assumed convention.
Best regards,
Tomasz
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH v2 07/22] ARM: Exynos4210: Remove '0x's from Exynos4110 DTSI file
2013-07-22 10:52 ` [PATCH 07/22] ARM: exynos: Remove '0x's from Exynos4110 DTSI file Lee Jones
2013-07-22 19:25 ` Tomasz Figa
@ 2013-07-24 12:39 ` Lee Jones
2013-07-24 14:03 ` Tomasz Figa
2013-07-24 15:09 ` [PATCH v3 07/22] ARM: dts: Remove '0x's from Exynos4210 " Lee Jones
2 siblings, 1 reply; 17+ messages in thread
From: Lee Jones @ 2013-07-24 12:39 UTC (permalink / raw)
To: linux-arm-kernel, linux-kernel, Tomasz Figa
Cc: arnd, linus.walleij, Kukjin Kim, linux-samsung-soc
... for the sake of consistency and assumed convention.
Cc: Kukjin Kim <kgene.kim@samsung.com>
Cc: linux-samsung-soc@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
index b7f358a..53e2527 100644
--- a/arch/arm/boot/dts/exynos4210.dtsi
+++ b/arch/arm/boot/dts/exynos4210.dtsi
@@ -72,7 +72,7 @@
};
};
- clock: clock-controller@0x10030000 {
+ clock: clock-controller@10030000 {
compatible = "samsung,exynos4210-clock";
reg = <0x10030000 0x20000>;
#clock-cells = <1>;
^ permalink raw reply related [flat|nested] 17+ messages in thread* Re: [PATCH v2 07/22] ARM: Exynos4210: Remove '0x's from Exynos4110 DTSI file
2013-07-24 12:39 ` [PATCH v2 07/22] ARM: Exynos4210: " Lee Jones
@ 2013-07-24 14:03 ` Tomasz Figa
2013-07-24 15:05 ` Lee Jones
0 siblings, 1 reply; 17+ messages in thread
From: Tomasz Figa @ 2013-07-24 14:03 UTC (permalink / raw)
To: Lee Jones
Cc: linux-arm-kernel, linux-kernel, Tomasz Figa, arnd, linus.walleij,
Kukjin Kim, linux-samsung-soc
On Wednesday 24 of July 2013 13:39:31 Lee Jones wrote:
> ... for the sake of consistency and assumed convention.
>
> Cc: Kukjin Kim <kgene.kim@samsung.com>
> Cc: linux-samsung-soc@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
>
> diff --git a/arch/arm/boot/dts/exynos4210.dtsi
> b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
> --- a/arch/arm/boot/dts/exynos4210.dtsi
> +++ b/arch/arm/boot/dts/exynos4210.dtsi
> @@ -72,7 +72,7 @@
> };
> };
>
> - clock: clock-controller@0x10030000 {
> + clock: clock-controller@10030000 {
> compatible = "samsung,exynos4210-clock";
> reg = <0x10030000 0x20000>;
> #clock-cells = <1>;
Looks good to me, except the subject which is still wrong -
s/Exynos4110/Exynos4210/.
I guess it might be fixed when applying, though.
Best regards,
Tomasz
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH v2 07/22] ARM: Exynos4210: Remove '0x's from Exynos4110 DTSI file
2013-07-24 14:03 ` Tomasz Figa
@ 2013-07-24 15:05 ` Lee Jones
0 siblings, 0 replies; 17+ messages in thread
From: Lee Jones @ 2013-07-24 15:05 UTC (permalink / raw)
To: Tomasz Figa
Cc: linux-arm-kernel, linux-kernel, Tomasz Figa, arnd, linus.walleij,
Kukjin Kim, linux-samsung-soc
On Wed, 24 Jul 2013, Tomasz Figa wrote:
> On Wednesday 24 of July 2013 13:39:31 Lee Jones wrote:
> > ... for the sake of consistency and assumed convention.
> >
> > Cc: Kukjin Kim <kgene.kim@samsung.com>
> > Cc: linux-samsung-soc@vger.kernel.org
> > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> >
> > diff --git a/arch/arm/boot/dts/exynos4210.dtsi
> > b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
> > --- a/arch/arm/boot/dts/exynos4210.dtsi
> > +++ b/arch/arm/boot/dts/exynos4210.dtsi
> > @@ -72,7 +72,7 @@
> > };
> > };
> >
> > - clock: clock-controller@0x10030000 {
> > + clock: clock-controller@10030000 {
> > compatible = "samsung,exynos4210-clock";
> > reg = <0x10030000 0x20000>;
> > #clock-cells = <1>;
>
> Looks good to me, except the subject which is still wrong -
> s/Exynos4110/Exynos4210/.
Ah, I get you know. My bad, will fix.
> I guess it might be fixed when applying, though.
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH v3 07/22] ARM: dts: Remove '0x's from Exynos4210 DTSI file
2013-07-22 10:52 ` [PATCH 07/22] ARM: exynos: Remove '0x's from Exynos4110 DTSI file Lee Jones
2013-07-22 19:25 ` Tomasz Figa
2013-07-24 12:39 ` [PATCH v2 07/22] ARM: Exynos4210: " Lee Jones
@ 2013-07-24 15:09 ` Lee Jones
2013-07-24 15:24 ` Tomasz Figa
2013-08-05 18:07 ` Kukjin Kim
2 siblings, 2 replies; 17+ messages in thread
From: Lee Jones @ 2013-07-24 15:09 UTC (permalink / raw)
To: linux-arm-kernel, linux-kernel, Tomasz Figa
Cc: arnd, linus.walleij, Kukjin Kim, linux-samsung-soc
... for the sake of consistency and assumed convention.
Cc: Kukjin Kim <kgene.kim@samsung.com>
Cc: linux-samsung-soc@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
index b7f358a..53e2527 100644
--- a/arch/arm/boot/dts/exynos4210.dtsi
+++ b/arch/arm/boot/dts/exynos4210.dtsi
@@ -72,7 +72,7 @@
};
};
- clock: clock-controller@0x10030000 {
+ clock: clock-controller@10030000 {
compatible = "samsung,exynos4210-clock";
reg = <0x10030000 0x20000>;
#clock-cells = <1>;
^ permalink raw reply related [flat|nested] 17+ messages in thread* Re: [PATCH v3 07/22] ARM: dts: Remove '0x's from Exynos4210 DTSI file
2013-07-24 15:09 ` [PATCH v3 07/22] ARM: dts: Remove '0x's from Exynos4210 " Lee Jones
@ 2013-07-24 15:24 ` Tomasz Figa
2013-07-25 7:52 ` Lee Jones
2013-08-05 18:07 ` Kukjin Kim
1 sibling, 1 reply; 17+ messages in thread
From: Tomasz Figa @ 2013-07-24 15:24 UTC (permalink / raw)
To: Lee Jones
Cc: linux-arm-kernel, linux-kernel, arnd, linus.walleij, Kukjin Kim,
linux-samsung-soc
On Wednesday 24 of July 2013 16:09:37 Lee Jones wrote:
> ... for the sake of consistency and assumed convention.
>
> Cc: Kukjin Kim <kgene.kim@samsung.com>
> Cc: linux-samsung-soc@vger.kernel.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
>
> diff --git a/arch/arm/boot/dts/exynos4210.dtsi
> b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
> --- a/arch/arm/boot/dts/exynos4210.dtsi
> +++ b/arch/arm/boot/dts/exynos4210.dtsi
> @@ -72,7 +72,7 @@
> };
> };
>
> - clock: clock-controller@0x10030000 {
> + clock: clock-controller@10030000 {
> compatible = "samsung,exynos4210-clock";
> reg = <0x10030000 0x20000>;
> #clock-cells = <1>;
Acked-by: Tomasz Figa <t.figa@samsung.com>
Best regards,
Tomasz
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH v3 07/22] ARM: dts: Remove '0x's from Exynos4210 DTSI file
2013-07-24 15:24 ` Tomasz Figa
@ 2013-07-25 7:52 ` Lee Jones
2013-08-05 17:58 ` Kukjin Kim
0 siblings, 1 reply; 17+ messages in thread
From: Lee Jones @ 2013-07-25 7:52 UTC (permalink / raw)
To: Tomasz Figa
Cc: linux-arm-kernel, linux-kernel, arnd, linus.walleij, Kukjin Kim,
linux-samsung-soc
On Wed, 24 Jul 2013, Tomasz Figa wrote:
> On Wednesday 24 of July 2013 16:09:37 Lee Jones wrote:
> > ... for the sake of consistency and assumed convention.
> >
> > Cc: Kukjin Kim <kgene.kim@samsung.com>
> > Cc: linux-samsung-soc@vger.kernel.org
> > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> >
> > diff --git a/arch/arm/boot/dts/exynos4210.dtsi
> > b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
> > --- a/arch/arm/boot/dts/exynos4210.dtsi
> > +++ b/arch/arm/boot/dts/exynos4210.dtsi
> > @@ -72,7 +72,7 @@
> > };
> > };
> >
> > - clock: clock-controller@0x10030000 {
> > + clock: clock-controller@10030000 {
> > compatible = "samsung,exynos4210-clock";
> > reg = <0x10030000 0x20000>;
> > #clock-cells = <1>;
>
> Acked-by: Tomasz Figa <t.figa@samsung.com>
Thanks Tomasz.
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH v3 07/22] ARM: dts: Remove '0x's from Exynos4210 DTSI file
2013-07-25 7:52 ` Lee Jones
@ 2013-08-05 17:58 ` Kukjin Kim
0 siblings, 0 replies; 17+ messages in thread
From: Kukjin Kim @ 2013-08-05 17:58 UTC (permalink / raw)
To: Lee Jones
Cc: Tomasz Figa, linux-arm-kernel, linux-kernel, arnd, linus.walleij,
Kukjin Kim, linux-samsung-soc
On 07/25/13 16:52, Lee Jones wrote:
> On Wed, 24 Jul 2013, Tomasz Figa wrote:
>
>> On Wednesday 24 of July 2013 16:09:37 Lee Jones wrote:
>>> ... for the sake of consistency and assumed convention.
>>>
>>> Cc: Kukjin Kim<kgene.kim@samsung.com>
>>> Cc: linux-samsung-soc@vger.kernel.org
>>> Signed-off-by: Lee Jones<lee.jones@linaro.org>
>>>
>>> diff --git a/arch/arm/boot/dts/exynos4210.dtsi
>>> b/arch/arm/boot/dts/exynos4210.dtsi index b7f358a..53e2527 100644
>>> --- a/arch/arm/boot/dts/exynos4210.dtsi
>>> +++ b/arch/arm/boot/dts/exynos4210.dtsi
>>> @@ -72,7 +72,7 @@
>>> };
>>> };
>>>
>>> - clock: clock-controller@0x10030000 {
>>> + clock: clock-controller@10030000 {
>>> compatible = "samsung,exynos4210-clock";
>>> reg =<0x10030000 0x20000>;
>>> #clock-cells =<1>;
>>
>> Acked-by: Tomasz Figa<t.figa@samsung.com>
>
> Thanks Tomasz.
>
Thanks, Lee Jones and Tomasz.
Applied #7 ~ #11 into the cleanup of samsung tree.
- Kukjin
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v3 07/22] ARM: dts: Remove '0x's from Exynos4210 DTSI file
2013-07-24 15:09 ` [PATCH v3 07/22] ARM: dts: Remove '0x's from Exynos4210 " Lee Jones
2013-07-24 15:24 ` Tomasz Figa
@ 2013-08-05 18:07 ` Kukjin Kim
2013-08-06 7:18 ` Lee Jones
1 sibling, 1 reply; 17+ messages in thread
From: Kukjin Kim @ 2013-08-05 18:07 UTC (permalink / raw)
To: Lee Jones
Cc: linux-arm-kernel, linux-kernel, Tomasz Figa, arnd, linus.walleij,
Kukjin Kim, linux-samsung-soc
On 07/25/13 00:09, Lee Jones wrote:
> ... for the sake of consistency and assumed convention.
>
> Cc: Kukjin Kim<kgene.kim@samsung.com>
> Cc: linux-samsung-soc@vger.kernel.org
> Signed-off-by: Lee Jones<lee.jones@linaro.org>
>
> diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
> index b7f358a..53e2527 100644
> --- a/arch/arm/boot/dts/exynos4210.dtsi
> +++ b/arch/arm/boot/dts/exynos4210.dtsi
> @@ -72,7 +72,7 @@
> };
^^^^^^^^^^^^^^^^
> };
^^^^^^^^
>
> - clock: clock-controller@0x10030000 {
^^^^^^^^
> + clock: clock-controller@10030000 {
^^^^^^^^
> compatible = "samsung,exynos4210-clock";
^^^^^^^^^^^^^^^^
> reg =<0x10030000 0x20000>;
^^^^^^^^^^^^^^^^
> #clock-cells =<1>;
^^^^^^^^^^^^^^^^
BTW, should be tab at the mark '^' not white space :(
- Kukjin
^ permalink raw reply [flat|nested] 17+ messages in thread* Re: [PATCH v3 07/22] ARM: dts: Remove '0x's from Exynos4210 DTSI file
2013-08-05 18:07 ` Kukjin Kim
@ 2013-08-06 7:18 ` Lee Jones
0 siblings, 0 replies; 17+ messages in thread
From: Lee Jones @ 2013-08-06 7:18 UTC (permalink / raw)
To: Kukjin Kim
Cc: linux-arm-kernel, linux-kernel, Tomasz Figa, arnd, linus.walleij,
linux-samsung-soc
On Tue, 06 Aug 2013, Kukjin Kim wrote:
> On 07/25/13 00:09, Lee Jones wrote:
> >... for the sake of consistency and assumed convention.
> >
> >Cc: Kukjin Kim<kgene.kim@samsung.com>
> >Cc: linux-samsung-soc@vger.kernel.org
> >Signed-off-by: Lee Jones<lee.jones@linaro.org>
> >
> >diff --git a/arch/arm/boot/dts/exynos4210.dtsi b/arch/arm/boot/dts/exynos4210.dtsi
> >index b7f358a..53e2527 100644
> >--- a/arch/arm/boot/dts/exynos4210.dtsi
> >+++ b/arch/arm/boot/dts/exynos4210.dtsi
> >@@ -72,7 +72,7 @@
> > };
> ^^^^^^^^^^^^^^^^
> > };
> ^^^^^^^^
> >
> >- clock: clock-controller@0x10030000 {
> ^^^^^^^^
> >+ clock: clock-controller@10030000 {
> ^^^^^^^^
> > compatible = "samsung,exynos4210-clock";
> ^^^^^^^^^^^^^^^^
> > reg =<0x10030000 0x20000>;
> ^^^^^^^^^^^^^^^^
> > #clock-cells =<1>;
> ^^^^^^^^^^^^^^^^
>
> BTW, should be tab at the mark '^' not white space :(
Agreed, but this should be a separate patch.
--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 08/22] ARM: dts: Remove '0x's from Exynos4x12 DTSI file
[not found] <1374490361-25769-1-git-send-email-lee.jones@linaro.org>
2013-07-22 10:52 ` [PATCH 07/22] ARM: exynos: Remove '0x's from Exynos4110 DTSI file Lee Jones
@ 2013-07-22 10:52 ` Lee Jones
2013-07-22 10:52 ` [PATCH 09/22] ARM: dts: Remove '0x's from Exynos5250 DTS file Lee Jones
` (2 subsequent siblings)
4 siblings, 0 replies; 17+ messages in thread
From: Lee Jones @ 2013-07-22 10:52 UTC (permalink / raw)
To: linux-arm-kernel, linux-kernel
Cc: arnd, linus.walleij, Lee Jones, Kukjin Kim, linux-samsung-soc
Cc: Kukjin Kim <kgene.kim@samsung.com>
Cc: linux-samsung-soc@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
arch/arm/boot/dts/exynos4x12.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/exynos4x12.dtsi b/arch/arm/boot/dts/exynos4x12.dtsi
index 01da194..3aa2f06 100644
--- a/arch/arm/boot/dts/exynos4x12.dtsi
+++ b/arch/arm/boot/dts/exynos4x12.dtsi
@@ -28,7 +28,7 @@
pinctrl3 = &pinctrl_3;
};
- clock: clock-controller@0x10030000 {
+ clock: clock-controller@10030000 {
compatible = "samsung,exynos4412-clock";
reg = <0x10030000 0x20000>;
#clock-cells = <1>;
--
1.8.1.2
^ permalink raw reply related [flat|nested] 17+ messages in thread* [PATCH 09/22] ARM: dts: Remove '0x's from Exynos5250 DTS file
[not found] <1374490361-25769-1-git-send-email-lee.jones@linaro.org>
2013-07-22 10:52 ` [PATCH 07/22] ARM: exynos: Remove '0x's from Exynos4110 DTSI file Lee Jones
2013-07-22 10:52 ` [PATCH 08/22] ARM: dts: Remove '0x's from Exynos4x12 " Lee Jones
@ 2013-07-22 10:52 ` Lee Jones
2013-07-22 10:52 ` [PATCH 10/22] ARM: dts: Remove '0x's from Exynos5420 " Lee Jones
2013-07-22 10:52 ` [PATCH 11/22] ARM: dts: Remove '0x's from Exynos5440 " Lee Jones
4 siblings, 0 replies; 17+ messages in thread
From: Lee Jones @ 2013-07-22 10:52 UTC (permalink / raw)
To: linux-arm-kernel, linux-kernel
Cc: arnd, linus.walleij, Lee Jones, Kukjin Kim, linux-samsung-soc
Cc: Kukjin Kim <kgene.kim@samsung.com>
Cc: linux-samsung-soc@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
arch/arm/boot/dts/exynos5250.dtsi | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi
index ef57277..24c0888 100644
--- a/arch/arm/boot/dts/exynos5250.dtsi
+++ b/arch/arm/boot/dts/exynos5250.dtsi
@@ -68,17 +68,17 @@
};
};
- pd_gsc: gsc-power-domain@0x10044000 {
+ pd_gsc: gsc-power-domain@10044000 {
compatible = "samsung,exynos4210-pd";
reg = <0x10044000 0x20>;
};
- pd_mfc: mfc-power-domain@0x10044040 {
+ pd_mfc: mfc-power-domain@10044040 {
compatible = "samsung,exynos4210-pd";
reg = <0x10044040 0x20>;
};
- clock: clock-controller@0x10010000 {
+ clock: clock-controller@10010000 {
compatible = "samsung,exynos5250-clock";
reg = <0x10010000 0x30000>;
#clock-cells = <1>;
@@ -562,7 +562,7 @@
};
};
- gsc_0: gsc@0x13e00000 {
+ gsc_0: gsc@13e00000 {
compatible = "samsung,exynos5-gsc";
reg = <0x13e00000 0x1000>;
interrupts = <0 85 0>;
@@ -571,7 +571,7 @@
clock-names = "gscl";
};
- gsc_1: gsc@0x13e10000 {
+ gsc_1: gsc@13e10000 {
compatible = "samsung,exynos5-gsc";
reg = <0x13e10000 0x1000>;
interrupts = <0 86 0>;
@@ -580,7 +580,7 @@
clock-names = "gscl";
};
- gsc_2: gsc@0x13e20000 {
+ gsc_2: gsc@13e20000 {
compatible = "samsung,exynos5-gsc";
reg = <0x13e20000 0x1000>;
interrupts = <0 87 0>;
@@ -589,7 +589,7 @@
clock-names = "gscl";
};
- gsc_3: gsc@0x13e30000 {
+ gsc_3: gsc@13e30000 {
compatible = "samsung,exynos5-gsc";
reg = <0x13e30000 0x1000>;
interrupts = <0 88 0>;
--
1.8.1.2
^ permalink raw reply related [flat|nested] 17+ messages in thread* [PATCH 10/22] ARM: dts: Remove '0x's from Exynos5420 DTS file
[not found] <1374490361-25769-1-git-send-email-lee.jones@linaro.org>
` (2 preceding siblings ...)
2013-07-22 10:52 ` [PATCH 09/22] ARM: dts: Remove '0x's from Exynos5250 DTS file Lee Jones
@ 2013-07-22 10:52 ` Lee Jones
2013-07-22 10:52 ` [PATCH 11/22] ARM: dts: Remove '0x's from Exynos5440 " Lee Jones
4 siblings, 0 replies; 17+ messages in thread
From: Lee Jones @ 2013-07-22 10:52 UTC (permalink / raw)
To: linux-arm-kernel, linux-kernel
Cc: arnd, linus.walleij, Lee Jones, Kukjin Kim, linux-samsung-soc
Cc: Kukjin Kim <kgene.kim@samsung.com>
Cc: linux-samsung-soc@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
arch/arm/boot/dts/exynos5420.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/exynos5420.dtsi b/arch/arm/boot/dts/exynos5420.dtsi
index 8c54c4b..9e90d1e 100644
--- a/arch/arm/boot/dts/exynos5420.dtsi
+++ b/arch/arm/boot/dts/exynos5420.dtsi
@@ -59,7 +59,7 @@
};
};
- clock: clock-controller@0x10010000 {
+ clock: clock-controller@10010000 {
compatible = "samsung,exynos5420-clock";
reg = <0x10010000 0x30000>;
#clock-cells = <1>;
--
1.8.1.2
^ permalink raw reply related [flat|nested] 17+ messages in thread* [PATCH 11/22] ARM: dts: Remove '0x's from Exynos5440 DTS file
[not found] <1374490361-25769-1-git-send-email-lee.jones@linaro.org>
` (3 preceding siblings ...)
2013-07-22 10:52 ` [PATCH 10/22] ARM: dts: Remove '0x's from Exynos5420 " Lee Jones
@ 2013-07-22 10:52 ` Lee Jones
4 siblings, 0 replies; 17+ messages in thread
From: Lee Jones @ 2013-07-22 10:52 UTC (permalink / raw)
To: linux-arm-kernel, linux-kernel
Cc: arnd, linus.walleij, Lee Jones, Kukjin Kim, linux-samsung-soc
Cc: Kukjin Kim <kgene.kim@samsung.com>
Cc: linux-samsung-soc@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
arch/arm/boot/dts/exynos5440.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/exynos5440.dtsi b/arch/arm/boot/dts/exynos5440.dtsi
index ff7f5d8..0cedba4 100644
--- a/arch/arm/boot/dts/exynos5440.dtsi
+++ b/arch/arm/boot/dts/exynos5440.dtsi
@@ -20,7 +20,7 @@
spi0 = &spi_0;
};
- clock: clock-controller@0x160000 {
+ clock: clock-controller@160000 {
compatible = "samsung,exynos5440-clock";
reg = <0x160000 0x1000>;
#clock-cells = <1>;
--
1.8.1.2
^ permalink raw reply related [flat|nested] 17+ messages in thread