2020-02-12 09:15:56

by Anson Huang

[permalink] [raw]
Subject: [PATCH] clk: imx7ulp: Include clk-provider.h instead of clk.h

The i.MX7ULP clock driver is provider, NOT consumer, so clk-provider.h
should be used instead of clk.h.

Signed-off-by: Anson Huang <[email protected]>
---
drivers/clk/imx/clk-imx7ulp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/imx/clk-imx7ulp.c b/drivers/clk/imx/clk-imx7ulp.c
index 0620d6c..3710aa0 100644
--- a/drivers/clk/imx/clk-imx7ulp.c
+++ b/drivers/clk/imx/clk-imx7ulp.c
@@ -8,7 +8,7 @@
*/

#include <dt-bindings/clock/imx7ulp-clock.h>
-#include <linux/clk.h>
+#include <linux/clk-provider.h>
#include <linux/err.h>
#include <linux/init.h>
#include <linux/io.h>
--
2.7.4


2020-02-12 20:58:55

by Stephen Boyd

[permalink] [raw]
Subject: Re: [PATCH] clk: imx7ulp: Include clk-provider.h instead of clk.h

Quoting Anson Huang (2020-02-12 01:09:43)
> The i.MX7ULP clock driver is provider, NOT consumer, so clk-provider.h
> should be used instead of clk.h.
>
> Signed-off-by: Anson Huang <[email protected]>
> ---

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

2020-02-17 06:34:34

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH] clk: imx7ulp: Include clk-provider.h instead of clk.h

On Wed, Feb 12, 2020 at 05:09:43PM +0800, Anson Huang wrote:
> The i.MX7ULP clock driver is provider, NOT consumer, so clk-provider.h
> should be used instead of clk.h.
>
> Signed-off-by: Anson Huang <[email protected]>

Applied, thanks.