Re: [PATCH 4/7] MIPS: jz4740: Replace <linux/clk-provider.h> by <linux/of_clk.h>

[Paul Cercueil]

Hi,


Le mer., févr. 12, 2020 at 11:15, Geert Uytterhoeven 
<geert+renesas@xxxxxxxxx> a écrit :
> The Ingenic JZ4740 platform code is not a clock provider, and just 
> needs
> to call of_clk_init().
>
> Hence it can include <linux/of_clk.h> instead of 
> <linux/clk-provider.h>.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>

Looks good to me.
Reviewed-by: Paul Cercueil <paul@xxxxxxxxxxxxxxx>


> ---
>  arch/mips/jz4740/time.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/mips/jz4740/time.c b/arch/mips/jz4740/time.c
> index 5476899f0882a4b4..605a84a250bfb299 100644
> --- a/arch/mips/jz4740/time.c
> +++ b/arch/mips/jz4740/time.c
> @@ -4,8 +4,8 @@
>   *  JZ4740 platform time support
>   */
>
> -#include <linux/clk-provider.h>
>  #include <linux/clocksource.h>
> +#include <linux/of_clk.h>
>
>  #include <asm/mach-jz4740/timer.h>
>
> --
> 2.17.1