Skip to content

Conversation

@tpambor
Copy link
Contributor

@tpambor tpambor commented Dec 11, 2025

Update hal_stm32 to include pin definitions for LPTIM peripheral.

@zephyrbot zephyrbot added the size: XS A PR changing only a single line of code label Dec 11, 2025
@github-actions
Copy link

github-actions bot commented Dec 11, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_stm32 zephyrproject-rtos/hal_stm32@9d05ebd zephyrproject-rtos/hal_stm32@90e120b (main) zephyrproject-rtos/hal_stm32@9d05ebdf..90e120b5

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-hal_stm32 DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Dec 11, 2025
@tpambor tpambor requested a review from erwango December 11, 2025 12:46
@erwango
Copy link
Member

erwango commented Dec 17, 2025

@tpambor For my information, is that the only missing piece to support PWM on LPTIM ? (Is the PWM driver otherwise compatible with use of lptim devices ?)

@tpambor
Copy link
Contributor Author

tpambor commented Dec 17, 2025

@tpambor For my information, is that the only missing piece to support PWM on LPTIM ? (Is the PWM driver otherwise compatible with use of lptim devices ?)

@erwango I wasn’t able to get it working using the workaround mentioned in #57316. I have implemented a new PWM driver for LPTIM. I plan to submit this driver for review soon. In the meantime, it would be great if the bindings could be merged.

Update hal_stm32 to include pin definitions for LPTIM peripheral.

Signed-off-by: Tim Pambor <tim.pambor@codewrights.de>
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Dec 19, 2025
@zephyrbot zephyrbot added the platform: STM32 ST Micro STM32 label Dec 19, 2025
Copy link
Member

@erwango erwango left a comment

Choose a reason for hiding this comment

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

Approved, but I feel this PR is a bit useless and #101205 could be used directly instead.

@sonarqubecloud
Copy link

@tpambor tpambor closed this Dec 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

manifest manifest-hal_stm32 platform: STM32 ST Micro STM32 size: XS A PR changing only a single line of code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants