[PATCH 01/10] ARM: shmobile: sh7372: Update DTS to include CPU frequency

[PATCH 01/10] ARM: shmobile: sh7372: Update DTS to include CPU frequency

[Magnus Damm]

From: Magnus Damm <damm+renesas@opensource.se>

Add CPU Frequency information to the sh7372 DTS file. This
will allow us to use the shared C code on sh7372 and Mackerel
which reads out the clock frequency from DT and calculates the
delay settings from there.

Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
---

 arch/arm/boot/dts/sh7372.dtsi |    1 +
 1 file changed, 1 insertion(+)

--- 0001/arch/arm/boot/dts/sh7372.dtsi
+++ work/arch/arm/boot/dts/sh7372.dtsi	2014-08-20 21:00:26.000000000 +0900
@@ -21,6 +21,7 @@
 			compatible = "arm,cortex-a8";
 			device_type = "cpu";
 			reg = <0x0>;
+			clock-frequency = <800000000>;
 		};
 	};
 

Re: [PATCH 01/10] ARM: shmobile: sh7372: Update DTS to include CPU frequency

[Geert Uytterhoeven]

Hi Magnus,

On Wed, Aug 20, 2014 at 3:02 PM, Magnus Damm <magnus.damm@gmail.com> wrote:
> Add CPU Frequency information to the sh7372 DTS file. This
> will allow us to use the shared C code on sh7372 and Mackerel
> which reads out the clock frequency from DT and calculates the
> delay settings from there.
>
> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> ---
>
>  arch/arm/boot/dts/sh7372.dtsi |    1 +
>  1 file changed, 1 insertion(+)
>
> --- 0001/arch/arm/boot/dts/sh7372.dtsi
> +++ work/arch/arm/boot/dts/sh7372.dtsi  2014-08-20 21:00:26.000000000 +0900
> @@ -21,6 +21,7 @@
>                         compatible = "arm,cortex-a8";
>                         device_type = "cpu";
>                         reg = <0x0>;
> +                       clock-frequency = <800000000>;

Should this be overridden in sh7372-mackerel.dtsi?

While the sh7372 manual states 800 MHz, the Mackerel manual states
806 MHz.

>                 };
>         };

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Re: [PATCH 01/10] ARM: shmobile: sh7372: Update DTS to include CPU frequency

[Magnus Damm]

Hi Geert,

On Wed, Aug 20, 2014 at 11:36 PM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> Hi Magnus,
>
> On Wed, Aug 20, 2014 at 3:02 PM, Magnus Damm <magnus.damm@gmail.com> wrote:
>> Add CPU Frequency information to the sh7372 DTS file. This
>> will allow us to use the shared C code on sh7372 and Mackerel
>> which reads out the clock frequency from DT and calculates the
>> delay settings from there.
>>
>> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
>> ---
>>
>>  arch/arm/boot/dts/sh7372.dtsi |    1 +
>>  1 file changed, 1 insertion(+)
>>
>> --- 0001/arch/arm/boot/dts/sh7372.dtsi
>> +++ work/arch/arm/boot/dts/sh7372.dtsi  2014-08-20 21:00:26.000000000 +0900
>> @@ -21,6 +21,7 @@
>>                         compatible = "arm,cortex-a8";
>>                         device_type = "cpu";
>>                         reg = <0x0>;
>> +                       clock-frequency = <800000000>;
>
> Should this be overridden in sh7372-mackerel.dtsi?
>
> While the sh7372 manual states 800 MHz, the Mackerel manual states
> 806 MHz.

I guess overriding makes sense for boards that operate out of spec... =/

For sh7372 this isn't really used by anything except the generic
(broken) DT case, so Mackerel legacy case doesn't care at this point.

Looking forward to throw sh7372 support out of the window. =)

/ magnus

[PATCH 01/10] ARM: shmobile: sh7372: Update DTS to include CPU frequency

[Simon Horman]

From: Magnus Damm <damm+renesas@opensource.se>

Add CPU Frequency information to the sh7372 DTS file. This
will allow us to use the shared C code on sh7372 and Mackerel
which reads out the clock frequency from DT and calculates the
delay settings from there.

Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/boot/dts/sh7372.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/sh7372.dtsi b/arch/arm/boot/dts/sh7372.dtsi
index 249f65be2..f863a10 100644
--- a/arch/arm/boot/dts/sh7372.dtsi
+++ b/arch/arm/boot/dts/sh7372.dtsi
@@ -21,6 +21,7 @@
 			compatible = "arm,cortex-a8";
 			device_type = "cpu";
 			reg = <0x0>;
+			clock-frequency = <800000000>;
 		};
 	};
 
-- 
2.0.1

[PATCH 01/10] ARM: shmobile: sh7372: Update DTS to include CPU frequency

[Simon Horman]

From: Magnus Damm <damm+renesas@opensource.se>

Add CPU Frequency information to the sh7372 DTS file. This
will allow us to use the shared C code on sh7372 and Mackerel
which reads out the clock frequency from DT and calculates the
delay settings from there.

Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/boot/dts/sh7372.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/sh7372.dtsi b/arch/arm/boot/dts/sh7372.dtsi
index 249f65be2..f863a10 100644
--- a/arch/arm/boot/dts/sh7372.dtsi
+++ b/arch/arm/boot/dts/sh7372.dtsi
@@ -21,6 +21,7 @@
 			compatible = "arm,cortex-a8";
 			device_type = "cpu";
 			reg = <0x0>;
+			clock-frequency = <800000000>;
 		};
 	};
 
-- 
2.0.1