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] PWC - Setup killpin/powerhold ASAP or printer suicides #20810

Merged
merged 4 commits into from
Jan 25, 2021

Conversation

Rockman18
Copy link
Contributor

Description

This PR fix the issue detected during MKS_WIFI_MODULE release (#20191). With a PWC the printer suicides during the serial init of the MKS wifi. The setup of killpin/powerhold occurs too late in the process.

Requirements

  • MKS PWC
  • MKS WIFI MODULE

Benefits

The printer doesn't suicide during init.

Configurations

N/A

Related Issues

#20191 (comment)

@Rockman18 Rockman18 changed the title [FIX] Setup killpin/powerhold ASAP or printer can suicide [FIX] PWC - Setup killpin/powerhold ASAP or printer suicides Jan 20, 2021
@X-Ryl669
Copy link
Contributor

Did you test this ? (I don't have the configuration to test it).
It looks good to me.

Marlin/src/MarlinCore.cpp Outdated Show resolved Hide resolved
@Rockman18
Copy link
Contributor Author

Rockman18 commented Jan 21, 2021

Yes i've tested it. It fixed my issue. I don't know but I wonder if only powerhold would be enough.

@thinkyhead thinkyhead merged commit b9ed139 into MarlinFirmware:bugfix-2.0.x Jan 25, 2021
TyMi pushed a commit to TyMi/Marlin that referenced this pull request Jan 25, 2021
…_bugfix

* commit '876c2586b9146dd123af4c7b21138b8239ef5d39': (116 commits)
  Clean up MMU2 code (MarlinFirmware#20794)
  Init KILL, SUICIDE, PSU earlier (MarlinFirmware#20810)
  "Move … code" followup (MarlinFirmware#20869)
  Apply SEC_TO_MS and other fixes
  Reformat abortSDPrinting
  Fix sign warning (MarlinFirmware#20872)
  Cosmetic changes (2) (MarlinFirmware#20876)
  "Move … code" followup (MarlinFirmware#20868)
  "Move … code" followup (MarlinFirmware#20874)
  Graphical TFT fixes, cleanup (MarlinFirmware#20861)
  🧻 Cosmetic changes (MarlinFirmware#20859)
  🛠Fix deps script version regex
  Fix Ender 3 V2 DWIN manual move (MarlinFirmware#20837)
  [cron] Bump distribution date (2021-01-25)
  Fix LiquidCrystal CI failures (MarlinFirmware#20873)
  Revert "Adding custom move feedrate for G26 (MarlinFirmware#20729)" (MarlinFirmware#20870)
  MeatPack serial encoding (MarlinFirmware#20802)
  [cron] Bump distribution date (2021-01-24)
  lcd_put_wchar_max for COLOR_UI (MarlinFirmware#20838)
  Adding custom move feedrate for G26 (MarlinFirmware#20729)
  ...

# Conflicts:
#	Marlin/Configuration.h
#	Marlin/Configuration_adv.h
@Rockman18 Rockman18 deleted the fix-suicide branch January 28, 2021 23:11
Jyers pushed a commit to Jyers/Marlin that referenced this pull request Feb 3, 2021
kpishere pushed a commit to kpishere/Marlin that referenced this pull request Feb 19, 2021
@fighter777
Copy link

fighter777 commented Feb 24, 2021

hello, now the psu pin on SKR pro 1.1 don't turn on -_-"

#21046

Before :

  1. PSU pin on
  2. Screen display : Switch Power Off
    image

After :

  1. PSU pin off
  2. Screen display : Switch Power Off

@Evg33
Copy link
Contributor

Evg33 commented Feb 24, 2021

#21182

zillarob pushed a commit to zillarob/Marlin that referenced this pull request Feb 25, 2021
Evg33 added a commit to Evg33/Marlin that referenced this pull request Feb 25, 2021
W4tel-BiDi pushed a commit to W4tel-BiDi/Marlin that referenced this pull request Apr 5, 2021
thinkyhead pushed a commit to thinkyhead/Marlin that referenced this pull request Apr 29, 2021
thinkyhead pushed a commit that referenced this pull request Apr 30, 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.

5 participants