| Message ID | 20140820130210.32075.68195.sendpatchset@w520 (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 2340cd112933ee837aa83017a6da23c72565d501 |
| Headers | show |
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 -- To unsubscribe from this list: send the line "unsubscribe linux-sh" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 -- To unsubscribe from this list: send the line "unsubscribe linux-sh" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
--- 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>; }; };