Other proposed changes

Author: thinkyhead

src/TMCStepper.h

@@ -1362,6 +1362,10 @@ class TMC2240Stepper {
 		uint8_t TPOWERDOWN();
 		void TPOWERDOWN(uint8_t input);
 
+		// RW: TPWMTHRS
+		uint32_t TPWMTHRS();
+		void TPWMTHRS(						uint32_t input);
+
 		uint8_t status_response;
 	protected:
 		INIT2240_REGISTER(GCONF)			{{.sr=0}};
@@ -1374,6 +1378,7 @@ class TMC2240Stepper {
 		INIT2240_REGISTER(GLOBAL_SCALER)	{{.sr=0}};		// 8b
 		INIT_REGISTER(COOLCONF){{.sr=0}};	// 32b
 		INIT_REGISTER(TCOOLTHRS){.sr=0};	// 32b
+		INIT_REGISTER(TPWMTHRS){.sr=0};			// 32b
 		INIT_REGISTER(THIGH){.sr=0};			// 32b
 
 		struct IFCNT_t 		{ constexpr static uint8_t address = 0x02; };

src/source/GCONF.cpp

@@ -69,10 +69,10 @@ Enable analog test output on pin DCO. IHOLD[1..0] selects the function of DCO:
 Not for user, set to 0 for normal operation!
 */
 
-void TMC5160Stepper::recalibrate(bool B)			{ SET_REG(recalibrate); 			}
-void TMC5160Stepper::faststandstill(bool B)			{ SET_REG(faststandstill); 			}
-void TMC5160Stepper::multistep_filt(bool B)			{ SET_REG(multistep_filt); 			}
-bool TMC5160Stepper::recalibrate()					{ GCONF_t r{0}; r.sr = GCONF(); return r.recalibrate;	}
+void TMC5160Stepper::recalibrate(bool B)			{ SET_REG(recalibrate); 	}
+void TMC5160Stepper::faststandstill(bool B)			{ SET_REG(faststandstill); 	}
+void TMC5160Stepper::multistep_filt(bool B)			{ SET_REG(multistep_filt); 	}
+bool TMC5160Stepper::recalibrate()					{ GCONF_t r{0}; r.sr = GCONF(); return r.recalibrate;		}
 bool TMC5160Stepper::faststandstill()				{ GCONF_t r{0}; r.sr = GCONF(); return r.faststandstill;	}
 bool TMC5160Stepper::multistep_filt()				{ GCONF_t r{0}; r.sr = GCONF(); return r.multistep_filt;	}
 

src/source/TMC2240Stepper.cpp

@@ -31,9 +31,14 @@ TMC2240Stepper::TMC2240Stepper(uint16_t pinCS, uint16_t pinMOSI, uint16_t pinMIS
 	}
 
 void TMC2240Stepper::defaults() {
-	//GCONF_register.sr = 0x0000;
-	//CHOPCONF_register.sr = 0x10000053;
-	//PWMCONF_register.sr = 0xC10D0024;
+	//GCONF_register.en_spreadcycle = 0;
+	GCONF_register.multistep_filt = 1;
+
+	IHOLD_IRUN_register.iholddelay = 1;
+	TPOWERDOWN_register.sr = 20;
+
+	CHOPCONF_register.sr = 0x10000053; 	// 1/16 x 256 single-edge
+	PWMCONF_register.sr = 0xC10D0024; 	// autoscale=1, freq=01, grad & amp, freewheel=00
 }
 
 __attribute__((weak))
@@ -152,7 +157,6 @@ void TMC2240Stepper::begin() {
 
 	//GCONF(GCONF_register.sr);
 	//CHOPCONF(CHOPCONF_register.sr);
-	//COOLCONF(COOLCONF_register.sr);
 	//PWMCONF(PWMCONF_register.sr);
 	//IHOLD_IRUN(IHOLD_IRUN_register.sr);
 
@@ -161,7 +165,10 @@ void TMC2240Stepper::begin() {
 }
 
 void TMC2240Stepper::push() {
+	TPOWERDOWN(TPOWERDOWN_register.sr);
+	TPWMTHRS(TPWMTHRS_register.sr);
 	GCONF(GCONF_register.sr);
+	SLAVECONF(SLAVECONF_register.sr);
 	DRV_CONF(DRV_CONF_register.sr);
 	IHOLD_IRUN(IHOLD_IRUN_register.sr);
 	CHOPCONF(CHOPCONF_register.sr);
@@ -327,6 +334,13 @@ void TMC2240Stepper::TPOWERDOWN(uint8_t input) {
 	write(TPOWERDOWN_register.address, TPOWERDOWN_register.sr);
 }
 
+// W: TPWMTHRS
+uint32_t TMC2240Stepper::TPWMTHRS() { return TPWMTHRS_register.sr; }
+void TMC2240Stepper::TPWMTHRS(uint32_t input) {
+  TPWMTHRS_register.sr = input;
+  write(TPWMTHRS_register.address, TPWMTHRS_register.sr);
+}
+
 void TMC2240Stepper::hysteresis_end(int8_t value) { hend(value+3); }
 int8_t TMC2240Stepper::hysteresis_end() { return hend() - 3; };
 

src/source/TMC2240_bitfields.h

@@ -67,8 +67,8 @@ namespace TMC2240_n {
     union {
       uint16_t sr : 12;
       struct {
-        uint16_t  SLAVEADDR : 8;
-        uint8_t   SENDDELAY : 4;
+        uint16_t  slaveaddr : 8;
+        uint8_t   senddelay : 4;
       };
     };
   };