According to the datasheet, the frequency of the ARM architecture timer
on RZ/G1E depends on the frequency of the ZS clock, just like on R-Car
E2 and V2H.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Untested due to lack of hardware.
v2:
- Rebased to avoid dependencies.
---
arch/arm/mach-shmobile/setup-rcar-gen2.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
@@ -70,7 +70,8 @@ void __init rcar_gen2_timer_init(void)void__iomem*base;u32freq;-if(of_machine_is_compatible("renesas,r8a7792")||+if(of_machine_is_compatible("renesas,r8a7745")||+of_machine_is_compatible("renesas,r8a7792")||of_machine_is_compatible("renesas,r8a7794")){freq=260000000/8;/* ZS / 8 *//* CNTVOFF has to be initialized either from non-secure
-----Original Message-----
From: Geert Uytterhoeven [mailto:geert+renesas at glider.be]
Sent: 18 July 2017 14:29
To: Simon Horman <horms@verge.net.au>; Magnus Damm
[off-list ref]
Cc: Sergei Shtylyov <redacted>; Biju Das
[off-list ref]; linux-renesas-soc at vger.kernel.org; linux-arm-
kernel at lists.infradead.org; Geert Uytterhoeven [off-list ref]
Subject: [PATCH] ARM: shmobile: rcar-gen2: Correct arch timer frequency on
RZ/G1E
According to the datasheet, the frequency of the ARM architecture timer on
RZ/G1E depends on the frequency of the ZS clock, just like on R-Car
E2 and V2H.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Untested due to lack of hardware.
v2:
- Rebased to avoid dependencies.
---
arch/arm/mach-shmobile/setup-rcar-gen2.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
@@ -70,7 +70,8 @@ void __init rcar_gen2_timer_init(void)void__iomem*base;u32freq;-if(of_machine_is_compatible("renesas,r8a7792")||+if(of_machine_is_compatible("renesas,r8a7745")||+of_machine_is_compatible("renesas,r8a7792")||of_machine_is_compatible("renesas,r8a7794")){freq=260000000/8;/* ZS / 8 *//* CNTVOFF has to be initialized either from non-secure--
2.7.4
Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
On Tue, Jul 18, 2017 at 01:59:58PM +0000, Biju Das wrote:
It looks good to me.
Regards,
Biju
Thanks for your review. A few comments on it to keep in mind for next time:
* Please don't top-post
* Please consider providing a more formal tag, e.g. Reviewed-by
On Tue, Jul 18, 2017 at 03:29:17PM +0200, Geert Uytterhoeven wrote:
According to the datasheet, the frequency of the ARM architecture timer
on RZ/G1E depends on the frequency of the ZS clock, just like on R-Car
E2 and V2H.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Untested due to lack of hardware.
v2:
- Rebased to avoid dependencies.