2020-02-12 10:16:17

by Geert Uytterhoeven

Subject: [PATCH 6/7] MIPS: Pistachio: Replace <linux/clk-provider.h> by <linux/of_clk.h>

The Pistachio 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 <[email protected]>
---
arch/mips/pistachio/time.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/mips/pistachio/time.c b/arch/mips/pistachio/time.c
index 3c9235c7710a608f..de64751dec401990 100644
--- a/arch/mips/pistachio/time.c
+++ b/arch/mips/pistachio/time.c
@@ -6,10 +6,10 @@
*/

#include <linux/clk.h>
-#include <linux/clk-provider.h>
#include <linux/clocksource.h>
#include <linux/init.h>
#include <linux/of.h>
+#include <linux/of_clk.h>

#include <asm/mips-cps.h>
#include <asm/time.h>
--
2.17.1

2020-02-12 20:53:43

by Stephen Boyd

Subject: Re: [PATCH 6/7] MIPS: Pistachio: Replace <linux/clk-provider.h> by <linux/of_clk.h>

Quoting Geert Uytterhoeven (2020-02-12 02:15:43)
> The Pistachio 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 <[email protected]>
> ---

Reviewed-by: Stephen Boyd <[email protected]>

2020-02-12 22:19:05

by James Hartley

Subject: Re: [PATCH 6/7] MIPS: Pistachio: Replace <linux/clk-provider.h> by <linux/of_clk.h>

On 12/02/2020 10:15, Geert Uytterhoeven wrote:
> The Pistachio 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 <[email protected]>
> ---
> arch/mips/pistachio/time.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/mips/pistachio/time.c b/arch/mips/pistachio/time.c
> index 3c9235c7710a608f..de64751dec401990 100644
> --- a/arch/mips/pistachio/time.c
> +++ b/arch/mips/pistachio/time.c
> @@ -6,10 +6,10 @@
> */
>
> #include <linux/clk.h>
> -#include <linux/clk-provider.h>
> #include <linux/clocksource.h>
> #include <linux/init.h>
> #include <linux/of.h>
> +#include <linux/of_clk.h>
>
> #include <asm/mips-cps.h>
> #include <asm/time.h>

Thanks for the update Geert.

Acked-by: James Hartley <[email protected]>