-
-
Notifications
You must be signed in to change notification settings - Fork 19.5k
Description
Did you test the latest bugfix-2.1.x
code?
Yes, and the problem still exists.
Bug Description
When using Marlin on an MiniRambo Board, where the Extruder is used as another axis (here i-axis) by defining
#define I_STEP_PIN 34
#define I_DIR_PIN 43
#define I_ENABLE_PIN 26
and
#define I_MS1_PIN 65
#define I_MS2_PIN 66
(basically replacing the E0 pins with I pins)
compilation fails when trying to access I_MS3 pin, which is not defined.
Changing
Marlin/Marlin/src/module/stepper.cpp
Line 4280 in 0589655
#if HAS_I_MS_PINS |
to
#if HAS_I_MS_PINS && PIN_EXISTS(I_MS3)
fixes this issue (at least for the i-axis).
Note, there is also a bunch of semicolons missing in the current stepper.cpp after "break" statements related to I, J, K, U, V and W axes in function Stepper::microstep_ms(...), e.g.
Marlin/Marlin/src/module/stepper.cpp
Line 4281 in 0589655
case I_AXIS: WRITE(I_MS3_PIN, ms3); break |
Adding the above pre-processor condition and fixing the semicolons makes the configuration compile and results in a working i-axis.
Bug Timeline
Tried branch lts-2.1.1 first and then bugfix 2,1,x, Both are broken.
Expected behavior
Should compile.
Actual behavior
Compilation fails with undefined I_MS3_PIN.
Steps to Reproduce
see bug details
Version of Marlin Firmware
2.1.x
Printer model
Custom, based on MiniRambo Board
Electronics
MiniRambo
LCD/Controller
not relevant
Other add-ons
none
Bed Leveling
None
Your Slicer
None
Host Software
None
Don't forget to include
- A ZIP file containing your
Configuration.h
andConfiguration_adv.h
.
Additional information & file uploads
No response