Fix: we need to supply the direct channel number as a parameter (1, 2, 3, 4, ...) not OA_PWM_CH_FIRST ... (8, 9, 10, 11, ...) because that's what getPwmPeriod expects.

Author: aentinger

examples/opcua_server/opcua_server.ino

@@ -360,7 +360,7 @@ void setup()
               },
               [i, p](void) -> uint32_t
               {
-                return reinterpret_cast<AnalogExpansion *>(OptaController.getExpansionPtr(i))->getPwmPeriod(p);
+                return reinterpret_cast<AnalogExpansion *>(OptaController.getExpansionPtr(i))->getPwmPeriod(p - OA_PWM_CH_FIRST);
               });
             pwm_output_num++;
           }

src/io/pwm/PwmOutput.cpp

@@ -186,7 +186,7 @@ PwmOutput::onReadRequestPwmPeriod(
   UA_Variant_setScalar(&in_x_val_opcua_variant, &in_x_val_opcua_value, &UA_TYPES[UA_TYPES_UINT32]);
   UA_Server_writeValue(server, *node_id, in_x_val_opcua_variant);
   /* Some debug output. */
-  UA_LOG_INFO(UA_Log_Stdout, UA_LOGCATEGORY_SERVER, "%s: value = %d", __PRETTY_FUNCTION__, in_x_val);
+  UA_LOG_INFO(UA_Log_Stdout, UA_LOGCATEGORY_SERVER, "%s: pwm period = %d ms", __PRETTY_FUNCTION__, in_x_val);
 }
 
 void