[PATCH v4 02/10] clk: renesas: prepare for non-RCAR clock drivers

[Marek Vasut]

On 3/8/23 21:26, Ralph Siemsen wrote:
> Allow CONFIG_CLK_RENESAS to be set without bringing in RCAR-GEN2/3 code.
> 
> CONFIG_RENESAS is used in drivers/clk/Makefile to control recursion into
> the drivers/clk/renesas subdirectory. It also controls compilation of
> renesas-cpg-mssr.c support code for the RCAR-GEN2 and RCAR-GEN3 devices.
> 
> The support code contains platform specific hardware access (TMU_BASE),
> and it is not needed for other Renesas devices such as RZ/N1. Therefore,
> alter Makefile to build renesas-cpg-mssr.c only for RCAR-GEN2/3.
> 
> Signed-off-by: Ralph Siemsen <ralph.siemsen at linaro.org>
> Reviewed-by: Sean Anderson <seanga2 at gmail.com>
> ---
> 
> (no changes since v3)
> 
> Changes in v3:
> - added tags
> 
>   drivers/clk/renesas/Kconfig  | 2 +-
>   drivers/clk/renesas/Makefile | 5 ++---
>   2 files changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/clk/renesas/Kconfig b/drivers/clk/renesas/Kconfig
> index 1686410d6d..6788415eed 100644
> --- a/drivers/clk/renesas/Kconfig
> +++ b/drivers/clk/renesas/Kconfig
> @@ -1,6 +1,6 @@
>   config CLK_RENESAS
>   	bool "Renesas clock drivers"
> -	depends on CLK && ARCH_RMOBILE
> +	depends on CLK && (ARCH_RMOBILE || ARCH_RZN1)
>   	help
>   	  Enable support for clock present on Renesas RCar SoCs.
>   
> diff --git a/drivers/clk/renesas/Makefile b/drivers/clk/renesas/Makefile
> index 8f82a7aa3e..a0d8c10bdb 100644
> --- a/drivers/clk/renesas/Makefile
> +++ b/drivers/clk/renesas/Makefile
> @@ -1,6 +1,5 @@
> -obj-$(CONFIG_CLK_RENESAS) += renesas-cpg-mssr.o
>   obj-$(CONFIG_CLK_RCAR_CPG_LIB) += rcar-cpg-lib.o
> -obj-$(CONFIG_CLK_RCAR_GEN2) += clk-rcar-gen2.o
> +obj-$(CONFIG_CLK_RCAR_GEN2) += clk-rcar-gen2.o renesas-cpg-mssr.o

Why not move the renesas-cpg-mssr.o into obj-$(CONFIG_CLK_RCAR_CPG_LIB) 
line instead ? I suspect RZN1 is not using the CPG lib anyway ?