♻️ (drivers): [CorePwm] Implement _is_suspend logic

This avoid unnecessary init/deinit and critical section calls to the underlyind mbed::PwmOut driver
leka · Nov 28, 2022 · a90ae3e · a90ae3e
1 parent 91b5687
commit a90ae3e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/drivers/CorePwm/include/CorePwm.h b/drivers/CorePwm/include/CorePwm.h
@@ -23,6 +23,7 @@ class CorePwm : public interface::PwmOut
 
   private:
 	mbed::PwmOut _pwm;
+	bool _is_suspended = false;
 };
 
 }	// namespace leka
diff --git a/drivers/CorePwm/source/CorePwm.cpp b/drivers/CorePwm/source/CorePwm.cpp
@@ -18,10 +18,20 @@ void CorePwm::write(float value)
 
 void CorePwm::suspend()
 {
+	if (_is_suspended) {
+		return;
+	}
+
 	_pwm.suspend();
+	_is_suspended = true;
 }
 
 void CorePwm::resume()
 {
+	if (!_is_suspended) {
+		return;
+	}
+
 	_pwm.resume();
+	_is_suspended = false;
 }