utility/spi_com.h Not found on PR1 branch about marlin

[DerAndere1]

I will have a look how to fix the missing spi_com.h. When upstream Marlin allows to use, BigTreeth_GTR_V1 + M5, there is a chance that PR1 could work. At the moment only 6 axes are possible. Extension to 9 axes is high on the priority list, But it is a lot of work, I will try to start extending to 9 axes soon. If you have experience with c++, you will certainly be quicker then I. Have a look at commit 363f654 There you see how axes IJK were added. You can do the same for AXIS7, AXIS8, and AXIS9.

[DerAndere1]

Have you verified that spi_com.h is found in current upstream Marlin bugfix-2.x? If I'm not mistaken, the file spi_com.h is not part of Marlin itself but a dependency (it is part of the STM32 Arduino core library (https://github.com/stm32duino/Arduino_Core_STM32/tree/master/libraries/SPI/src/utility ). If you configured everything correctly for your board, Marlin should compile with the correct Arduino core library. Please verify, that your board uses the STM32 core.

[paloky]

Hi DerAndere1.
Finally, I am working on the "6axis-dev" branch.

This weekend, I have successfully managed to add a new Axis.
I have tested it with the DRV8825 and with the TMC2209 in UART Mode.
I have found some bugs in this branch, for the use of the TMC2209 in UART Mode on axis "I, J and K", that I have managed to fix.
For the time being, I have internally numbered it as "M".

I will finish implementing the rest of axes, to get all the BTT GTR V1+M5 (11 axis in total).

Since I'm going to add the new axes, I'm going to change the actual internal name (I, J and K) to "A, B, C" and continue with the rest "U, V, W".

In total, we will get (X, Y, Z, A, B, C, U, V, W, E, ?? ) Axis.
Any suggestions for the name of axis number 11? "D" maybe?

P.S. :
I just saw that the active branch is "6axis-PR1"?
In the Wiki, you have put that the main one is "6axis_dev".
Can you confirm me which branch is the active one?

Thank's.
Best Regards.
Paloky.

[DerAndere1]

The recommended branch depends on where you want your code to land. Historically I used the 6axis_dev branch to develop the PR branch. Some time after I had opened pull request 19112 at https://github.com/MarlinFirmware/Marlin to get the 6axis support merged into upstream MarlinFirmware/Marlin, I stopped synchronizing with the PR1 branch. Currently, the 6axis_PR1 branch gets often updated by MarlinFirmware developer thinkyhead to prepare the merge. If you want to get your changes merged into upstream Marlin, I recommend you to prepare a pull request targeted at the PR1 branch and fx all issues there. This approach comes with the risk that maybe initially nothing works and that the pull request never gets merged if thinkyhead does not give his approval.

Because upstream Marlin development is so fast, it is very difficult to get a working snapshot from the PR1 branch. so if the 6axis_dev branch works for you and you just want a working firmware, just go ahead with the 6axis_dev branch and dont hesitate to share a link to your fork or send me a pull request. Your changes just cannot be merged into upstram MarlinFirmware/Marlin, then.

In any case, you should be aware, that thinkyhead made a seperate pull request (21932 at https://github.com/MarlinFirmware/Marlin) that prepares for addition of more than 8 axes. The problem is that for more than 8 axes (or 7 + extruder ?), the size of some 8bit variables is not enough to store data for all axes. Axis name D sounds fine. If you want your code to be useful for others, you should make user-facing axis names (those used in G-code) configurable. Maybe you can get in touch with thinkyhead, he wants support for 9 axes in Marlin, so he may be already working on it

[DerAndere1]

I cherry-picked your 9 axis code and I am now updating it to latest state of multi-axis Marlin. I made sure that the git history including your authorship will remain intact. As soon as I publish the updated code, I will mention your contribution in my documentation, too. Thanks for doing the hard work.

[paloky]

Hi DerAndere1.
Thank you.
I haven't had time to do the PR yet.
I have put a lot of "/SG/" comments, in all the lines I have modified. You can erase this comments...

I'm not been able to implement the 2 missing axes, for my BTT GTR1.0 + M5 Board (11 axis total)
I think it's a problem with the arrays that Marlin uses and I had compilation problems.

Best Regars.

[DerAndere1]

No worries, I was just too curious to test your code. I finished rebasing it onto the 6axis_PR1 branch and will test-compile after some clean-up. current 6axis_PR1 branch is probably already prepared for more than 8 non-extruder axes plus extruder. We will see during hardware testing.

[DerAndere1]

I close this as fixed. My branch 9axis_pull now compiles, including utility/spi_com.h using PlatformIO for your board. Configs for 7 axes from JanHendrik-5D can be compiled. The configs can be found as part of the attachment of the following post:
#44 (comment)