Skip to content

Fix starting fan with bad material preset index#19773

Merged
thinkyhead merged 4 commits intoMarlinFirmware:bugfix-2.0.xfrom
rhapsodyv:bed-no-fan-on-preheat
Oct 17, 2020
Merged

Fix starting fan with bad material preset index#19773
thinkyhead merged 4 commits intoMarlinFirmware:bugfix-2.0.xfrom
rhapsodyv:bed-no-fan-on-preheat

Conversation

@rhapsodyv
Copy link
Member

@rhapsodyv rhapsodyv commented Oct 16, 2020

Description

When pre heating only bed, the pre head code receives -1 to not start fan, but the code don't handle it, passing a random memory value to the set_fan_speed.

This fix that.

Benefits

Fix #19737

Related Issues

#19737

@thinkyhead thinkyhead changed the title Fix - #19737 - Starting fun with invalid material index Fix starting fan with invalid material index Oct 17, 2020
@thinkyhead thinkyhead changed the title Fix starting fan with invalid material index Fix starting fan with bad material preset index Oct 17, 2020
@thinkyhead thinkyhead merged commit 313efa3 into MarlinFirmware:bugfix-2.0.x Oct 17, 2020
Zorchz pushed a commit to Zorchz/Marlin-1 that referenced this pull request Oct 17, 2020
Zorchz pushed a commit to Zorchz/Marlin-1 that referenced this pull request Oct 17, 2020
thinkyhead pushed a commit to thinkyhead/Marlin that referenced this pull request Oct 21, 2020
Speaka pushed a commit to Speaka/Marlin that referenced this pull request Oct 23, 2020
@rhapsodyv rhapsodyv deleted the bed-no-fan-on-preheat branch October 25, 2020 00:32
Speaka pushed a commit to Speaka/Marlin that referenced this pull request Nov 2, 2020
vgadreau pushed a commit to vgadreau/Marlin that referenced this pull request Dec 9, 2020
tharts pushed a commit to tharts/Marlin that referenced this pull request Jan 6, 2021
kpishere pushed a commit to kpishere/Marlin that referenced this pull request Feb 19, 2021
W4tel-BiDi pushed a commit to W4tel-BiDi/Marlin that referenced this pull request Apr 5, 2021
thinkyhead added a commit to thinkyhead/Marlin that referenced this pull request Apr 28, 2021
thinkyhead added a commit to thinkyhead/Marlin that referenced this pull request Apr 29, 2021
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.

2 participants