| Message ID | 20230329-dt-cpu-header-cleanups-v1-11-581e2605fe47@kernel.org |
|---|---|
| State | Accepted |
| Commit | bd08b691615f342b3a10468ee8e182a3707041cc |
| Headers | show |
| Series | DT header disentangling, part 1 | expand |
On 29/03/2023 17:52, Rob Herring wrote: > Removing include of cpu.h from of_device.h (included by of_platform.h) > causes an error in ingenic-timer: > > drivers/clocksource/ingenic-timer.c: In function ‘ingenic_tcu_init’: > drivers/clocksource/ingenic-timer.c:338:15: error: implicit declaration of function ‘cpuhp_setup_state’ > > The of_platform.h header is not necessary either, so it and of_address.h > can be dropped. > > Signed-off-by: Rob Herring <robh@kernel.org> > --- > Please ack and I will take the series via the DT tree. > --- Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org>
diff --git a/drivers/clocksource/ingenic-timer.c b/drivers/clocksource/ingenic-timer.c index 24ed0f1f089b..089ce64b1c3f 100644 --- a/drivers/clocksource/ingenic-timer.c +++ b/drivers/clocksource/ingenic-timer.c @@ -9,13 +9,12 @@ #include <linux/clk.h> #include <linux/clockchips.h> #include <linux/clocksource.h> +#include <linux/cpuhotplug.h> #include <linux/interrupt.h> #include <linux/mfd/ingenic-tcu.h> #include <linux/mfd/syscon.h> #include <linux/of.h> -#include <linux/of_address.h> #include <linux/of_irq.h> -#include <linux/of_platform.h> #include <linux/overflow.h> #include <linux/platform_device.h> #include <linux/regmap.h>
Removing include of cpu.h from of_device.h (included by of_platform.h) causes an error in ingenic-timer: drivers/clocksource/ingenic-timer.c: In function ‘ingenic_tcu_init’: drivers/clocksource/ingenic-timer.c:338:15: error: implicit declaration of function ‘cpuhp_setup_state’ The of_platform.h header is not necessary either, so it and of_address.h can be dropped. Signed-off-by: Rob Herring <robh@kernel.org> --- Please ack and I will take the series via the DT tree. --- drivers/clocksource/ingenic-timer.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)