It looks good to me. Regards, Biju > -----Original Message----- > From: Geert Uytterhoeven [mailto:geert+renesas@xxxxxxxxx] > Sent: 18 July 2017 14:29 > To: Simon Horman <horms@xxxxxxxxxxxx>; Magnus Damm > <magnus.damm@xxxxxxxxx> > Cc: Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx>; Biju Das > <biju.das@xxxxxxxxxxxxxx>; linux-renesas-soc@xxxxxxxxxxxxxxx; linux-arm- > kernel@xxxxxxxxxxxxxxxxxxx; Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > 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@xxxxxxxxx> > --- > 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(-) > > diff --git a/arch/arm/mach-shmobile/setup-rcar-gen2.c b/arch/arm/mach- > shmobile/setup-rcar-gen2.c > index 3bd505da31726869..7ab1690fab8299eb 100644 > --- a/arch/arm/mach-shmobile/setup-rcar-gen2.c > +++ b/arch/arm/mach-shmobile/setup-rcar-gen2.c > @@ -70,7 +70,8 @@ void __init rcar_gen2_timer_init(void) > void __iomem *base; > u32 freq; > > -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.