[2.0.x] UltiMachine Archim2 #8190

teemuatlut · 2017-10-31T18:09:48Z

Support for UltiMachine Archim2.0 board with 32bit SAM3X8E (Due) MCU and five TMC2130 stepper drivers.
Makes use of the Due HAL but requires a customized Arduino Due core, made by UltiMachine.
Install instructions for Arduino Archim addons.

Initially tested by UltiMachine.

Currently compiling is broken due to a recent commit on the Due HAL.

Tagging @tonokip

thinkyhead · 2017-10-31T18:35:53Z

Marlin/src/module/stepper_indirection.cpp

@@ -181,7 +181,7 @@
    st.power_down_delay(128); // ~2s until driver lowers to hold current
    st.hysterisis_start(0); // HSTRT = 1
    st.hysterisis_low(1); // HEND = -2
-    st.diag1_active_high(1); // For sensorless homing
+    st.diag1_active_high(0); // For sensorless homing


Fixes a bug?

It was advised by UltiMachine because the diag pins are connected. I didn't ask further questions and for sensorless homing it doesn't really matter as the triggering logic can be inverted in the configuration files.

How will this affect boards other than Archim?

The user will need to change the _ENDSTOP_INVERTING setting. No change is needed if sensorless homing isn't used.

Seems sensible!

thinkyhead · 2017-10-31T18:36:22Z

Should compile now. const was removed from the isrtab declaration.

thinkyhead · 2017-10-31T18:37:56Z

requires a customized Arduino Due core, made by UltiMachine.

Would you mind adding a comment at the top of the ARCHIM pins file explaining this in brief and giving a link to the relevant web page? Thanks!

thinkyhead · 2017-10-31T18:40:44Z

TMC2130 + 32-bit is the froshizzle!

teemuatlut · 2017-10-31T19:19:13Z

Would you mind adding a comment at the top of the ARCHIM pins file explaining this in brief and giving a link to the relevant web page? Thanks!

Sure thing. I also pitched UltiMachine the idea to create a README.md file into the Archim2 folder and there detail any additional instructions.

teemuatlut · 2017-10-31T19:56:28Z

The isrtab error is cleared now but now it fails at MarlinSerial_Due.cpp because for
Archim SERIAL_PORT is -1 and that isn't handled. The error itself is about a missing definition for HWUART and I don't really have the knowledge to properly expand that part.

thinkyhead · 2017-11-01T03:10:17Z

The -1 SERIAL_PORT option should be added pretty soon. Assuming it means the same thing for Archim as it does for #8148. @tcm0116 should be able to confirm.

thinkyhead · 2017-11-01T03:13:47Z

Pro-tip: Only a single back-tick is needed to format text in code style.
Three backticks plus an optional file suffix (```cpp) on a line by itself for multi-line code:

# Comment
int a = 5;

Diff with ```diff example…

  # Comment
- int a = 5;
+ int a = 4;

teemuatlut · 2017-11-01T08:51:06Z

The -1 SERIAL_PORT option should be added pretty soon. Assuming it means the same thing for Archim as it does for #8148. @tcm0116 should be able to confirm.

The board uses the native serialUSB for communication so I'm not sure if there's any need to even compile the whole MarlinSerial class. I don't think this problem here is too closely related to having multiple serial ports in use.

thinkyhead · 2017-11-01T16:04:46Z

Marlin/Marlin.ino

@@ -54,7 +54,6 @@
 #endif

 #if ENABLED(HAVE_TMC2130)
-  #include <SPI.h>


Already included by TMC2130Stepper.h?

Yep. Also the Archim2 uses software SPI and the library handles it according to flags set by the core.

thinkyhead · 2017-11-02T05:32:56Z

Is there a specific reason why USE_WATCHDOG is disabled on the config?

teemuatlut · 2017-11-02T06:59:43Z

Is there a specific reason why USE_WATCHDOG is disabled on the config?

"Also you will need to disable the watchdog in Configuration.h for now as the Archim core disables it. I'll need to do an updated version to fix this similar to newer Arduino Due versions. "
-UltiMachine

jhitesma · 2017-11-03T05:03:29Z

Marlin/src/pins/pins_ARCHIM2.h

+
+#if ENABLED(ULTRA_LCD)
+  #if ENABLED(NEWPANEL)
+    #define BEEPER         23   // D24 PA15_CTS1


BEEPER is now BEEPER_PIN

jhitesma · 2017-11-03T05:12:59Z

Marlin/src/pins/pins_ARCHIM2.h

+#define TEMP_1_PIN          9   // D9 PB18 THERM AN2
+#define TEMP_2_PIN          8   // D8 PB17 THERM AN4
+#define TEMP_BED_PIN       11   // D11 PB20 THERM AN3
+


Very minor, but these four are also different from v1 and could be marked with * for completeness.

thinkyhead · 2017-11-03T09:47:22Z

Marlin/src/pins/pins_ARCHIM2.h

+#define HEATER_0_PIN        6   // D6 PC24 FET_PWM3
+#define HEATER_1_PIN        7   // D7 PC23 FET_PWM4
+#define HEATER_2_PIN        8   // D8 PC22 FET_PWM5
+#define HEATER_BED_PIN      9   // D9 PC21 BED_PWM


We could follow the convention set forth in pins_RAMPS.h where we can swap around the functions based on user selection of extruders, fans, and bed, assigning EEF, EEB, EFB, etc.

tonokip · 2017-11-03T18:17:30Z

Regarding the watchdog, it can be re-enabled. It used to be disabled in the Archim core/variant but that is not the case anymore.

jhitesma · 2017-11-03T19:52:49Z

Marlin/src/pins/pins_ARCHIM2.h

+    #define LCD_PINS_D7    34   // D34 PC2_PWML0
+
+    #define SD_DETECT_PIN   2   // D2  PB25_TIOA0
+    #define SDSS           87   // D87 PA29 CS


I know this merged already...but I was only able to get it to build if I moved the define for the SDSS pin out of the LCD section (even if I tried configuring an LCD when building.) Looking at other pins files it appears this is now always defined on other boards and no longer conditional.

thinkyhead changed the title ~~UltiMachine Archim2~~ [2.0.x] UltiMachine Archim2 Oct 31, 2017

thinkyhead added C: Boards/Pins T: HAL & APIs Topic related to the HAL and internal APIs. labels Oct 31, 2017

thinkyhead reviewed Oct 31, 2017

View reviewed changes

teemuatlut added 8 commits October 31, 2017 21:41

UltiMachine Archim2 support

fb7f6a5

SPI gets included by the library

82f1060

Invert TMC2130 diag pin state

90b095d

Clean up Archim2 pins

5af11a1

Update Archim2 config

3554ea1

FAN2 -> FAN1

bb90afb

Cleanup

abace44

Add brief Archim installation instructions

0f98d20

teemuatlut force-pushed the Archim2 branch from a4b62f4 to 0f98d20 Compare October 31, 2017 19:49

thinkyhead added 2 commits October 31, 2017 22:17

Merge branch 'bugfix-2.0.x' into Archim2

64fd6a6

Merge branch 'bugfix-2.0.x' into Archim2

5067c63

thinkyhead reviewed Nov 1, 2017

View reviewed changes

thinkyhead added 3 commits November 1, 2017 11:28

Bring pins into compliance

89eba78

Update 1.0 SD Detect pin for 1.1

11dddcf

Update Configuration.h

d363ed1

jhitesma mentioned this pull request Nov 2, 2017

[FR] [2.0.x] Ultimachine Archim (SAM3X8E) work-in-progress #7776

Closed

jhitesma reviewed Nov 3, 2017

View reviewed changes

Update pins_ARCHIM2.h

c6f8375

thinkyhead reviewed Nov 3, 2017

View reviewed changes

thinkyhead merged commit 70be4d2 into MarlinFirmware:bugfix-2.0.x Nov 3, 2017

jhitesma reviewed Nov 3, 2017

View reviewed changes

teemuatlut mentioned this pull request Nov 5, 2017

2.0.x Archim2 Compile error #8276

Closed

teemuatlut deleted the Archim2 branch November 23, 2017 09:40

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[2.0.x] UltiMachine Archim2 #8190

[2.0.x] UltiMachine Archim2 #8190

teemuatlut commented Oct 31, 2017 •

edited

thinkyhead Oct 31, 2017

teemuatlut Oct 31, 2017

thinkyhead Nov 1, 2017

teemuatlut Nov 1, 2017

thinkyhead Nov 1, 2017

thinkyhead commented Oct 31, 2017

thinkyhead commented Oct 31, 2017

thinkyhead commented Oct 31, 2017

teemuatlut commented Oct 31, 2017 •

edited

teemuatlut commented Oct 31, 2017 •

edited by thinkyhead

thinkyhead commented Nov 1, 2017

thinkyhead commented Nov 1, 2017 •

edited

teemuatlut commented Nov 1, 2017

thinkyhead Nov 1, 2017

teemuatlut Nov 1, 2017

thinkyhead commented Nov 2, 2017 •

edited

teemuatlut commented Nov 2, 2017

jhitesma Nov 3, 2017

jhitesma Nov 3, 2017

thinkyhead Nov 3, 2017

tonokip commented Nov 3, 2017

jhitesma Nov 3, 2017

[2.0.x] UltiMachine Archim2 #8190

[2.0.x] UltiMachine Archim2 #8190

Conversation

teemuatlut commented Oct 31, 2017 • edited

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

thinkyhead commented Oct 31, 2017

thinkyhead commented Oct 31, 2017

thinkyhead commented Oct 31, 2017

teemuatlut commented Oct 31, 2017 • edited

teemuatlut commented Oct 31, 2017 • edited by thinkyhead

thinkyhead commented Nov 1, 2017

thinkyhead commented Nov 1, 2017 • edited

teemuatlut commented Nov 1, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

thinkyhead commented Nov 2, 2017 • edited

teemuatlut commented Nov 2, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tonokip commented Nov 3, 2017

Choose a reason for hiding this comment

teemuatlut commented Oct 31, 2017 •

edited

teemuatlut commented Oct 31, 2017 •

edited

teemuatlut commented Oct 31, 2017 •

edited by thinkyhead

thinkyhead commented Nov 1, 2017 •

edited

thinkyhead commented Nov 2, 2017 •

edited