Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix MKS Robin E3 extruder SPI bus conflicts #24889

Closed

Conversation

ellensp
Copy link
Contributor

@ellensp ellensp commented Oct 17, 2022

Description

Default environment used SPI3, this conflicted with extruder IO pins.
SDCARD definition was a mess
Updated and cleaned up and added some useful comments.

Tested on BOARD_MKS_ROBIN_E3D_V1_1

Requirements

Any of these 4 boards with SDSUPPORT

BOARD_MKS_ROBIN_E3
BOARD_MKS_ROBIN_E3_V1_1
BOARD_MKS_ROBIN_E3D
BOARD_MKS_ROBIN_E3D_V1_1

Benefits

Extruder moves as expected.

Configurations

Bugfix 2.1.x https://github.com/MarlinFirmware/Marlin/files/9799514/changes.zip

Related Issues

#24882
#24380
#23715

@thinkyhead thinkyhead force-pushed the extruder-conflict-robin-e3d branch from cb68ce3 to 44a21cf Compare October 20, 2022 21:10
@ellensp ellensp closed this Oct 21, 2022
@thinkyhead thinkyhead changed the title fix extruder spi bus conflicts on mks robin 3e boards Fix MKS Robin E3 extruder SPI bus conflicts Oct 21, 2022
@thinkyhead
Copy link
Member

Not needed after all, or fixed in a different manner?

@thisiskeithb
Copy link
Member

Not needed after all, or fixed in a different manner?

#24380 was reopened, so I’m guessing this is still needed.

@thinkyhead
Copy link
Member

So, is this a safe fix, or is it too broad and affecting other boards it should not?

@ellensp
Copy link
Contributor Author

ellensp commented Oct 22, 2022

Is needed but is reported that it randomly locks up.. so more work is needed that just fixing the conflict

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants