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

Sanity check for unsupported emergency parser config #19745

Conversation

rhapsodyv
Copy link
Member

Description

STM32F1 have:

  • HardwareSerial
  • USBSerial
  • USBComponsite

Of these 3 class, HardwareSerial and USBComponsite already have support for Emergency Parser. But USBSerial not.

This PR add a sanity check for that.

Benefits

Avoid users using a invalid config.

Related Issues

@rhapsodyv rhapsodyv force-pushed the sanity-check-unsupported-emergency-parser-stm32f1 branch from 1330565 to 175648f Compare October 15, 2020 12:43
@thinkyhead thinkyhead merged commit 5247bb1 into MarlinFirmware:bugfix-2.0.x Oct 16, 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 sanity-check-unsupported-emergency-parser-stm32f1 branch October 25, 2020 00:17
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
@arjanmels
Copy link
Contributor

I added support for USB Serial in: #19745

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 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.

3 participants