2020-11-10 15:53:22

by Geert Uytterhoeven

Subject: [PATCH resend v2] xtensa: Replace <linux/clk-provider.h> by <linux/of_clk.h>

The Xtensa time 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]>
Acked-by: Max Filippov <[email protected]>
---
v2:
- Add Reviewed-by, Acked-by.
---
arch/xtensa/kernel/time.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/xtensa/kernel/time.c b/arch/xtensa/kernel/time.c
index 77971fe4cc95ba2b..e8ceb1528608188c 100644
--- a/arch/xtensa/kernel/time.c
+++ b/arch/xtensa/kernel/time.c
@@ -13,7 +13,7 @@
*/

#include <linux/clk.h>
-#include <linux/clk-provider.h>
+#include <linux/of_clk.h>
#include <linux/errno.h>
#include <linux/sched.h>
#include <linux/time.h>
--
2.25.1

2020-12-07 22:05:39

by Stephen Boyd

Subject: Re: [PATCH resend v2] xtensa: Replace <linux/clk-provider.h> by <linux/of_clk.h>

Quoting Geert Uytterhoeven (2020-11-10 07:51:17)
> The Xtensa time 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]>
> Acked-by: Max Filippov <[email protected]>
> ---

Applied to clk-next