refactor: move Log2Representation to log2/ subdirectory

Hide internal log2 math implementation from public API by moving
Log2Representation.h/.cpp and Log2RepresentationConst.h to src/log2/.
Also fix circular include between Log2Representation.h and Const.h.

Author: gin66

CMakeLists.txt

@@ -1,6 +1,6 @@
 idf_component_register(
-    SRC_DIRS "src" "src/pd_esp32" "src/fas_ramp"
-    INCLUDE_DIRS "src" "src/pd_esp32" "src/fas_ramp"
+    SRC_DIRS "src" "src/pd_esp32" "src/fas_ramp" "src/log2"
+    INCLUDE_DIRS "src" "src/pd_esp32" "src/fas_ramp" "src/log2"
     REQUIRES 
         esp_driver_gpio
         esp_driver_rmt

extras/tests/pc_based/Makefile

@@ -18,7 +18,7 @@ ramp_helper: ramp_helper.o $(LIB_O)
 ramp_helper.o: ramp_helper.cpp $(SRC_LIB_H) stubs.h
 	g++ -c $(CXXFLAGS) -o $@ $<
 
-LIB_H=FastAccelStepper.h Log2Representation.h StepperISR.h \
+LIB_H=FastAccelStepper.h log2/Log2Representation.h log2/Log2RepresentationConst.h StepperISR.h \
 	  fas_ramp/RampGenerator.h fas_ramp/RampControl.h fas_ramp/RampCalculator.h \
 	  fas_arch/common.h fas_queue/base.h pd_test/test_queue.h
 LIB_O=FastAccelStepper.o Log2Representation.o StepperISR_test.o \
@@ -33,15 +33,15 @@ test_%.o: test_%.cpp $(SRC_LIB_H) RampChecker.h stubs.h
 	g++ -c $(CXXFLAGS) -o $@ $<
 
 log2_test: log2_test.o Log2Representation.o
-log2_test.o: log2_test.cpp $(PRJ_ROOT)/src/Log2Representation.h stubs.h test_03.h
+log2_test.o: log2_test.cpp $(PRJ_ROOT)/src/log2/Log2Representation.h stubs.h test_03.h
 
 rmc_test: rmc_test.o Log2Representation.o RampCalculator.o
-rmc_test.o: rmc_test.cpp $(PRJ_ROOT)/src/Log2Representation.h $(PRJ_ROOT)/src/fas_ramp/RampCalculator.h stubs.h test_03.h
+rmc_test.o: rmc_test.cpp $(PRJ_ROOT)/src/log2/Log2Representation.h $(PRJ_ROOT)/src/fas_ramp/RampCalculator.h stubs.h test_03.h
 
 FastAccelStepper.o: $(PRJ_ROOT)/src/FastAccelStepper.cpp $(SRC_LIB_H)
 	$(COMPILE.cpp) $< -o $@
 
-Log2Representation.o: $(PRJ_ROOT)/src/Log2Representation.cpp $(PRJ_ROOT)/src/Log2Representation.h
+Log2Representation.o: $(PRJ_ROOT)/src/log2/Log2Representation.cpp $(PRJ_ROOT)/src/log2/Log2Representation.h
 	$(COMPILE.cpp) $< -o $@
 
 RampGenerator.o: $(PRJ_ROOT)/src/fas_ramp/RampGenerator.cpp $(SRC_LIB_H)

extras/tests/pc_based/log2_test.cpp

@@ -1,6 +1,6 @@
 #include <stdint.h>
 
-#include "Log2Representation.h"
+#include "log2/Log2Representation.h"
 
 //
 // This file can be renamed to a .ino and compiled as sketch to be run on the

extras/tests/pc_based/rmc_test.cpp

@@ -3,6 +3,7 @@
 #include <stdio.h>
 #include <stdlib.h>
 
+#include "log2/Log2Representation.h"
 #include "fas_ramp/RampCalculator.h"
 
 // Not a real test case

src/fas_ramp/RampCalculator.h

@@ -3,7 +3,7 @@
 
 #include <stdint.h>
 
-#include "Log2Representation.h"
+#include "log2/Log2Representation.h"
 #include "fas_arch/common.h"
 
 #if (TICKS_PER_S == 16000000L)

src/Log2Representation.cpp src/log2/Log2Representation.cppsrc/Log2Representation.cpp renamed to src/log2/Log2Representation.cpp

@@ -5,7 +5,7 @@
 #define PROGMEM
 #define pgm_read_byte_near(x) (*(x))
 #endif
-#include "Log2Representation.h"
+#include "log2/Log2Representation.h"
 #ifdef TEST
 #include <stdio.h>
 #endif

src/Log2Representation.h src/log2/Log2Representation.hsrc/Log2Representation.h renamed to src/log2/Log2Representation.h

@@ -2,7 +2,7 @@
 #define LOG2REPRESENTATION_H
 #include <stdint.h>
 
-#include <Log2RepresentationConst.h>
+#include "log2/Log2RepresentationConst.h"
 
 typedef int16_t pmf_logarithmic;
 

src/Log2RepresentationConst.h src/log2/Log2RepresentationConst.hsrc/Log2RepresentationConst.h renamed to src/log2/Log2RepresentationConst.h

@@ -3,7 +3,7 @@
 #ifndef LOG2REPRESENTATIONCONST_H
 #define LOG2REPRESENTATIONCONST_H
 
-#include <Log2Representation.h>
+// pmf_logarithmic must be defined before including this file
 
 // LOG2_CONST_1 = 0.000000  0 = 0x0000
 #define LOG2_CONST_1 ((pmf_logarithmic)0x0000)