Merge pull request #11610 from Freeviv/bugfix-2.0.x
[2.0.x] Editable servo angles
This commit is contained in:
@ -37,7 +37,7 @@
|
||||
*/
|
||||
|
||||
// Change EEPROM version if the structure changes
|
||||
#define EEPROM_VERSION "V55"
|
||||
#define EEPROM_VERSION "V56"
|
||||
#define EEPROM_OFFSET 100
|
||||
|
||||
// Check the integrity of data offsets.
|
||||
@ -70,6 +70,10 @@
|
||||
#include "../feature/bedlevel/bedlevel.h"
|
||||
#endif
|
||||
|
||||
#if HAS_SERVOS
|
||||
#include "servo.h"
|
||||
#endif
|
||||
|
||||
#if HAS_BED_PROBE
|
||||
#include "../module/probe.h"
|
||||
#endif
|
||||
@ -174,6 +178,11 @@ typedef struct SettingsDataStruct {
|
||||
bool planner_leveling_active; // M420 S planner.leveling_active
|
||||
int8_t ubl_storage_slot; // ubl.storage_slot
|
||||
|
||||
//
|
||||
// SERVO_ANGLES
|
||||
//
|
||||
uint16_t servo_angles[MAX_SERVOS][2]; // M281 P L U
|
||||
|
||||
//
|
||||
// DELTA / [XYZ]_DUAL_ENDSTOPS
|
||||
//
|
||||
@ -532,6 +541,26 @@ void MarlinSettings::postprocess() {
|
||||
EEPROM_WRITE(storage_slot);
|
||||
#endif // AUTO_BED_LEVELING_UBL
|
||||
|
||||
#if !HAS_SERVOS || DISABLED(EDITABLE_SERVO_ANGLES)
|
||||
#if ENABLED(SWITCHING_EXTRUDER)
|
||||
constexpr uint16_t sesa[][2] = SWITCHING_EXTRUDER_SERVO_ANGLES;
|
||||
#endif
|
||||
constexpr uint16_t servo_angles[MAX_SERVOS][2] = {
|
||||
#if ENABLED(SWITCHING_EXTRUDER)
|
||||
[SWITCHING_EXTRUDER_SERVO_NR] = { sesa[0], sesa[1] }
|
||||
#if EXTRUDERS > 3
|
||||
, [SWITCHING_EXTRUDER_E23_SERVO_NR] = { sesa[2], sesa[3] }
|
||||
#endif
|
||||
#elif ENABLED(SWITCHING_NOZZLE)
|
||||
[SWITCHING_NOZZLE_SERVO_NR] = SWITCHING_NOZZLE_SERVO_ANGLES
|
||||
#elif defined(Z_SERVO_ANGLES) && defined(Z_PROBE_SERVO_NR)
|
||||
[Z_PROBE_SERVO_NR] = Z_SERVO_ANGLES
|
||||
#endif
|
||||
};
|
||||
#endif
|
||||
|
||||
EEPROM_WRITE(servo_angles);
|
||||
|
||||
// 11 floats for DELTA / [XYZ]_DUAL_ENDSTOPS
|
||||
#if ENABLED(DELTA)
|
||||
|
||||
@ -1135,6 +1164,14 @@ void MarlinSettings::postprocess() {
|
||||
EEPROM_READ(dummyui8);
|
||||
#endif // AUTO_BED_LEVELING_UBL
|
||||
|
||||
//
|
||||
// SERVO_ANGLES
|
||||
//
|
||||
#if !HAS_SERVOS || DISABLED(EDITABLE_SERVO_ANGLES)
|
||||
uint16_t servo_angles[MAX_SERVOS][2];
|
||||
#endif
|
||||
EEPROM_READ(servo_angles);
|
||||
|
||||
//
|
||||
// DELTA Geometry or Dual Endstops offsets
|
||||
//
|
||||
@ -1774,6 +1811,44 @@ void MarlinSettings::reset(PORTARG_SOLO) {
|
||||
zprobe_zoffset = Z_PROBE_OFFSET_FROM_EXTRUDER;
|
||||
#endif
|
||||
|
||||
//
|
||||
// Servo Angles
|
||||
//
|
||||
|
||||
#if HAS_SERVOS && ENABLED(EDITABLE_SERVO_ANGLES)
|
||||
|
||||
#if ENABLED(SWITCHING_EXTRUDER)
|
||||
|
||||
#if EXTRUDERS > 3
|
||||
#define REQ_ANGLES 4
|
||||
#else
|
||||
#define REQ_ANGLES 2
|
||||
#endif
|
||||
constexpr uint16_t extruder_angles[] = SWITCHING_EXTRUDER_SERVO_ANGLES;
|
||||
static_assert(COUNT(extruder_angles) == REQ_ANGLES, "SWITCHING_EXTRUDER_SERVO_ANGLES needs " STRINGIFY(REQ_ANGLES) " angles.");
|
||||
servo_angles[SWITCHING_EXTRUDER_SERVO_NR][0] = extruder_angles[0];
|
||||
servo_angles[SWITCHING_EXTRUDER_SERVO_NR][1] = extruder_angles[1];
|
||||
#if EXTRUDERS > 3
|
||||
servo_angles[SWITCHING_EXTRUDER_E23_SERVO_NR][0] = extruder_angles[2];
|
||||
servo_angles[SWITCHING_EXTRUDER_E23_SERVO_NR][1] = extruder_angles[3];
|
||||
#endif
|
||||
|
||||
#elif ENABLED(SWITCHING_NOZZLE)
|
||||
|
||||
constexpr uint16_t nozzle_angles[2] = SWITCHING_NOZZLE_SERVO_ANGLES;
|
||||
servo_angles[SWITCHING_NOZZLE_SERVO_NR][0] = nozzle_angles[0];
|
||||
servo_angles[SWITCHING_NOZZLE_SERVO_NR][1] = nozzle_angles[1];
|
||||
|
||||
#elif defined(Z_SERVO_ANGLES) && defined(Z_PROBE_SERVO_NR)
|
||||
|
||||
constexpr uint16_t z_probe_angles[2] = Z_SERVO_ANGLES;
|
||||
servo_angles[Z_PROBE_SERVO_NR][0] = z_probe_angles[0];
|
||||
servo_angles[Z_PROBE_SERVO_NR][1] = z_probe_angles[1];
|
||||
|
||||
#endif
|
||||
|
||||
#endif // HAS_SERVOS && EDITABLE_SERVO_ANGLES
|
||||
|
||||
#if ENABLED(DELTA)
|
||||
const float adj[ABC] = DELTA_ENDSTOP_ADJ,
|
||||
dta[ABC] = DELTA_TOWER_ANGLE_TRIM;
|
||||
@ -2253,6 +2328,35 @@ void MarlinSettings::reset(PORTARG_SOLO) {
|
||||
|
||||
#endif // HAS_LEVELING
|
||||
|
||||
#if HAS_SERVOS && ENABLED(EDITABLE_SERVO_ANGLES)
|
||||
|
||||
if (!forReplay) {
|
||||
CONFIG_ECHO_START;
|
||||
SERIAL_ECHOLNPGM_P(port, "Servo Angles:");
|
||||
}
|
||||
for (uint8_t i = 0; i < NUM_SERVOS; i++) {
|
||||
switch (i) {
|
||||
#if ENABLED(SWITCHING_EXTRUDER)
|
||||
case SWITCHING_EXTRUDER_SERVO_NR:
|
||||
#if EXTRUDERS > 3
|
||||
case SWITCHING_EXTRUDER_E23_SERVO_NR:
|
||||
#endif
|
||||
#elif ENABLED(SWITCHING_NOZZLE)
|
||||
case SWITCHING_NOZZLE_SERVO_NR:
|
||||
#elif defined(Z_SERVO_ANGLES) && defined(Z_PROBE_SERVO_NR)
|
||||
case Z_PROBE_SERVO_NR:
|
||||
#endif
|
||||
CONFIG_ECHO_START;
|
||||
SERIAL_ECHOPAIR_P(port, " M281 P", int(i));
|
||||
SERIAL_ECHOPAIR_P(port, " L", servo_angles[i][0]);
|
||||
SERIAL_ECHOPAIR_P(port, " U", servo_angles[i][1]);
|
||||
SERIAL_EOL_P(port);
|
||||
default: break;
|
||||
}
|
||||
}
|
||||
|
||||
#endif // HAS_SERVOS && EDITABLE_SERVO_ANGLES
|
||||
|
||||
#if ENABLED(DELTA)
|
||||
|
||||
if (!forReplay) {
|
||||
|
@ -54,7 +54,6 @@ float zprobe_zoffset; // Initialized by settings.load()
|
||||
|
||||
#if HAS_Z_SERVO_PROBE
|
||||
#include "../module/servo.h"
|
||||
const int z_servo_angle[2] = Z_SERVO_ANGLES;
|
||||
#endif
|
||||
|
||||
#if ENABLED(Z_PROBE_SLED)
|
||||
@ -435,7 +434,7 @@ bool set_probe_deployed(const bool deploy) {
|
||||
|
||||
#elif HAS_Z_SERVO_PROBE && DISABLED(BLTOUCH)
|
||||
|
||||
MOVE_SERVO(Z_PROBE_SERVO_NR, z_servo_angle[deploy ? 0 : 1]);
|
||||
MOVE_SERVO(Z_PROBE_SERVO_NR, servo_angles[Z_PROBE_SERVO_NR][deploy ? 0 : 1]);
|
||||
|
||||
#elif ENABLED(Z_PROBE_ALLEN_KEY)
|
||||
|
||||
|
@ -53,7 +53,6 @@
|
||||
#endif
|
||||
|
||||
#if HAS_Z_SERVO_PROBE
|
||||
extern const int z_servo_angle[2];
|
||||
void servo_probe_init();
|
||||
#endif
|
||||
|
||||
|
@ -31,6 +31,7 @@
|
||||
#include "servo.h"
|
||||
|
||||
HAL_SERVO_LIB servo[NUM_SERVOS];
|
||||
uint16_t servo_angles[NUM_SERVOS][2];
|
||||
|
||||
void servo_init() {
|
||||
#if NUM_SERVOS >= 1 && HAS_SERVO_0
|
||||
|
@ -30,6 +30,7 @@
|
||||
#include "../HAL/shared/servo.h"
|
||||
|
||||
extern HAL_SERVO_LIB servo[NUM_SERVOS];
|
||||
extern uint16_t servo_angles[NUM_SERVOS][2];
|
||||
extern void servo_init();
|
||||
|
||||
#define MOVE_SERVO(I, P) servo[I].move(P)
|
||||
@ -37,8 +38,8 @@ extern void servo_init();
|
||||
#include "../inc/MarlinConfig.h"
|
||||
|
||||
#if HAS_Z_SERVO_PROBE
|
||||
#define DEPLOY_Z_SERVO() MOVE_SERVO(Z_PROBE_SERVO_NR, z_servo_angle[0])
|
||||
#define STOW_Z_SERVO() MOVE_SERVO(Z_PROBE_SERVO_NR, z_servo_angle[1])
|
||||
#define DEPLOY_Z_SERVO() MOVE_SERVO(Z_PROBE_SERVO_NR, servo_angles[Z_PROBE_SERVO_NR][0])
|
||||
#define STOW_Z_SERVO() MOVE_SERVO(Z_PROBE_SERVO_NR, servo_angles[Z_PROBE_SERVO_NR][1])
|
||||
#endif
|
||||
|
||||
#endif // _SERVO_H_
|
||||
|
@ -60,22 +60,18 @@
|
||||
#if DO_SWITCH_EXTRUDER
|
||||
|
||||
#if EXTRUDERS > 3
|
||||
#define REQ_ANGLES 4
|
||||
#define _SERVO_NR (e < 2 ? SWITCHING_EXTRUDER_SERVO_NR : SWITCHING_EXTRUDER_E23_SERVO_NR)
|
||||
#define _SERVO_NR(E) ((E) < 2 ? SWITCHING_EXTRUDER_SERVO_NR : SWITCHING_EXTRUDER_E23_SERVO_NR)
|
||||
#else
|
||||
#define REQ_ANGLES 2
|
||||
#define _SERVO_NR SWITCHING_EXTRUDER_SERVO_NR
|
||||
#define _SERVO_NR(E) SWITCHING_EXTRUDER_SERVO_NR
|
||||
#endif
|
||||
|
||||
void move_extruder_servo(const uint8_t e) {
|
||||
constexpr int16_t angles[] = SWITCHING_EXTRUDER_SERVO_ANGLES;
|
||||
static_assert(COUNT(angles) == REQ_ANGLES, "SWITCHING_EXTRUDER_SERVO_ANGLES needs " STRINGIFY(REQ_ANGLES) " angles.");
|
||||
planner.synchronize();
|
||||
#if EXTRUDERS & 1
|
||||
if (e < EXTRUDERS - 1)
|
||||
#endif
|
||||
{
|
||||
MOVE_SERVO(_SERVO_NR, angles[e]);
|
||||
MOVE_SERVO(_SERVO_NR(e), servo_angles[_SERVO_NR(e)][e]);
|
||||
safe_delay(500);
|
||||
}
|
||||
}
|
||||
@ -85,9 +81,8 @@
|
||||
#if ENABLED(SWITCHING_NOZZLE)
|
||||
|
||||
void move_nozzle_servo(const uint8_t e) {
|
||||
const int16_t angles[2] = SWITCHING_NOZZLE_SERVO_ANGLES;
|
||||
planner.synchronize();
|
||||
MOVE_SERVO(SWITCHING_NOZZLE_SERVO_NR, angles[e]);
|
||||
MOVE_SERVO(SWITCHING_NOZZLE_SERVO_NR, servo_angles[SWITCHING_NOZZLE_SERVO_NR][e]);
|
||||
safe_delay(500);
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user