fix(hardwaretimer): resume if not running

fpistm · fpistm · commit a17fc44c7e55 · 2024-11-21T18:08:36.000+01:00
not only if previous mode is different.

Signed-off-by: Frederic Pillon &lt;frederic.pillon@st.com&gt;
diff --git a/libraries/SrcWrapper/src/HardwareTimer.cpp b/libraries/SrcWrapper/src/HardwareTimer.cpp
@@ -937,7 +937,7 @@ void HardwareTimer::setPWM(uint32_t channel, PinName pin, uint32_t frequency, ui
   if (CompareCallback) {
     attachInterrupt(channel, CompareCallback);
   }
-  if (previousMode != TIMER_OUTPUT_COMPARE_PWM1) {
+  if (!isRunning() || !isRunningChannel(channel) || (previousMode != TIMER_OUTPUT_COMPARE_PWM1)) {
     resume();
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -937,7 +937,7 @@ void HardwareTimer::setPWM(uint32_t channel, PinName pin, uint32_t frequency, ui`
`937`	`937`	`if (CompareCallback) {`
`938`	`938`	`attachInterrupt(channel, CompareCallback);`
`939`	`939`	`}`
`940`		`- if (previousMode != TIMER_OUTPUT_COMPARE_PWM1) {`
	`940`	`+ if (!isRunning() \|\| !isRunningChannel(channel) \|\| (previousMode != TIMER_OUTPUT_COMPARE_PWM1)) {`
`941`	`941`	`resume();`
`942`	`942`	`}`
`943`	`943`	`}`