Merged
Conversation
Co-Authored-By: zarthcode <anthony.clay@zarthcode.com>
Don't build and link with different flags, the binary may not work.
- Respect axis max acceleration limits instead of forcing a fixed acceleration value. - The `junction_unit_vec` ensures proper handling of entry and exit speeds even when the axes involved have different limits.
[2.0.x] Fix / improve junction deviation
* Misc fixes and improvements - Get rid of most critical sections on the Serial port drivers for AVR and DUE. Proper usage of FIFOs should allow interrupts to stay enabled without harm to queuing and dequeuing. Also, with 8-bit indices (for AVR) and up to 32-bit indices (for ARM), there is no need to protect reads and writes to those indices. - Simplify the XON/XOFF logic quite a bit. Much cleaner now (both for AVR and ARM) - Prevent a race condition (edge case) that could happen when estimating the proper value for the stepper timer (by reading it) and writing the calculated value for the time to the next ISR by disabling interrupts in those critical and small sections of the code - The problem could lead to lost steps. - Fix dual endstops not properly homing bug (maybe). * Set position immediately when possible
[2.0.x] Manual Z Start Height
Co-Authored-By: teemuatlut <teemu.mantykallio@live.fi>
[2.0.x] Configure stepper drivers per axis
Typo fix
[2.0.x] Tweak some SoftPWM code
Co-Authored-By: vitekn <vitekn@users.noreply.github.com>
The current Marlin implementation relies on a timer interrupt to start the ADC conversion and read it. However in some circumstances the interrupt can be delayed resulting in insufficient time being available for the ADC conversion. This results in a bad reading and false temperature fluctuations. These changes make sure that the conversion is complete (by checking the ADC hardware via the HAL) before reading a value. See: #11323
Co-Authored-By: bleughb <36889022+bleughb@users.noreply.github.com>
Indeed the HAL does not need to be mucked around with to expose ATmega2560 pins not available as numbers on the MEGA board, I'll need to update the wiki with that tidbit and a reference to the pin-mapping comment in `fastio_1280.h`.
-normalize `env` and `board` to lowercase naming convention. -make board `name` follow descriptive convention. -implement `-fsingle-precision-constant` compile optimization per #11178 (comment) -fix typo in 5DPRINT entry.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requirements
Description
Benefits
Related Issues