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

* [tip: timers/clocksource] clocksource/drivers/stm32-lp: Add module owner
  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
  1 sibling, 0 replies; 3+ messages in thread
From: tip-bot2 for Daniel Lezcano @ 2025-07-25 10:31 UTC (permalink / raw)
  To: linux-tip-commits
  Cc: Daniel Lezcano, Ingo Molnar, Will McVicker, x86, linux-kernel

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

Commit-ID:     d3362af84d0c3aa02d3ebd0600c3a2640b2bd06a
Gitweb:        https://git.kernel.org/tip/d3362af84d0c3aa02d3ebd0600c3a2640b2bd06a
Author:        Daniel Lezcano <daniel.lezcano@linaro.org>
AuthorDate:    Mon, 02 Jun 2025 17:18:46 +02:00
Committer:     Ingo Molnar <mingo@kernel.org>
CommitterDate: Fri, 25 Jul 2025 11:43:10 +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 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>
Signed-off-by: Ingo Molnar <mingo@kernel.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

* [tip: timers/clocksource] clocksource/drivers/stm32-lp: Add module owner
  2025-06-02 15:18 [PATCH v1 2/7] " Daniel Lezcano
@ 2025-07-23  7:17 ` tip-bot2 for Daniel Lezcano
  2025-07-25 10:31 ` tip-bot2 for Daniel Lezcano
  1 sibling, 0 replies; 3+ messages in thread
From: tip-bot2 for Daniel Lezcano @ 2025-07-23  7:17 UTC (permalink / raw)
  To: linux-tip-commits; +Cc: Will McVicker, Daniel Lezcano, x86, linux-kernel

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

Commit-ID:     0625a7e5e34e952352bc81a305e5308c0a9cafcd
Gitweb:        https://git.kernel.org/tip/0625a7e5e34e952352bc81a305e5308c0a9cafcd
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: Thu, 10 Jul 2025 11:28:29 +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.

Reviewed-by: Will McVicker <willmcvicker@google.com>
Link: https://lore.kernel.org/r/20250602151853.1942521-3-daniel.lezcano@linaro.org
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 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

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®