Fix and cosmetics

radio/src/gui/480x272/model_setup.cpp

@@ -69,17 +69,13 @@ enum MenuModelSetupItems {
 #if defined(HARDWARE_INTERNAL_MODULE)
   ITEM_MODEL_SETUP_INTERNAL_MODULE_LABEL,
   ITEM_MODEL_SETUP_INTERNAL_MODULE_TYPE,
-#if defined(INTERNAL_MODULE_MULTI)
   ITEM_MODEL_SETUP_INTERNAL_MODULE_STATUS,
   ITEM_MODEL_SETUP_INTERNAL_MODULE_SYNCSTATUS,
-#endif
   ITEM_MODEL_SETUP_INTERNAL_MODULE_CHANNELS,
-  ITEM_MODEL_SETUP_INTERNAL_MODULE_NOT_ACCESS_BIND,
+  ITEM_MODEL_SETUP_INTERNAL_MODULE_NOT_ACCESS_RXNUM_BIND_RANGE,
   ITEM_MODEL_SETUP_INTERNAL_MODULE_PXX2_MODEL_NUM,
   ITEM_MODEL_SETUP_INTERNAL_MODULE_OPTIONS,
-#if defined(INTERNAL_MODULE_MULTI)
   ITEM_MODEL_SETUP_INTERNAL_MODULE_AUTOBIND,
-#endif
 #if defined(INTERNAL_MODULE_PXX1) && defined(EXTERNAL_ANTENNA)
   ITEM_MODEL_SETUP_INTERNAL_MODULE_ANTENNA,
 #endif
@@ -545,21 +541,21 @@ void onModelAntennaSwitchConfirm(const char * result)
 #if defined(HARDWARE_INTERNAL_MODULE)
   #define INTERNAL_MODULE_ROWS \
          LABEL(InternalModule), \
-           MODULE_TYPE_ROWS(INTERNAL_MODULE), \
-           MULTIMODULE_STATUS_ROWS(INTERNAL_MODULE) \
-           MODULE_CHANNELS_ROWS(INTERNAL_MODULE), \
-           IF_NOT_ACCESS_MODULE_RF(INTERNAL_MODULE, IF_INTERNAL_MODULE_ON(IF_INTERNAL_MODULE_ON(isModuleRxNumAvailable(INTERNAL_MODULE) ? (uint8_t)2 : (uint8_t)1))), \
-           IF_ACCESS_MODULE_RF(INTERNAL_MODULE, 0), \
-           MODULE_OPTION_ROW(INTERNAL_MODULE), \
-           MULTIMODULE_MODULE_ROWS(INTERNAL_MODULE)  \
-           EXTERNAL_ANTENNA_ROW \
-           MODULE_POWER_ROW(INTERNAL_MODULE), \
-           IF_INTERNAL_MODULE_ON(FAILSAFE_ROWS(INTERNAL_MODULE)), \
-           IF_ACCESS_MODULE_RF(INTERNAL_MODULE, 1), /* Range check and Register buttons */ \
-           IF_PXX2_MODULE(INTERNAL_MODULE, 0), /* Module options */ \
-           IF_ACCESS_MODULE_RF(INTERNAL_MODULE, 0), /* Receiver 1 */ \
-           IF_ACCESS_MODULE_RF(INTERNAL_MODULE, 0), /* Receiver 2 */  \
-           IF_ACCESS_MODULE_RF(INTERNAL_MODULE, 0), /* Receiver 3 */
+         MODULE_TYPE_ROWS(INTERNAL_MODULE),         /* ITEM_MODEL_SETUP_INTERNAL_MODULE_TYPE*/ \
+         MULTIMODULE_STATUS_ROWS(INTERNAL_MODULE)   /* ITEM_MODEL_SETUP_INTERNAL_MODULE_STATUS, ITEM_MODEL_SETUP_INTERNAL_MODULE_SYNCSTATUS */ \
+         MODULE_CHANNELS_ROWS(INTERNAL_MODULE),     /* ITEM_MODEL_SETUP_INTERNAL_MODULE_CHANNELS*/ \
+         IF_NOT_ACCESS_MODULE_RF(INTERNAL_MODULE, IF_INTERNAL_MODULE_ON(IF_INTERNAL_MODULE_ON(isModuleRxNumAvailable(INTERNAL_MODULE) ? (uint8_t)2 : (uint8_t)1))), /* *ITEM_MODEL_SETUP_INTERNAL_MODULE_NOT_ACCESS_RXNUM_BIND_RANGE */\
+         IF_ACCESS_MODULE_RF(INTERNAL_MODULE, 0),   /* ITEM_MODEL_SETUP_INTERNAL_MODULE_PXX2_MODEL_NUM*/ \
+         MODULE_OPTION_ROW(INTERNAL_MODULE),        /* ITEM_MODEL_SETUP_INTERNAL_MODULE_OPTIONS */ \
+         MULTIMODULE_MODULE_ROWS(INTERNAL_MODULE)   /* ITEM_MODEL_SETUP_INTERNAL_MODULE_AUTOBIND */  \
+         EXTERNAL_ANTENNA_ROW                       /* ITEM_MODEL_SETUP_INTERNAL_MODULE_ANTENNA */ \
+         MODULE_POWER_ROW(INTERNAL_MODULE),         /* ITEM_MODEL_SETUP_INTERNAL_MODULE_POWER */ \
+         IF_INTERNAL_MODULE_ON(FAILSAFE_ROWS(INTERNAL_MODULE)), /* ITEM_MODEL_SETUP_INTERNAL_MODULE_FAILSAFE */ \
+         IF_ACCESS_MODULE_RF(INTERNAL_MODULE, 1),   /* ITEM_MODEL_SETUP_INTERNAL_MODULE_PXX2_REGISTER_RANGE */ \
+         IF_PXX2_MODULE(INTERNAL_MODULE, 0),        /* ITEM_MODEL_SETUP_INTERNAL_MODULE_PXX2_OPTIONS*/ \
+         IF_ACCESS_MODULE_RF(INTERNAL_MODULE, 0),   /* ITEM_MODEL_SETUP_INTERNAL_MODULE_PXX2_RECEIVER_1 */ \
+         IF_ACCESS_MODULE_RF(INTERNAL_MODULE, 0),   /* ITEM_MODEL_SETUP_INTERNAL_MODULE_PXX2_RECEIVER_2 */ \
+         IF_ACCESS_MODULE_RF(INTERNAL_MODULE, 0),   /* ITEM_MODEL_SETUP_INTERNAL_MODULE_PXX2_RECEIVER_3 */
 #else
   #define INTERNAL_MODULE_ROWS
 #endif
@@ -1432,7 +1428,7 @@ bool menuModelSetup(event_t event)
       }
       break;
 #if defined(HARDWARE_INTERNAL_MODULE)
-      case ITEM_MODEL_SETUP_INTERNAL_MODULE_NOT_ACCESS_BIND:
+      case ITEM_MODEL_SETUP_INTERNAL_MODULE_NOT_ACCESS_RXNUM_BIND_RANGE:
 #endif
       case ITEM_MODEL_SETUP_EXTERNAL_MODULE_NOT_ACCESS_BIND:
       {
@@ -1665,7 +1661,7 @@ bool menuModelSetup(event_t event)
       break;
 
 #if defined(MULTIMODULE)
-#if defined(INTERNAL_MODULE_MULTI)
+#if defined(HARDWARE_INTERNAL_MODULE)
     case ITEM_MODEL_SETUP_INTERNAL_MODULE_AUTOBIND:
 #endif
     case ITEM_MODEL_SETUP_EXTERNAL_MODULE_AUTOBIND:
@@ -1675,7 +1671,8 @@ bool menuModelSetup(event_t event)
         lcdDrawText(MENUS_MARGIN_LEFT, y, STR_MULTI_AUTOBIND);
       g_model.moduleData[moduleIdx].multi.autoBindMode = editCheckBox(g_model.moduleData[moduleIdx].multi.autoBindMode, MODEL_SETUP_2ND_COLUMN, y, attr, event);
       break;
-#if defined(INTERNAL_MODULE_MULTI)
+
+#if defined(HARDWARE_INTERNAL_MODULE)
     case ITEM_MODEL_SETUP_INTERNAL_MODULE_STATUS:
 #endif
     case ITEM_MODEL_SETUP_EXTERNAL_MODULE_STATUS:
@@ -1687,7 +1684,8 @@ bool menuModelSetup(event_t event)
       lcdDrawText(MODEL_SETUP_2ND_COLUMN, y, statusText);
       break;
     }
-#if defined(INTERNAL_MODULE_MULTI)
+
+#if defined(HARDWARE_INTERNAL_MODULE)
     case ITEM_MODEL_SETUP_INTERNAL_MODULE_SYNCSTATUS:
 #endif
     case ITEM_MODEL_SETUP_EXTERNAL_MODULE_SYNCSTATUS:
@@ -1718,7 +1716,7 @@ bool menuModelSetup(event_t event)
           mod_cell->setModelName(g_model.header.name);
           break;
 #if defined(HARDWARE_INTERNAL_MODULE)
-        case ITEM_MODEL_SETUP_INTERNAL_MODULE_NOT_ACCESS_BIND:
+        case ITEM_MODEL_SETUP_INTERNAL_MODULE_NOT_ACCESS_RXNUM_BIND_RANGE:
           if (menuHorizontalPosition != 0)
             break;
         case ITEM_MODEL_SETUP_INTERNAL_MODULE_TYPE:

radio/src/pulses/modules_helpers.h

@@ -339,24 +339,24 @@ inline bool isModuleRxNumAvailable(uint8_t moduleIdx)
   return false;
 }
 
-inline bool isModuleFailsafeAvailable(uint8_t idx)
+inline bool isModuleFailsafeAvailable(uint8_t moduleIdx)
 {
 #if defined(PXX2)
-  if (isModuleISRM(idx))
+  if (isModuleISRM(moduleIdx))
     return true;
 #endif
 
-  if (isModuleXJT(idx))
-    return g_model.moduleData[idx].subType == MODULE_SUBTYPE_PXX1_ACCST_D16;
+  if (isModuleXJT(moduleIdx))
+    return g_model.moduleData[moduleIdx].subType == MODULE_SUBTYPE_PXX1_ACCST_D16;
 
 #if defined(MULTIMODULE)
-  if (isModuleMultimodule(idx)){
-    MultiModuleStatus& status = getMultiModuleStatus(idx);
+  if (isModuleMultimodule(moduleIdx)){
+    MultiModuleStatus& status = getMultiModuleStatus(moduleIdx);
     return status.isValid() && status.supportsFailsafe();
   }
 #endif
 
-  if (isModuleR9M(idx))
+  if (isModuleR9M(moduleIdx))
     return true;
 
   return false;

radio/src/targets/horus/CMakeLists.txt

@@ -110,8 +110,8 @@ if(INTERNAL_MODULE_PXX2)
 endif()
 
 if(INTERNAL_MODULE_MULTI)
-    add_definitions(-DHARDWARE_INTERNAL_MODULE)
-    add_definitions(-DINTERNAL_MODULE_MULTI)
+  add_definitions(-DHARDWARE_INTERNAL_MODULE)
+  add_definitions(-DINTERNAL_MODULE_MULTI)
 endif()
 
 include_directories(${RADIO_SRC_DIRECTORY}/fonts/480x272 gui/${GUI_DIR} gui/${GUI_DIR}/layouts)