[0/3] ARM: dts: renesas: r8a73a4: Clock fixes and improvements

Message

Geert Uytterhoeven Jan. 15, 2024, 11:03 a.m. UTC

Hi all,

This patch series contains fixes and improvements for clock descriptions
of the Renesas R-Mobile APE6 SoC and the APE6-EVM development board.

After comparing CPU core performance over a wide range of SoCs, I had
been wondering for a long time why DMIPS/MHZ for the Cortex-A15 CPU
cores on R-Mobile APE6 is slightly higher than on R-Car Gen2 SoCs.
It turned out to be untrue, as the R-Mobile APE6 DTS contains a wrong
crystal osciallator freqency, causing the CPU cores on R-Mobile APE6 to
run 4% faster than assumed by Linux.  The first patch fixes this.
The two other patches contain small improvements and a small fix, none
of which have any functional impact.

I plan to queue this in renesas-devel for v6.9.


Thanks for your comments!

Geert Uytterhoeven (3):
  ARM: dts: renesas: r8a73a4: Fix external clocks and clock rate
  ARM: dts: renesas: r8a73a4: Add cp clock
  ARM: dts: renesas: r8a73a4: Fix thermal parent clock

 arch/arm/boot/dts/renesas/r8a73a4-ape6evm.dts | 12 ++++++++++
 arch/arm/boot/dts/renesas/r8a73a4.dtsi        | 23 +++++++++++++------
 2 files changed, 28 insertions(+), 7 deletions(-)

Comments

Niklas Söderlund Jan. 15, 2024, 5:04 p.m. UTC | #1

Hi Geert,

Thanks for your work.

On 2024-01-15 12:03:02 +0100, Geert Uytterhoeven wrote:
> 	Hi all,
> 
> This patch series contains fixes and improvements for clock descriptions
> of the Renesas R-Mobile APE6 SoC and the APE6-EVM development board.
> 
> After comparing CPU core performance over a wide range of SoCs, I had
> been wondering for a long time why DMIPS/MHZ for the Cortex-A15 CPU
> cores on R-Mobile APE6 is slightly higher than on R-Car Gen2 SoCs.
> It turned out to be untrue, as the R-Mobile APE6 DTS contains a wrong
> crystal osciallator freqency, causing the CPU cores on R-Mobile APE6 to
> run 4% faster than assumed by Linux.  The first patch fixes this.
> The two other patches contain small improvements and a small fix, none
> of which have any functional impact.
> 
> I plan to queue this in renesas-devel for v6.9.

For the whole series,

Reviewed-by: Niklas Söderlund <[e-mail]>

> 
> 
> Thanks for your comments!
> 
> Geert Uytterhoeven (3):
>   ARM: dts: renesas: r8a73a4: Fix external clocks and clock rate
>   ARM: dts: renesas: r8a73a4: Add cp clock
>   ARM: dts: renesas: r8a73a4: Fix thermal parent clock
> 
>  arch/arm/boot/dts/renesas/r8a73a4-ape6evm.dts | 12 ++++++++++
>  arch/arm/boot/dts/renesas/r8a73a4.dtsi        | 23 +++++++++++++------
>  2 files changed, 28 insertions(+), 7 deletions(-)
> 
> -- 
> 2.34.1
> 
> Gr{oetje,eeting}s,
> 
> 						Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [e-mail]
> 
> 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
>