mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [tip: timers/clocksource] clocksource/drivers/stm32-lp: Add module owner
@ 2025-09-25 21:33 tip-bot2 for Daniel Lezcano
  0 siblings, 0 replies; 3+ messages in thread
From: tip-bot2 for Daniel Lezcano @ 2025-09-25 21:33 UTC (permalink / raw)
  To: linux-tip-commits; +Cc: Daniel Lezcano, Will McVicker, x86, linux-kernel

The following commit has been merged into the timers/clocksource branch of tip:

Commit-ID:     2cf51ab7f5c52f11dc12a11055f4b628121802d9
Gitweb:        https://git.kernel.org/tip/2cf51ab7f5c52f11dc12a11055f4b628121802d9
Author:        Daniel Lezcano <daniel.lezcano@linaro.org>
AuthorDate:    Mon, 02 Jun 2025 17:18:46 +02:00
Committer:     Daniel Lezcano <daniel.lezcano@linaro.org>
CommitterDate: Tue, 23 Sep 2025 10:51:21 +02:00

clocksource/drivers/stm32-lp: Add module owner

The conversion to modules requires a correct handling of the module
refcount in order to prevent to unload it if it is in use. That is
especially true with the clockevents where there is no function to
unregister them.

The core time framework correctly handles the module refcount with the
different clocksource and clockevents if the module owner is set.

Add the module owner to make sure the core framework will prevent
stupid things happening when the driver will be converted into a
module.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Reviewed-by: Will McVicker <willmcvicker@google.com>
Link: https://lore.kernel.org/r/20250602151853.1942521-3-daniel.lezcano@linaro.org
---
 drivers/clocksource/timer-stm32-lp.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clocksource/timer-stm32-lp.c b/drivers/clocksource/timer-stm32-lp.c
index 6e7944f..c2a699f 100644
--- a/drivers/clocksource/timer-stm32-lp.c
+++ b/drivers/clocksource/timer-stm32-lp.c
@@ -211,6 +211,7 @@ static void stm32_clkevent_lp_init(struct stm32_lp_private *priv,
 	priv->clkevt.rating = STM32_LP_RATING;
 	priv->clkevt.suspend = stm32_clkevent_lp_suspend;
 	priv->clkevt.resume = stm32_clkevent_lp_resume;
+	priv->clkevt.owner = THIS_MODULE;
 
 	clockevents_config_and_register(&priv->clkevt, rate, 0x1,
 					STM32_LPTIM_MAX_ARR);

^ permalink raw reply	[flat|nested] 3+ messages in thread
* [PATCH v1 2/7] clocksource/drivers/stm32-lp: Add module owner
@ 2025-06-02 15:18 Daniel Lezcano
  2025-07-23  7:17 ` [tip: timers/clocksource] " tip-bot2 for Daniel Lezcano
  2025-07-25 10:31 ` tip-bot2 for Daniel Lezcano
  0 siblings, 2 replies; 3+ messages in thread
From: Daniel Lezcano @ 2025-06-02 15:18 UTC (permalink / raw)
  To: daniel.lezcano, tglx
  Cc: Jim Cromie, Maxime Coquelin, Alexandre Torgue, Chen-Yu Tsai,
	Jernej Skrabec, Samuel Holland, Thierry Reding, Jonathan Hunter,
	Peter Zijlstra (Intel),
	Marco Elver, Nam Cao, linux-kernel, linux-stm32,
	linux-arm-kernel, linux-sunxi, linux-tegra, John Stulz,
	Will McVicker, Peter Griffin, Saravan Kanna

The conversion to modules requires a correct handling of the module
refcount in order to prevent to unload it if it is in use. That is
especially true with the clockevents where there is no function to
unregister them.

The core time framework correctly handles the module refcount with the
different clocksource and clockevents if the module owner is set.

Add the module owner to make sure the core framework will prevent
stupid things happening when the driver will be converted into a
module.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
 drivers/clocksource/timer-stm32-lp.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clocksource/timer-stm32-lp.c b/drivers/clocksource/timer-stm32-lp.c
index 928da2f6de69..cf1423ca00d0 100644
--- a/drivers/clocksource/timer-stm32-lp.c
+++ b/drivers/clocksource/timer-stm32-lp.c
@@ -159,6 +159,7 @@ static void stm32_clkevent_lp_init(struct stm32_lp_private *priv,
 	priv->clkevt.rating = STM32_LP_RATING;
 	priv->clkevt.suspend = stm32_clkevent_lp_suspend;
 	priv->clkevt.resume = stm32_clkevent_lp_resume;
+	priv->clkevt.owner = THIS_MODULE;
 
 	clockevents_config_and_register(&priv->clkevt, rate, 0x1,
 					STM32_LPTIM_MAX_ARR);
-- 
2.43.0


^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-09-25 21:34 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-09-25 21:33 [tip: timers/clocksource] clocksource/drivers/stm32-lp: Add module owner tip-bot2 for Daniel Lezcano
  -- strict thread matches above, loose matches on Subject: below --
2025-06-02 15:18 [PATCH v1 2/7] " Daniel Lezcano
2025-07-23  7:17 ` [tip: timers/clocksource] " tip-bot2 for Daniel Lezcano
2025-07-25 10:31 ` tip-bot2 for Daniel Lezcano

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®