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

Software reset via panel button press #13091

Merged
merged 3 commits into from
Mar 10, 2019
Merged

Software reset via panel button press #13091

merged 3 commits into from
Mar 10, 2019

Conversation

bjbeare
Copy link
Contributor

@bjbeare bjbeare commented Feb 7, 2019

Pressing the kill button a second time (once in killed state)
will call the Arduino reset vector and restart Marlin.

Signed-off-by: Bruce Beare [email protected]

Requirements

  • Filling out this template is required. Pull Requests without a clear description may be closed at the maintainers' discretion.

Description

Benefits

No longer have to power cycle after hitting the front panel switch (if HAS_KILL). Instead, you can press it a second time and the board restarts Marlin.

Related Issues

bjbeare and others added 2 commits February 9, 2019 07:39
Pressing the kill button a second time (once in killed state)
will call the Arduino reset vector and restart Marlin.

Signed-off-by: Bruce Beare <[email protected]>
@thinkyhead
Copy link
Member

It's a good concept, but… This technique to reset the board doesn't work reliably. We had a long thread about it. If I can find the issue number, I will post a link.

@bjbeare
Copy link
Contributor Author

bjbeare commented Feb 10, 2019

Thanks, please do. Glad to rework.
FWIW, I've not had any problems with it. I use a programmer to flash the board rather than downloading from the IDE. Perhaps that gives it more reliability.

@thinkyhead
Copy link
Member

See #8994 for a discussion of the reset problems.

@thinkyhead thinkyhead requested a review from AnHardt February 13, 2019 00:49
@bjbeare
Copy link
Contributor Author

bjbeare commented Feb 22, 2019

Putting my printer back together after a bit of upgrade. I'll try this out today or tomorrow.

@thinkyhead thinkyhead removed the request for review from AnHardt February 22, 2019 21:37
@psavva
Copy link
Contributor

psavva commented Feb 27, 2019

I would love to see this work. I can test on the re-arm

@thinkyhead thinkyhead merged commit e4d080e into MarlinFirmware:bugfix-2.0.x Mar 10, 2019
@J-charles-C
Copy link

J-charles-C commented Mar 10, 2019

I,

great job, many Thanks !

Best

EDIT : I test on my re-arm. It's OK.

First click Kill Marlin. Second click reset it !

Thanks.

@psavva
Copy link
Contributor

psavva commented Mar 10, 2019 via email

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.

4 participants