Add button combination to set custom windingWidth with poti

control:
    - Add Button combination SET+PRESET1 to set winding width with potentiometer
    - Add case and display output in that state

config: move config macros from guide_stepper to common config

guide-stepper:
    - outsource config definitions
    - add functions to get and set variable winding width
    - add mutex

main/config.h

@@ -101,6 +101,24 @@
 //options affecting movement are currently defined in guide-stepper.cpp
 
 
+//---------------------------
+//------- cable guide -------
+//---------------------------
+// default axis coordinates the guide changes direction (winding width)
+#define GUIDE_MIN_MM 0  // TODO add feature so guide stays at zero for some steps (negative MIN_MM?), currently seems appropriate for even winding
+#define GUIDE_MAX_MM 95 // actual reel is 110, but currently guide turned out to stay at max position for too long
+
+// tolerance added to last stored position at previous shutdown.
+// When calibrating at startup the stepper moves for that sum to get track of zero position (ensure crashes into hardware limit for at least some time)
+#define AUTO_HOME_TRAVEL_ADD_TO_LAST_POS_MM 20
+#define MAX_TOTAL_AXIS_TRAVEL_MM 103 // max possible travel distance, needed as fallback for auto-home
+#define LAYER_THICKNESS_MM 5         // height of one cable layer on reel -> increase in radius every layer
+#define D_CABLE 6                    // determines winds per layer / guide speed
+#define D_REEL 160                   // start diameter of empty reel
+
+// max winding width that can be set using potentiometer (SET+PRESET1 buttons)
+#define MAX_WINDING_WIDTH_MM 100;
+
 
 //--------------------------
 //------ calibration -------

main/control.cpp

@@ -252,9 +252,38 @@ void task_control(void *pvParameter)
             buzzer.beep(3, 100, 60);
         }
 
-        //##### SET switch #####
+        //##### SET switch + Potentiometer #####
+        //## set winding-width (SET+PRESET1+POTI) ##
+        // set winding width (axis travel) with poti position
+        // when SET and PRESET1 button are pressed
+        if (SW_SET.state == true && SW_PRESET1.state == true) {
+            //read adc
+            potiRead = gpio_readAdc(ADC_CHANNEL_POTI); //0-4095
+            //scale to target length range
+            uint8_t windingWidthNew = (float)potiRead / 4095 * MAX_WINDING_WIDTH_MM;
+            //apply hysteresis and round to whole meters //TODO optimize this
+            if (windingWidthNew % 5 < 2) { //round down if remainder less than 2mm
+                ESP_LOGD(TAG, "Poti input = %d -> rounding down", windingWidthNew);
+                windingWidthNew = (windingWidthNew/5 ) * 5; //round down
+            } else if (windingWidthNew % 5 > 4 ) { //round up if remainder more than 4mm
+                ESP_LOGD(TAG, "Poti input = %d -> rounding up", windingWidthNew);
+                windingWidthNew = (windingWidthNew/5 + 1) * 5; //round up
+            } else {
+                ESP_LOGD(TAG, "Poti input = %d -> hysteresis", windingWidthNew);
+                windingWidthNew = lengthTarget;
+            }
+            //update target width and beep when effectively changed
+            if (windingWidthNew != guide_getWindingWidth()) {
+                //TODO update at button release only?
+                guide_setWindingWidth(windingWidthNew);
+                ESP_LOGW(TAG, "Changed winding width to %d mm", windingWidthNew);
+                buzzer.beep(1, 30, 10);
+            }
+        }
+
+        //## set target length (SET+POTI) ##
         //set target length to poti position when SET switch is pressed
-        if (SW_SET.state == true) {
+        else if (SW_SET.state == true) {
             //read adc
             potiRead = gpio_readAdc(ADC_CHANNEL_POTI); //0-4095
             //scale to target length range
@@ -475,6 +504,10 @@ void task_control(void *pvParameter)
         if (controlState == systemState_t::AUTO_CUT_WAITING) {
             displayTop.blinkStrings(" CUT 1N ", "        ", 70, 30);
         }
+        //setting winding width: blink info message
+        else if (SW_SET.state && SW_PRESET1.state){
+            displayTop.blinkStrings("  SET   ", " WIDTH  ", 400, 400);
+        }
         //otherwise show current position
         else {
             sprintf(buf_tmp, "1ST %5.4f", (float)lengthNow/1000);
@@ -489,17 +522,8 @@ void task_control(void *pvParameter)
         //--------------------------
         //run handle function
         displayBot.handle();
-        //setting target length: blink target length
-        if (SW_SET.state == true) {
-            sprintf(buf_tmp, "S0LL%5.3f", (float)lengthTarget/1000);
-            displayBot.blinkStrings(buf_tmp, "S0LL    ", 300, 100);
-        }
-        //manual state: blink "manual"
-        else if (controlState == systemState_t::MANUAL) {
-            displayBot.blinkStrings(" MANUAL ", buf_disp2, 400, 800);
-        }
         //notify that cutter is active
-        else if (cutter_isRunning()) {
+        if (cutter_isRunning()) {
             displayBot.blinkStrings("CUTTING]", "CUTTING[", 100, 100);
         }
         //show ms countdown to cut when pending
@@ -508,6 +532,20 @@ void task_control(void *pvParameter)
             //displayBot.showString(buf_disp2); //TODO:blink "erreicht" overrides this. for now using blink as workaround
             displayBot.blinkStrings(buf_disp2, buf_disp2, 100, 100);
         }
+        //manual state: blink "manual"
+        else if (controlState == systemState_t::MANUAL) {
+            displayBot.blinkStrings(" MANUAL ", buf_disp2, 400, 800);
+        }
+        //setting winding width: blink currently set windingWidth
+        else if (SW_SET.state && SW_PRESET1.state){
+            sprintf(buf_tmp, "  %03d mm", lengthNow/1000);
+            displayBot.blinkStrings(buf_tmp, "        ", 300, 100);
+        }
+        //setting target length: blink target length
+        else if (SW_SET.state == true) {
+            sprintf(buf_tmp, "S0LL%5.3f", (float)lengthTarget/1000);
+            displayBot.blinkStrings(buf_tmp, "S0LL    ", 300, 100);
+        }
         //otherwise show target length
         else {
             //sprintf(buf_disp2, "%06.1f cm", (float)lengthTarget/10); //cm

main/guide-stepper.cpp

@@ -6,6 +6,7 @@ extern "C"
 #include "esp_log.h"
 #include "driver/adc.h"
 #include "freertos/queue.h"
+#include "freertos/semphr.h"
 }
 
 #include "stepper.hpp"
@@ -22,36 +23,24 @@ extern "C"
 //---------------------
 //--- configuration ---
 //---------------------
-//also see config.h
+//also see config.h 
-//for pin definition
+//for pin definitions and guide parameters
 
+// configure testing modes
 #define STEPPER_TEST_TRAVEL 65     // mm
-
-#define MIN_MM 0 //TODO add feature so guide stays at zero for some steps (negative MIN_MM?), currently seems appropriate for even winding
-#define MAX_MM 95 //actual reel is 110, but currently guide turned out to stay at max position for too long
-#define POS_MAX_STEPS MAX_MM * STEPPER_STEPS_PER_MM
-#define POS_MIN_STEPS MIN_MM * STEPPER_STEPS_PER_MM
-
-//tolerance added to last stored position at previous shutdown.
-//When calibrating at startup the stepper moves for that sum to get track of zero position (ensure crashes into hardware limit for at least some time)
-#define AUTO_HOME_TRAVEL_ADD_TO_LAST_POS_MM 20
-#define MAX_TOTAL_AXIS_TRAVEL_MM 103 //max possible travel distance, needed for auto-home
-
 // speeds for testing with potentiometer (test task only)
 #define SPEED_MIN 2.0   // mm/s
 #define SPEED_MAX 70.0  // mm/s
 //note: actual speed is currently defined in config.h with STEPPER_SPEED_DEFAULT
-
-#define LAYER_THICKNESS_MM 5 //height of one cable layer on reel -> increase in radius
-#define D_CABLE 6
-#define D_REEL 160
-#define PI 3.14159
-
-
 //simulate encoder with reset button to test stepper ctl task
 //note STEPPER_TEST has to be defined as well
 //#define STEPPER_SIMULATE_ENCODER
 
+#define PI 3.14159
+//#define POS_MAX_STEPS GUIDE_MAX_MM * STEPPER_STEPS_PER_MM  //note replaced with posMaxSteps
+#define POS_MIN_STEPS GUIDE_MIN_MM * STEPPER_STEPS_PER_MM
+
+
 //----------------------
 //----- variables ------
 //----------------------
@@ -67,6 +56,12 @@ static int layerCount = 0;
 // queue for sending commands to task handling guide movement
 static QueueHandle_t queue_commandsGuideTask;
 
+// mutex to prevent multiple axis to config variables also accessed/modified by control task
+SemaphoreHandle_t configVariables_mutex = xSemaphoreCreateMutex();
+
+// configured winding width: position the axis returns again in steps
+static uint32_t posMaxSteps = GUIDE_MAX_MM * STEPPER_STEPS_PER_MM; //assign default width
+
 
 //----------------------
 //----- functions ------
@@ -82,6 +77,36 @@ int guide_getAxisPosSteps(){
 }
 
 
+//=============================
+//=== guide_setWindingWidth ===
+//=============================
+// set custom winding width (axis position the guide returns in mm)
+void guide_setWindingWidth(uint8_t maxPosMm)
+{
+    if (xSemaphoreTake(configVariables_mutex, portMAX_DELAY) == pdTRUE) // mutex to prevent multiple access by control and stepper-ctl task
+    {
+        posMaxSteps = maxPosMm * STEPPER_STEPS_PER_MM;
+        ESP_LOGI(TAG, "set winding width / max pos to %dmm", maxPosMm);
+        xSemaphoreGive(configVariables_mutex);
+    }
+}
+
+
+//=============================
+//=== guide_getWindingWidth ===
+//=============================
+// get currently configured winding width (axis position the guide returns in mm)
+uint8_t guide_getWindingWidth()
+{
+    if (xSemaphoreTake(configVariables_mutex, portMAX_DELAY) == pdTRUE) // mutex to prevent multiple access by control and stepper-ctl task
+    {
+        return posMaxSteps / STEPPER_STEPS_PER_MM;
+        xSemaphoreGive(configVariables_mutex);
+    }
+    return 0;
+}
+
+
 //==========================
 //==== guide_moveToZero ====
 //==========================
@@ -113,11 +138,12 @@ void travelSteps(int stepsTarget){
     while (stepsToGo != 0){
         //--- currently moving right ---
         if (currentAxisDirection == AXIS_MOVING_RIGHT){               //currently moving right
-            remaining = POS_MAX_STEPS - posNow;     //calc remaining distance fom current position to limit
+        if (xSemaphoreTake(configVariables_mutex, portMAX_DELAY) == pdTRUE) { //prevent multiple acces on posMaxSteps by control-task
+            remaining = posMaxSteps - posNow;     //calc remaining distance fom current position to limit
             if (stepsToGo > remaining){             //new distance will exceed limit
-                stepper_setTargetPosSteps(POS_MAX_STEPS);        //move to limit
+                stepper_setTargetPosSteps(posMaxSteps);        //move to limit
 				stepper_waitForStop(1000);
-                posNow = POS_MAX_STEPS;
+                posNow = posMaxSteps;
                 currentAxisDirection = AXIS_MOVING_LEFT;            //change current direction for next iteration
                 //increment/decrement layer count depending on current cable direction
                 layerCount += (stepsTarget > 0) - (stepsTarget < 0);
@@ -131,6 +157,8 @@ void travelSteps(int stepsTarget){
                 posNow += stepsToGo;
                 stepsToGo = 0;                      //finished, reset target length (could as well exit loop/break)
             }
+            xSemaphoreGive(configVariables_mutex);
+        }
         }
 
         //--- currently moving left ---

main/guide-stepper.hpp

@@ -15,6 +15,13 @@ void task_stepper_ctl(void *pvParameter);
 //tell stepper-control task to move cable guide to zero position
 void guide_moveToZero();
 
+
 // return local variable posNow that stores the current position of cable guide axis in steps
 // needed by shutdown to store last axis position in nvs
-int guide_getAxisPosSteps();
+int guide_getAxisPosSteps();
+
+// set custom winding width (axis position the guide returns in mm)
+void guide_setWindingWidth(uint8_t maxPosMm);
+
+// get currently configured winding width (axis position the guide returns in mm)
+uint8_t guide_getWindingWidth();