Skip to content

Conversation

kbidani
Copy link
Contributor

@kbidani kbidani commented Sep 1, 2025

The previous configuration used TIM3_REMAP1 for mapping TIM3_CH2 to PB5,
which is not supported according to the STM32F103 reference manual.
This commit updates the pinmux to use TIM3_REMAP2, enabling proper PWM output on PB5.

fixes: #94213
bug created to fix xml file: STMicroelectronics/STM32_open_pin_data#14

@kbidani kbidani changed the title stm32: dts: st: f1: fix pinmux to use TIM3_REMAP1 stm32: dts: st: f1: fix pinmux to use TIM3_REMAP2 Sep 1, 2025
@kbidani kbidani force-pushed the fix_pin_tim3_pwm_F1 branch from f81a655 to c3f5dd9 Compare September 1, 2025 13:53
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.

Change is OK but please document dts/README.rst so we can track this change.

@kbidani kbidani force-pushed the fix_pin_tim3_pwm_F1 branch from c3f5dd9 to 9541d2d Compare September 5, 2025 09:29
@erwango erwango requested a review from gautierg-st September 5, 2025 13:15
This commit updates the pinmux to use TIM3_REMAP2,
 enabling proper PWM output on PB5.

Signed-off-by: Khaoula Bidani <[email protected]>
@kbidani kbidani force-pushed the fix_pin_tim3_pwm_F1 branch from 9541d2d to 6f779e5 Compare September 15, 2025 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

hal_stm32: STM32F103VET6 TIM3 PWM channel remapping failure
2 participants