5.5.33 config_servo

EXRAILMacros.h

@@ -95,21 +95,29 @@
 #define STEALTH_GLOBAL(code...) code
 #include "myAutomation.h"
 
-// Pass 1h Implements HAL macro by creating exrailHalSetup function
+// Pass 1h Implements HAL macro by creating exrailHalSetup1 function
 // Also allows creating EXTurntable object
 #include "EXRAIL2MacroReset.h"
 #undef HAL
 #define HAL(haltype,params...)  haltype::create(params);
 #undef HAL_IGNORE_DEFAULTS
 #define HAL_IGNORE_DEFAULTS ignore_defaults=true;
+bool exrailHalSetup1() {
+   bool ignore_defaults=false;
+   #include "myAutomation.h"
+   return ignore_defaults;
+}
+
+// Pass 1s Implements servos by creating exrailHalSetup2
+// TODO Turnout and turntable creation should be moved to here instead of 
+// the first pass from the opcode table. 
+#include "EXRAIL2MacroReset.h"
 #undef JMRI_SENSOR
 #define JMRI_SENSOR(vpin,count...) Sensor::createMultiple(vpin,##count);
 #undef  CONFIGURE_SERVO
 #define CONFIGURE_SERVO(vpin,pos1,pos2,profile) IODevice::configureServo(vpin,pos1,pos2,PCA9685::profile);
-bool exrailHalSetup() {
+void exrailHalSetup2() {
-   bool ignore_defaults=false;
    #include "myAutomation.h"
-   return ignore_defaults;
 }
 
 // Pass 1c detect compile time featurtes

IODevice.cpp

@@ -33,7 +33,9 @@
 
 // Link to halSetup function.  If not defined, the function reference will be NULL.
 extern __attribute__((weak)) void halSetup();
-extern __attribute__((weak)) bool exrailHalSetup();
+extern __attribute__((weak)) bool exrailHalSetup1();
+extern __attribute__((weak)) bool exrailHalSetup2();
+
 
 //==================================================================================================================
 // Static methods
@@ -59,11 +61,19 @@ void IODevice::begin() {
   if (halSetup)
     halSetup();
 
-  // include any HAL devices defined in exrail. 
+  // Include any HAL devices defined in exrail.
+  // The first pass call only creates HAL devices, 
+  // the second pass will apply servo settings etc which can only be 
+  // done after all devices (including the defaults) are created.
+  // If exrailHalSetup1 is not defined, then it will be NULL and the call
+  // will be ignored.
+  // If it returns true, then the default HAL devices will not be created.
+
   bool ignoreDefaults=false;
-  if (exrailHalSetup)
+  if (exrailHalSetup1)
-    ignoreDefaults=exrailHalSetup();
+    ignoreDefaults=exrailHalSetup1();
-  if (ignoreDefaults) return;
+  
+  if (!ignoreDefaults) {
   
     // Predefine two PCA9685 modules 0x40-0x41 if no conflicts
     // Allocates 32 pins 100-131
@@ -87,6 +97,12 @@ void IODevice::begin() {
     } 
   }
 
+  // apply any second pass HAL setup from EXRAIL.
+  // This will typically set up servo profiles, or create turnouts.
+  if (exrailHalSetup2)
+    exrailHalSetup2();
+}
+
 // reset() function to reinitialise all devices
 void IODevice::reset() {
   unsigned long currentMicros = micros();

version.h

@@ -3,7 +3,8 @@
 
 #include "StringFormatter.h"
 
-#define VERSION "5.5.32"
+#define VERSION "5.5.33"
+// 5.5.33 - Fix CONFIG_SERVO when default PCA9685 definition used.
 // 5.5.32 - Feature: Enable sniffer on CSB-1
 // 5.5.31 - <JL screen startRow> track status command
 //        - myTrackStatus.example.h added