Tidy handler for PWMWriteDutycycle

Author: brentru

src/components/pwm/controller.cpp

@@ -56,8 +56,49 @@ bool PWMController::Handle_PWM_Add(pb_istream_t *stream) {
   return true;
 }
 
+/**************************************************************************/
+/*!
+    @brief  Returns the index of the PWM hardware object that corresponds
+            to the given pin.
+    @param  pin The pin number to search for.
+    @return The index of the PWM hardware object, or -1 if not found.
+*/
+/**************************************************************************/
+int PWMController::GetPWMHardwareIdx(uint8_t pin) {
+  for (int i = 0; i < _active_pwm_pins; i++) {
+    if (_pwm_hardware[i]->GetPin() == pin) {
+      return i;
+    }
+  }
+  return -1;
+}
+
 bool PWMController::Handle_PWM_Write_DutyCycle(pb_istream_t *stream) {
-  return false;
+  if (!_pwm_model->DecodePWMWriteDutyCycle(stream)) {
+    WS_DEBUG_PRINTLN("[pwm] Error: Failed to decode PWMWriteDutyCycle message!");
+    return false;
+  }
+
+  wippersnapper_pwm_PWMWriteDutyCycle msg_write_duty_cycle = *_pwm_model->GetPWMWriteDutyCycleMsg();
+  uint8_t pin = atoi(msg_write_duty_cycle.pin + 1);
+
+  // Check if the pin is already attached
+  int pin_idx = GetPWMHardwareIdx(pin);
+  if (pin_idx == -1) {
+    WS_DEBUG_PRINTLN("[pwm] Error: pin not found!");
+    return false;
+  }
+
+  // Write the duty cycle to the pin
+  if (! _pwm_hardware[pin_idx]->WriteDutyCycle(msg_write_duty_cycle.duty_cycle)) {
+    WS_DEBUG_PRINTLN("[pwm] Error: Failed to write duty cycle!");
+    return false;
+  }
+  WS_DEBUG_PRINTLN("[pwm] Wrote duty cycle: ");
+  WS_DEBUG_PRINT(msg_write_duty_cycle.duty_cycle);
+  WS_DEBUG_PRINTLN(" to pin: ");
+  WS_DEBUG_PRINT(msg_write_duty_cycle.pin);
+  return true;
 }
 
 bool PWMController::Handle_PWM_Write_DutyCycle_Multi(pb_istream_t *stream) {

src/components/pwm/controller.h

@@ -39,6 +39,7 @@ class PWMController {
   bool Handle_PWM_Write_DutyCycle_Multi(pb_istream_t *stream);
   bool Handle_PWM_Write_Frequency(pb_istream_t *stream);
   bool Handle_PWM_Remove(pb_istream_t *stream);
+  int GetPWMHardwareIdx(uint8_t pin);
 
 private:
   PWMModel *_pwm_model;

src/components/pwm/hardware.cpp

@@ -157,6 +157,16 @@ void PWMHardware::WriteNoTone() {
   #endif
 }
 
+/**************************************************************************/
+/*!
+    @brief  Returns the pin number of the PWM pin
+    @return The logical pin number of the PWM pin
+*/
+/**************************************************************************/
+uint8_t PWMHardware::GetPin() {
+    return _pin;
+}
+
 // LEDC API Wrappers
 #ifdef ARDUINO_ARCH_ESP32
 /**************************************************************************/

src/components/pwm/hardware.h

@@ -34,6 +34,7 @@ class PWMHardware {
   bool WriteDutyCycle(uint32_t duty);
   uint32_t WriteTone(uint32_t freq);
   void WriteNoTone();
+  uint8_t GetPin();
 
   // Abstractions for LEDC API
   #ifdef ARDUINO_ARCH_ESP32

src/components/pwm/model.cpp

@@ -105,11 +105,26 @@ bool PWMModel::DecodePWMRemove(pb_istream_t *stream) {
 wippersnapper_pwm_PWMRemove *PWMModel::GetPWMRemoveMsg() {
 }
 
+/**************************************************************************/
+/*! 
+    @brief  Decodes a PWMWriteDutyCycle message from an input stream.
+    @param  stream  The stream to decode from.
+    @return true if successful, false otherwise.
+*/
+/**************************************************************************/
 bool PWMModel::DecodePWMWriteDutyCycle(pb_istream_t *stream) {
-  return false;
+  memset(&_msg_pwm_write_duty_cycle, 0, sizeof(_msg_pwm_write_duty_cycle));
+  return pb_decode(stream, wippersnapper_pwm_PWMWriteDutyCycle_fields, &_msg_pwm_write_duty_cycle);
 }
 
+/**************************************************************************/
+/*! 
+    @brief  Returns a pointer to the PWMWriteDutyCycle message.
+    @return Pointer to the PWMWriteDutyCycle message.
+*/
+/**************************************************************************/
 wippersnapper_pwm_PWMWriteDutyCycle *PWMModel::GetPWMWriteDutyCycleMsg() {
+  return &_msg_pwm_write_duty_cycle;
 }
 
 bool PWMModel::DecodePWMWriteDutyCycleMulti(pb_istream_t *stream) {