Re: [PATCH 0/6] Basic SoC support for r8a7743/r8a7745

Biju Das <biju.das@...>

-----Original Message-----
From: Ben Hutchings [mailto:ben.hutchings@...]
Sent: 19 July 2017 14:54
To: Biju Das <biju.das@...>
Cc: Chris Paterson <Chris.Paterson2@...>; cip-dev@...
Subject: Re: [PATCH 0/6] Basic SoC support for r8a7743/r8a7745

On Wed, 2017-07-19 at 08:24 +0000, Biju Das wrote:
Actually, there's something not quite right here. With just
CONFIG_ARCH_R8A7743 and CONFIG_ARCH_R8A7745 enabled (but no
other R-
platforms) I get:
The patches are for Basic SoC support. Next I am planning Clock
driver, then DT support for SoC, then Board DTS and then Enabling the config
option(CONFIG_ARCH_R8A7743 and CONFIG_ARCH_R8A7745) .

arch/arm/mach-shmobile/built-in.o: In function `rcar_gen2_timer_init':
undefined reference to `rcar_gen2_clocks_init'

There seems to have been a lot of refactoring of
drivers/clk/{shmobile,renesas} since 4.4 so I'm not sure how this should be

Current upstreamed RZ/G1M clock driver uses the new Renesas CPG/MSSR
clock driver frame work.
Backporting requires lot of refactoring of drivers/clk/{shmobile,renesas}.
So I don't think backporting the RZ/G1M clock driver is the right choice.

However r8a7743( RZ/G1M) is identical to r8a7791(RCar-M2) platform. We
ported r8a7743( RZ/G1M) clk driver based on this.
The current patch series, which I submitted is tested with this clk driver.
Then I think I should just make this change:

Subject: CIP: Build essential clock driver for Renesas RZ/G1 platforms

In mainline, clk-rcar-gen2 is selected by CONFIG_CLK_RCAR_GEN2 but here
there is no such common config symbol and we need to select it for each

Signed-off-by: Ben Hutchings <ben.hutchings@...>
drivers/clk/shmobile/Makefile | 2 ++
1 file changed, 2 insertions(+)

diff --git a/drivers/clk/shmobile/Makefile b/drivers/clk/shmobile/Makefile
index 97c71c885e4f..b339c0b6bb2f 100644
--- a/drivers/clk/shmobile/Makefile
+++ b/drivers/clk/shmobile/Makefile
@@ -2,6 +2,8 @@ obj-$(CONFIG_ARCH_EMEV2)+= clk-
obj-$(CONFIG_ARCH_R7S72100)+= clk-rz.o
obj-$(CONFIG_ARCH_R8A73A4)+= clk-r8a73a4.o
obj-$(CONFIG_ARCH_R8A7740)+= clk-r8a7740.o
+obj-$(CONFIG_ARCH_R8A7743)+= clk-rcar-gen2.o
+obj-$(CONFIG_ARCH_R8A7745)+= clk-rcar-gen2.o
obj-$(CONFIG_ARCH_R8A7778)+= clk-r8a7778.o
obj-$(CONFIG_ARCH_R8A7779)+= clk-r8a7779.o
obj-$(CONFIG_ARCH_R8A7790)+= clk-rcar-gen2.o
--- END ---

Yes. That is correct.


Ben Hutchings
Software Developer, Codethink Ltd.

Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.

Join to automatically receive all group messages.