MSP 1.54 support

locales/en/messages.json

@@ -2410,7 +2410,15 @@
     "motorsEnableControl": {
         "message": "<strong>I understand the risks</strong>, the propellers are removed - enable motor control and arming, and disable Runaway Takeoff Prevention."
     },
-
+    "motorConfigNotice": {
+        "message": "ESC/Motor Features are on the Configuration tab."
+    },
+    "motorPoleCount": {
+        "message": "Motor Poles"
+    },
+    "motorPoleCountHelp": {
+        "message": "The number of magnets on the motor bell."
+    },
     "sensorsInfo": {
         "message": "Keep in mind that using fast update periods and rendering multiple graphs at the same time is resource heavy and will burn your battery quicker if you use a laptop.<br />We recommend to only render graphs for sensors you are interested in while using reasonable update periods."
     },

manifest.json

@@ -2,7 +2,7 @@
     "manifest_version": 2,
     "minimum_chrome_version": "49",
     "version": "0.4.1",
-    "max_msp": "1.53.0",
+    "max_msp": "1.54.0",
     "COMMENT": "MAX_MSP required!!!!",
     "author": "Emuflight Team",
     "name": "Emuflight Configurator",

package.json

@@ -2,7 +2,7 @@
   "name": "emuflight-configurator",
   "description": "Crossplatform configuration tool for Emuflight flight control system.",
   "version": "0.4.1",
-  "max_msp": "1.53.0",
+  "max_msp": "1.54.0",
   "COMMENT": "MAX_MSP required!!!!",
   "main": "main.html",
   "chromium-args": "--disable-features=nw2",

src/js/msp/MSPHelper.js

@@ -790,6 +790,11 @@ MspHelper.prototype.process_data = function(dataHandler) {
                     }
                 }
                 CONFIG.mcuTypeId = 255;
+                //MSP 1.54
+                if (semver.gte(CONFIG.apiVersion, "1.54.0")) {
+                    CONFIG.gyroSampleRateHz = data.readU16();
+                }
+                //End MSP 1.54
                 break;
 
             case MSPCodes.MSP_NAME:
@@ -990,6 +995,11 @@ MspHelper.prototype.process_data = function(dataHandler) {
                         PID_ADVANCED_CONFIG.gyroUse32kHz = gyroUse32kHz;
                     }
                 }
+                //MSP 1.54 - insert here to avoid new unnecessary MSP code
+                if (semver.gte(CONFIG.apiVersion, "1.54.0")) {
+                    PID_ADVANCED_CONFIG.motorPoleCount = data.readU8();
+                }
+                //End MSP 1.54
                 break;
 
             case MSPCodes.MSP_FILTER_CONFIG:
@@ -1994,6 +2004,11 @@ MspHelper.prototype.crunch = function(code) {
                     buffer.push8(gyroUse32kHz);
                 }
             }
+            //MSP 1.54 - insert here to avoid new unnecessary MSP code
+            if (semver.gte(CONFIG.apiVersion, "1.54.0")) {
+                buffer.push8(PID_ADVANCED_CONFIG.motorPoleCount);
+            }
+            //End MSP 1.54
             break;
 
         case MSPCodes.MSP_SET_FILTER_CONFIG:

src/js/tabs/configuration.js

@@ -431,6 +431,14 @@ TABS.configuration.initialize = function (callback, scrollPosition) {
         $('input[name="unsyncedpwmfreq"]').val(PID_ADVANCED_CONFIG.motor_pwm_rate);
         $('input[name="digitalIdlePercent"]').val(PID_ADVANCED_CONFIG.digitalIdlePercent);
 
+        //MSP 1.54
+        if (semver.gte(CONFIG.apiVersion, "1.54.0")) {
+            $('input[name="motorPoleCount"]').val(PID_ADVANCED_CONFIG.motorPoleCount);
+            $('div.motorPoleCount').show();
+        } else {
+            $('div.motorPoleCount').hide();
+        }
+        //End MSP 1.54
 
         esc_protocol_e.val(PID_ADVANCED_CONFIG.fast_pwm_protocol + 1);
         esc_protocol_e.change(function () {
@@ -485,7 +493,6 @@ TABS.configuration.initialize = function (callback, scrollPosition) {
             if (semver.gte(CONFIG.apiVersion, "1.25.0")) {
                 while (denom <= 32) {
                      addDenomOption(gyro_select_e, denom, gyroBaseFreq);
-
                      denom ++;
                 }
             }
@@ -503,7 +510,11 @@ TABS.configuration.initialize = function (callback, scrollPosition) {
                 if ($(this).is(':checked')) {
                     gyroBaseFreq = 32;
                 } else {
-                    gyroBaseFreq = 8;
+                    if (semver.gte(CONFIG.apiVersion, "1.54.0") && CONFIG.gyroSampleRateHz) {
+                        gyroBaseFreq = CONFIG.gyroSampleRateHz / 1000;
+                    } else {
+                        gyroBaseFreq = 8;
+                    }
                 }
 
                 updateGyroDenom(gyroBaseFreq);
@@ -521,7 +532,12 @@ TABS.configuration.initialize = function (callback, scrollPosition) {
         gyro_select_e.change(function () {
             var originalPidDenom = pid_select_e.val();
 
-            var pidBaseFreq = 8;
+            let pidBaseFreq;
+            if (semver.gte(CONFIG.apiVersion, "1.54.0") && CONFIG.gyroSampleRateHz) {
+                pidBaseFreq = CONFIG.gyroSampleRateHz / 1000;
+            } else {
+                pidBaseFreq = 8;
+            }
             if (semver.gte(CONFIG.apiVersion, "1.25.0") && gyroUse32kHz_e.is(':checked')) {
                 pidBaseFreq = 32;
             }
@@ -1114,6 +1130,12 @@ TABS.configuration.initialize = function (callback, scrollPosition) {
             }
             //end MSP 1.51
 
+            //MSP 1.54
+            if (semver.gte(CONFIG.apiVersion, "1.54.0")) {
+                PID_ADVANCED_CONFIG.motorPoleCount = parseInt($('input[name="motorPoleCount"]').val());
+            }
+            //End MSP 1.54
+
             function save_serial_config() {
                 var next_callback = save_feature_config;
                 MSP.send_message(MSPCodes.MSP_SET_CF_SERIAL_CONFIG, mspHelper.crunch(MSPCodes.MSP_SET_CF_SERIAL_CONFIG), false, next_callback);

src/tabs/configuration.html

@@ -242,6 +242,17 @@
                                         <div class="helpicon cf_tip" i18n_title="configurationThrottleMinimumCommandHelp"></div>
                                     </label>
                                 </div>
+                                <!-- MSP 1.54 -->
+                                <div class="number motorPoleCount" id="motorPoleCount">
+                                    <label>
+                                        <div class="numberspacer">
+                                            <input type="number" name="motorPoleCount" min="1" max="255" />
+                                        </div>
+                                        <span i18n="motorPoleCount"></span>
+                                        <div class="helpicon cf_tip" i18n_title="motorPoleCountHelp"></div>
+                                    </label>
+                                </div>
+                                <!-- End MSP 1.54 -->
                             </div>
                         </div>
                     </div>

src/tabs/motors.html

@@ -4,6 +4,13 @@
         <div class="cf_doc_version_bt">
             <a id="button-documentation" href="https://github.com/emuflight/EmuFlight/releases" target="_blank"></a>
         </div>
+
+        <div class="note spacebottom">
+            <div class="note_spacer">
+                <p i18n="motorConfigNotice"></p>
+            </div>
+        </div>
+
         <div class="gui_box grey" style="margin-bottom: 15px;">
             <div class="wrapper modelAndGraph">
                 <div class="mixerPreview">
@@ -171,7 +178,7 @@
                         <label><input id="motorsEnableTestMode" type="checkbox" class="togglesmall"/><span
                             class="motorsEnableTestMode" i18n="motorsEnableControl"></span></label>
                     </div>
-                    <div class="cler-both"></div>
+                    <div class="clear-both"></div>
                 </div>
             </div>
         </div>