2020-02-18 11:10:11

by Anson Huang

Subject: [PATCH 1/3] clk: imx8mp: Include slab.h instead of clkdev.h

slab.h is necessary and included indirectly by clkdev.h,
actually, there is nothing in use from clkdev.h, so just
include slab.h instead of clkdev.h.

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

diff --git a/drivers/clk/imx/clk-imx8mp.c b/drivers/clk/imx/clk-imx8mp.c
index a16af4f..3adc8aa 100644
--- a/drivers/clk/imx/clk-imx8mp.c
+++ b/drivers/clk/imx/clk-imx8mp.c
@@ -4,13 +4,13 @@
*/

#include <dt-bindings/clock/imx8mp-clock.h>
-#include <linux/clkdev.h>
#include <linux/clk-provider.h>
#include <linux/err.h>
#include <linux/io.h>
#include <linux/module.h>
#include <linux/of_address.h>
#include <linux/platform_device.h>
+#include <linux/slab.h>
#include <linux/types.h>

#include "clk.h"
--
2.7.4

2020-02-18 17:03:43

by Stephen Boyd

Subject: Re: [PATCH 1/3] clk: imx8mp: Include slab.h instead of clkdev.h

Quoting Anson Huang (2020-02-18 03:03:24)
> slab.h is necessary and included indirectly by clkdev.h,
> actually, there is nothing in use from clkdev.h, so just
> include slab.h instead of clkdev.h.
>
> Signed-off-by: Anson Huang <[email protected]>
> ---

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

2020-02-19 02:21:52

by Shawn Guo

Subject: Re: [PATCH 1/3] clk: imx8mp: Include slab.h instead of clkdev.h

On Tue, Feb 18, 2020 at 07:03:24PM +0800, Anson Huang wrote:
> slab.h is necessary and included indirectly by clkdev.h,
> actually, there is nothing in use from clkdev.h, so just
> include slab.h instead of clkdev.h.
>
> Signed-off-by: Anson Huang <[email protected]>

Applied all, thanks.