diff mbox series

clocksource/drivers/timer-imx-gpt: Remove non-DT function

Message ID 20230307124313.708255-1-festevam@denx.de (mailing list archive)
State New, archived
Headers show
Series clocksource/drivers/timer-imx-gpt: Remove non-DT function | expand

Commit Message

Fabio Estevam March 7, 2023, 12:43 p.m. UTC
mxc_timer_init() was originally only used by non-DT i.MX platforms.

i.MX has already been converted to be a DT-only platform.

Remove the unused mxc_timer_init() function.

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 drivers/clocksource/timer-imx-gpt.c | 19 -------------------
 include/soc/imx/timer.h             |  7 -------
 2 files changed, 26 deletions(-)

Comments

Uwe Kleine-König March 28, 2023, 10:33 a.m. UTC | #1
On Tue, Mar 07, 2023 at 09:43:13AM -0300, Fabio Estevam wrote:
> mxc_timer_init() was originally only used by non-DT i.MX platforms.
> 
> i.MX has already been converted to be a DT-only platform.
> 
> Remove the unused mxc_timer_init() function.
> 
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

Actually I created the same patch and have another patch that depends on
this one. See
https://lore.kernel.org/linux-arm-kernel/20230328100531.879485-1-u.kleine-koenig@pengutronix.de/#t
.

Best regards
Uwe
Daniel Lezcano April 6, 2023, 1:42 p.m. UTC | #2
On 07/03/2023 13:43, Fabio Estevam wrote:
> mxc_timer_init() was originally only used by non-DT i.MX platforms.
> 
> i.MX has already been converted to be a DT-only platform.
> 
> Remove the unused mxc_timer_init() function.
> 
> Signed-off-by: Fabio Estevam <festevam@denx.de>
> ---

Applied, thanks
diff mbox series

Patch

diff --git a/drivers/clocksource/timer-imx-gpt.c b/drivers/clocksource/timer-imx-gpt.c
index 7b2c70f2f353..ca3e4cbc80c6 100644
--- a/drivers/clocksource/timer-imx-gpt.c
+++ b/drivers/clocksource/timer-imx-gpt.c
@@ -420,25 +420,6 @@  static int __init _mxc_timer_init(struct imx_timer *imxtm)
 	return mxc_clockevent_init(imxtm);
 }
 
-void __init mxc_timer_init(unsigned long pbase, int irq, enum imx_gpt_type type)
-{
-	struct imx_timer *imxtm;
-
-	imxtm = kzalloc(sizeof(*imxtm), GFP_KERNEL);
-	BUG_ON(!imxtm);
-
-	imxtm->clk_per = clk_get_sys("imx-gpt.0", "per");
-	imxtm->clk_ipg = clk_get_sys("imx-gpt.0", "ipg");
-
-	imxtm->base = ioremap(pbase, SZ_4K);
-	BUG_ON(!imxtm->base);
-
-	imxtm->type = type;
-	imxtm->irq = irq;
-
-	_mxc_timer_init(imxtm);
-}
-
 static int __init mxc_timer_init_dt(struct device_node *np,  enum imx_gpt_type type)
 {
 	struct imx_timer *imxtm;
diff --git a/include/soc/imx/timer.h b/include/soc/imx/timer.h
index b888d5076b4d..25f29c6bbd0b 100644
--- a/include/soc/imx/timer.h
+++ b/include/soc/imx/timer.h
@@ -13,11 +13,4 @@  enum imx_gpt_type {
 	GPT_TYPE_IMX6DL,	/* i.MX6DL/SX/SL */
 };
 
-/*
- * This is a stop-gap solution for clock drivers like imx1/imx21 which call
- * mxc_timer_init() to initialize timer for non-DT boot.  It can be removed
- * when these legacy non-DT support is converted or dropped.
- */
-void mxc_timer_init(unsigned long pbase, int irq, enum imx_gpt_type type);
-
 #endif  /* __SOC_IMX_TIMER_H__ */