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

Creality Ender 3 Pro – 4.2.X CPUs – 512K swapped out for 256K – WARNING POTENTIAL FIRE HAZARDS #23596

Closed
garaysantiago opened this issue Jan 22, 2022 · 7 comments
Labels

Comments

@garaysantiago
Copy link

garaysantiago commented Jan 22, 2022

Did you test the latest bugfix-2.0.x code?

Yes, and the problem still exists.

Bug Description

In Discord server a user was working on their machine and was having issues with the firmware. They posted a picture of the board and it revealed a lesser STM32 part number on that board. The 4.2.2 and 4.2.7 boards have all shipped with the STM32F103 RET6 CPUs since release. Now there is confirmation that 4.2.2 boards have shipped out from Creality with lesser 256K parts on them that are NOT safe to use with current Marlin and Unified firmware builds.

There are 4.2.2 (we would assume that 4.2.7s may have the lesser CPUs soon, we will be adding part number checks to the 4.2.7s that we sell going forward to make sure they have 512K parts on them) out in the wild with STM32F103 RCT6 CPUs. These CPUs are 256K parts whereas the RET6 versions were 512K parts.

Bug Timeline

No response

Expected behavior

No response

Actual behavior

No response

Steps to Reproduce

No response

Version of Marlin Firmware

2.0.9.3

Printer model

Ender 3 Pro

Electronics

4.2.2 and 4.2.7 boards have all shipped with the STM32F103 RET6 CPUs since release

Add-ons

No response

Bed Leveling

No response

Your Slicer

No response

Host Software

No response

Additional information & file uploads

No response

@garaysantiago garaysantiago changed the title Creality – 4.2.X CPUs – 512K swapped out for 256K – WARNING POTENTIAL FIRE HAZARDS Creality Ender 3 Pro – 4.2.X CPUs – 512K swapped out for 256K – WARNING POTENTIAL FIRE HAZARDS Jan 22, 2022
@thisiskeithb
Copy link
Member

Have any other hardware changes been made? Adding a 256K variant is straight forward.

@thisiskeithb thisiskeithb added C: Boards/Pins Needs: Patch A patch is needed to fix this labels Jan 22, 2022
@tome9111991
Copy link
Contributor

tome9111991 commented Jan 22, 2022

@jbuck2005
Copy link
Contributor

Pardon my ignorance but why the flag "potential fire hazard" ?
Are you proposing that the reduced memory space will force users to forgo the thermal runaway protection mechanism?

@thisiskeithb
Copy link
Member

Pardon my ignorance but why the flag "potential fire hazard" ?

Overflashing could potentially lead to unexplained lockups with the heaters on.

I didn’t experience that issue personally, but some folks did, so YMMV.

@jbuck2005
Copy link
Contributor

Oh wow. Fair point!! I had not even considered that the firmware flash procedure would fail to check for out-of-bounds writes. Good lord.

@thisiskeithb thisiskeithb added the T: Feature Request Features requested by users. label Jan 22, 2022
@thisiskeithb
Copy link
Member

I've created a PR, so please test & report any issues: #23599

@thisiskeithb thisiskeithb added A: STM32 and removed Needs: Patch A patch is needed to fix this labels Jan 22, 2022
@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators Mar 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants