RC_Channel: added set_trim() API to SRV_Channels object

libraries/RC_Channel/RC_Channel.cpp

@@ -582,28 +582,28 @@ bool RC_Channel::in_trim_dz()
 
   For range outputs the returned value is from 0 to 1
  */
-float RC_Channel::get_radio_out_normalised(void) const
+float RC_Channel::get_radio_out_normalised(uint16_t pwm) const
 {
     if (_radio_max <= _radio_min) {
         return 0;
     }
     float ret;
     if (_type_out == RC_CHANNEL_TYPE_RANGE) {
-        if (_radio_out <= _radio_min) {
+        if (pwm <= _radio_min) {
             ret = 0;
-        } else if (_radio_out >= _radio_max) {
+        } else if (pwm >= _radio_max) {
             ret = 1;
         } else {
-            ret = (_radio_out - _radio_min) / float(_radio_max - _radio_min);
+            ret = (pwm - _radio_min) / float(_radio_max - _radio_min);
         }
         if (_reverse == -1) {
             ret = 1 - ret;
         }
     } else {
-        if (_radio_out < _radio_trim) {
+        if (pwm < _radio_trim) {
-            ret = -(_radio_trim - _radio_out) / float(_radio_trim - _radio_min);
+            ret = -(_radio_trim - pwm) / float(_radio_trim - _radio_min);
         } else {
-            ret = (_radio_out - _radio_trim) / float(_radio_max - _radio_trim);
+            ret = (pwm - _radio_trim) / float(_radio_max - _radio_trim);
         }
         if (_reverse == -1) {
             ret = -ret;

libraries/RC_Channel/RC_Channel.h

@@ -155,7 +155,7 @@ public:
     
     // get the current radio_out value as a floating point number
     // normalised so that 1.0 is full output
-    float      get_radio_out_normalised() const;
+    float      get_radio_out_normalised(uint16_t pwm) const;
     
     bool min_max_configured()
     {

libraries/RC_Channel/SRV_Channel.cpp

@@ -179,21 +179,14 @@ SRV_Channels::SRV_Channels(void)
 }
 
 
-/*
+// remap a PWM value from a channel in value
-  remap radio_out values for first 4 servos using SERVO* parameters, if enabled
+uint16_t SRV_Channels::remap_pwm(uint8_t i, uint16_t pwm) const
-  This should be called with cork enabled in hal.rcout
- */
-void SRV_Channels::remap_servo_output(void)
 {
-    if (!enable) {
+    const RC_Channel *ch = RC_Channel::rc_channel(i);
-        return;
-    }
-    for (uint8_t i=0; i<NUM_SERVO_RANGE_CHANNELS; i++) {
-        RC_Channel *ch = RC_Channel::rc_channel(i);
     if (ch == nullptr) {
-            continue;
+        return 0;
     }
-        float v = ch->get_radio_out_normalised();
+    float v = ch->get_radio_out_normalised(pwm);
     uint16_t radio_out;
     if (ch->get_type_out() == RC_CHANNEL_TYPE_RANGE) {
         if (reverse[i] == -1) {
@@ -210,7 +203,48 @@ void SRV_Channels::remap_servo_output(void)
             radio_out = servo_trim[i] + v * (servo_trim[i] - servo_min[i]);
         }
     }
+    return radio_out;
+}
+
+/*
+  remap radio_out values for first 4 servos using SERVO* parameters, if enabled
+  This should be called with cork enabled in hal.rcout
+ */
+void SRV_Channels::remap_servo_output(void)
+{
+    if (!enable) {
+        return;
+    }
+    for (uint8_t i=0; i<NUM_SERVO_RANGE_CHANNELS; i++) {
+        RC_Channel *ch = RC_Channel::rc_channel(i);
+        if (ch == nullptr) {
+            continue;
+        }
+        uint16_t radio_out = remap_pwm(i, ch->get_radio_out());
         ch->set_radio_out(radio_out);
         ch->output();
     }
 }
+
+
+/*
+  set trim values for output channels
+ */
+void SRV_Channels::set_trim(void)
+{
+    if (!enable) {
+        return;
+    }
+    for (uint8_t i=0; i<NUM_SERVO_RANGE_CHANNELS; i++) {
+        const RC_Channel *ch = RC_Channel::rc_channel(i);
+        if (ch == nullptr) {
+            continue;
+        }
+        if (ch->get_type_out() == RC_CHANNEL_TYPE_RANGE) {
+            // we don't trim range channels (like throttle)
+            continue;
+        }
+        uint16_t new_trim = remap_pwm(i, ch->get_radio_trim());
+        servo_trim[i].set_and_save(new_trim);
+    }
+}

libraries/RC_Channel/SRV_Channel.h

@@ -16,6 +16,7 @@
 
 #include <AP_Common/AP_Common.h>
 #include <AP_Param/AP_Param.h>
+#include "RC_Channel.h"
 
 #define NUM_SERVO_RANGE_CHANNELS 4
 
@@ -33,6 +34,9 @@ public:
     // servo ranges if enabled
     void remap_servo_output(void);
 
+    // set and save trim values from current RC input trim
+    void set_trim(void);
+    
 private:
     AP_Int8 enable;
 
@@ -43,4 +47,7 @@ private:
 
     // reversal, following convention that < 0 means reversed, >= 0 means normal
     AP_Int8 reverse[NUM_SERVO_RANGE_CHANNELS];
+
+    // remap a PWM value from a channel in value
+    uint16_t remap_pwm(uint8_t ch, uint16_t pwm) const;
 };