RFC Dynamic Idle

Author: haslinghuis

locales/en/messages.json

@@ -1281,6 +1281,9 @@
     "configurationDigitalIdlePercent": {
         "message": "Motor Idle Throttle Value [percent]"
     },
+    "configurationDigitalIdlePercentDisabled": {
+        "message": "Motor Idle Throttle Value [percent] is disabled because Dynamic Idle is enabled in PID tuning tab"
+    },
     "configurationDigitalIdlePercentHelp": {
         "message": "The 'DShot idle' value is the percent of maximum throttle that is sent to the ESCs when the throttle at minimum stick position and the craft is armed. Increase it to gain more idle speed and avoid desyncs.  Too high and the craft feels floaty."
     },

src/js/tabs/motors.js

@@ -61,6 +61,7 @@ TABS.motors.initialize = function (callback) {
     }
 
     MSP.promise(MSPCodes.MSP_STATUS)
+    .then(() => MSP.promise(MSPCodes.MSP_PID_ADVANCED))
     .then(() => MSP.promise(MSPCodes.MSP_FEATURE_CONFIG))
     .then(() => MSP.promise(MSPCodes.MSP_MIXER_CONFIG))
     .then(() => FC.MOTOR_CONFIG.use_dshot_telemetry || FC.MOTOR_CONFIG.use_esc_sensor ? MSP.promise(MSPCodes.MSP_MOTOR_TELEMETRY) : true)
@@ -735,6 +736,15 @@ TABS.motors.initialize = function (callback) {
             divUnsyncedPWMFreq.toggle(protocolConfigured && !digitalProtocol);
 
             $('div.digitalIdlePercent').toggle(protocolConfigured && digitalProtocol);
+
+            $('input[name="digitalIdlePercent"]').prop('disabled', protocolConfigured && digitalProtocol && FC.ADVANCED_TUNING.idleMinRpm && FC.MOTOR_CONFIG.use_dshot_telemetry);
+
+            if (FC.ADVANCED_TUNING.idleMinRpm && FC.MOTOR_CONFIG.use_dshot_telemetry) {
+                $('span.digitalIdlePercentDisabled').text(i18n.getMessage('configurationDigitalIdlePercentDisabled'));
+            } else {
+                $('span.digitalIdlePercentDisabled').text(i18n.getMessage('configurationDigitalIdlePercent'));
+            }
+
             $('.escSensor').toggle(protocolConfigured && digitalProtocol);
 
             $('div.checkboxDshotBidir').toggle(protocolConfigured && semver.gte(FC.CONFIG.apiVersion, API_VERSION_1_42) && digitalProtocol);

src/js/tabs/pid_tuning.js

@@ -457,6 +457,7 @@ TABS.pid_tuning.initialize = function (callback) {
             $('.pid_tuning input[name="motorLimit"]').val(FC.ADVANCED_TUNING.motorOutputLimit);
             $('.pid_tuning input[name="cellCount"]').val(FC.ADVANCED_TUNING.autoProfileCellCount);
             $('input[name="idleMinRpm-number"]').val(FC.ADVANCED_TUNING.idleMinRpm);
+            $('.idleMinRpm').toggle(FC.MOTOR_CONFIG.use_dshot_telemetry);
         } else {
             $('.motorOutputLimit').hide();
             $('.idleMinRpm').hide();

src/tabs/motors.html

@@ -105,9 +105,9 @@
                                         <div class="numberspacer">
                                             <input type="number" name="digitalIdlePercent" min="0.00" max="20.00" step="0.01"/>
                                         </div>
-                                        <span i18n="configurationDigitalIdlePercent"></span>
+                                        <span class="digitalIdlePercentDisabled"></span>
+                                        <div class="helpicon cf_tip" i18n_title="configurationDigitalIdlePercentHelp"></div>
                                     </label>
-                                    <div class="helpicon cf_tip" i18n_title="configurationDigitalIdlePercentHelp"></div>
                                 </div>
                                 <div class="number minthrottle">
                                     <label>