Re: [PATCH] ARM: shmobile: rcar-gen2: Correct arch timer frequency on RZ/G1E

[Simon Horman]

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 
> ---
> Untested due to lack of hardware.
> 
> v2:
>   - Rebased to avoid dependencies.

Thanks. I have applied this for v4.14.

Re: [PATCH] ARM: shmobile: rcar-gen2: Correct arch timer frequency on RZ/G1E

[Simon Horman]

On Tue, Jul 18, 2017 at 01:59:58PM +, 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

RE: [PATCH] ARM: shmobile: rcar-gen2: Correct arch timer frequency on RZ/G1E

[Biju Das]

It looks good to me.

Regards,
Biju

> -Original Message-
> From: Geert Uytterhoeven [mailto:geert+rene...@glider.be]
> Sent: 18 July 2017 14:29
> To: Simon Horman <ho...@verge.net.au>; Magnus Damm
> <magnus.d...@gmail.com>
> Cc: Sergei Shtylyov <sergei.shtyl...@cogentembedded.com>; Biju Das
> <biju@bp.renesas.com>; linux-renesas-soc@vger.kernel.org; linux-arm-
> ker...@lists.infradead.org; Geert Uytterhoeven <geert+rene...@glider.be>
> 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+rene...@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(-)
>
> 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 = 26000 / 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.

[PATCH] ARM: shmobile: rcar-gen2: Correct arch timer frequency on RZ/G1E

[Geert Uytterhoeven]

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 
---
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 = 26000 / 8;   /* ZS / 8 */
/* CNTVOFF has to be initialized either from non-secure
-- 
2.7.4