Skip to content

Conversation

@Floppy
Copy link

@Floppy Floppy commented Jun 28, 2021

Description

Some SKR Mini E3 v1.2 boards have 512k of available memory, not 256k, and need the RE build to make use of it.

Requirements

BigTreeTech SKR Mini E3 v1.2 (sub-variant with a secret 512k of memory)

Benefits

Re-enables support for these variant boards, which many of us have been using happily for over a year.

Related Issues

Some boards have 512k of available memory, not 256k, and need the RE build to make use of it. See MarlinFirmware#22050 for discussion and testing, and MarlinFirmware#15890 for the original discussion of 1.2 RET6 boards.
@Floppy
Copy link
Author

Floppy commented Jun 28, 2021

@Taomyn @rhapsodyv I know you folks have the same boards, if you could test this specific PR, I'm sure that would be helpful in getting this re-enabled in the official releass.

Copy link

@Taomyn Taomyn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me

image

@thisiskeithb
Copy link
Member

thisiskeithb commented Jun 28, 2021

This was removed on purpose.

If it works for your board, feel free to enable it in your fork, but it was not safe for everyone, and that is why the fake 512k envs were removed.

@Floppy
Copy link
Author

Floppy commented Jun 28, 2021

@thisiskeithb that makes sense, for sure. Would adding a separate motherboard name like BTT_SKR_MINI_E3_V1_2_RET6 be more acceptable, or would you not want this capability in core at all?

Either way, this PR can act as a single-commit reference for people wanting to make the same change in future 👍

@thisiskeithb
Copy link
Member

Would adding a separate motherboard name like BTT_SKR_MINI_E3_V1_2_RET6 be more acceptable, or would you not want this capability in core at all?

The 1.2 didn’t ship with an RET6, so adding a new fake board or environment is just bypassing the safety aspect and reason why the fake 512k envs were removed.

Again, feel free to overflash your board if it works for you, but those envs were removed for a reason.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants