Skip to content

STM32: drop Kconfig support for LPTIM timer clock source selection #97546

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Oct 16, 2025
+14 −31
Merged

STM32: drop Kconfig support for LPTIM timer clock source selection #97546

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
a62bc03
drivers: timer: stm32_lptim: drop clock source configuration via Kconfig
mathieuchopstm Oct 14, 2025
8190805
release-notes: 4.3: add STM32 LPTIM clock source Kconfig removal notice
mathieuchopstm Oct 14, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions doc/releases/release-notes-4.3.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,8 @@ Removed APIs and options
* ``bt_le_set_auto_conn``
* ``CONFIG_BT_BUF_ACL_RX_COUNT``
* ``ok`` enum value has now been removed completely from ``base.yaml`` binding ``status`` property in devicetree.
* STM32 LPTIM clock source selection through Kconfig was removed. Device Tree must now be used instead.
Affected Kconfig symbols: ``CONFIG_STM32_LPTIM_CLOCK_LSI`` / ``CONFIG_STM32_LPTIM_CLOCK_LSI``

Deprecated APIs and options
===========================
Expand Down
20 changes: 8 additions & 12 deletions drivers/timer/Kconfig.stm32_lptim
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
# SPDX-License-Identifier: Apache-2.0

DT_CHOSEN_STDBY_TIMER := st,lptim-stdby-timer
DT_STM32_LPTIM_PATH := $(dt_nodelabel_path,stm32_lp_tick_source)

menuconfig STM32_LPTIM_TIMER
bool "STM32 Low Power Timer [EXPERIMENTAL]"
Expand All @@ -18,23 +19,18 @@ menuconfig STM32_LPTIM_TIMER

if STM32_LPTIM_TIMER

choice STM32_LPTIM_CLOCK
prompt "LPTIM clock value configuration"
help
This option is deprecated and configuration of LPTIM domain clock
using devicetree should be preferred.

# Invisible symbols exposing the selected LPTIM source to Kconfig
# NOTE: the values 2/3 correspond too STM32_SRC_LSE/STM32_SRC_LSI
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo:

Suggested change
# NOTE: the values 2/3 correspond too STM32_SRC_LSE/STM32_SRC_LSI
# NOTE: the values 2/3 correspond to STM32_SRC_LSE/STM32_SRC_LSI

# defined in include/zephyr/dt-bindings/clock/stm32_common_clocks.h
config STM32_LPTIM_CLOCK_LSI
bool "LSI"
def_bool "$(dt_node_ph_array_prop_int,$(DT_STM32_LPTIM_PATH),clocks,1,bus)" = 3
help
Use LSI as LPTIM clock
LSI used as LPTIM clock source

config STM32_LPTIM_CLOCK_LSE
bool "LSE"
def_bool "$(dt_node_ph_array_prop_int,$(DT_STM32_LPTIM_PATH),clocks,1,bus)" = 2
help
Use LSE as LPTIM clock

endchoice
LSE used as LPTIM clock source

config STM32_LPTIM_CLOCK
int
Expand Down
18 changes: 4 additions & 14 deletions drivers/timer/stm32_lptim_timer.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,23 +28,13 @@
#error Only one LPTIM instance should be enabled
#endif

#if DT_INST_NUM_CLOCKS(0) <= 1
#error "LPTIM source clock must be provided in Device Tree"
#endif

#define LPTIM (LPTIM_TypeDef *) DT_INST_REG_ADDR(0)

#if DT_INST_NUM_CLOCKS(0) == 1
#warning Kconfig for LPTIM source clock (LSI/LSE) is deprecated, use device tree.
static const struct stm32_pclken lptim_clk[] = {
STM32_CLOCK_INFO(0, DT_DRV_INST(0)),
/* Use Kconfig to configure source clocks fields */
/* Fortunately, values are consistent across enabled series */
#ifdef CONFIG_STM32_LPTIM_CLOCK_LSI
{.bus = STM32_SRC_LSI, .enr = LPTIM1_SEL(1)}
#else
{.bus = STM32_SRC_LSE, .enr = LPTIM1_SEL(3)}
#endif
};
#else
static const struct stm32_pclken lptim_clk[] = STM32_DT_INST_CLOCKS(0);
#endif

static const struct device *const clk_ctrl = DEVICE_DT_GET(STM32_CLOCK_CONTROL_NODE);

Expand Down
5 changes: 0 additions & 5 deletions soc/st/stm32/Kconfig.defconfig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,11 +72,6 @@ config SYS_CLOCK_TICKS_PER_SEC
default 250 if "$(dt_node_int_prop_int,$(DT_STM32_LPTIM_PATH),$(DT_ST_PRESCALER))" = 128
depends on STM32_LPTIM_TIMER && STM32_LPTIM_CLOCK_LSI

choice STM32_LPTIM_CLOCK
default STM32_LPTIM_CLOCK_LSE if "$(dt_node_ph_array_prop_int,$(DT_STM32_LPTIM_PATH),clocks,1,bus)" = 2
default STM32_LPTIM_CLOCK_LSI if "$(dt_node_ph_array_prop_int,$(DT_STM32_LPTIM_PATH),clocks,1,bus)" = 3
endchoice

config CLOCK_CONTROL_INIT_PRIORITY
default 1
depends on CLOCK_CONTROL
Expand Down