Added/improved a couple of comments to better explain what is going on. #107
Conversation
This file contains 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
damosvil
approved these changes
Jun 12, 2023
|
Hey @damosvil - it's just a draft for now. I want to go through the rest of the files too and add comments, just to makes sure I can follow everything too ;-) |
damosvil
approved these changes
Jun 12, 2023
stylesuxx
force-pushed
the
chore/refactor-ram-clear
branch
from
55283e7
to
ed98094
Compare
stylesuxx
force-pushed
the
chore/refactor-ram-clear
branch
from
d1a9c89
to
f49d99b
Compare
damosvil
reviewed
Jun 14, 2023
stylesuxx
added a commit
that referenced
this pull request
Jul 14, 2023
* Codebase splitted in modules * Updated makefile for splitting * 'Cherry picked' 0.19.2 bugfix. * Update editorconfig * Removed dangling ENDIF * Open bootloader file with banner * Add autoformatter and linting tool * Initial pass with the formatter * Disable comment formatting by default (except inline comments) * Make sure all files end with an empty line * Unified comments in layout files according to #96 * Unified comments in accordance to #96 * Unified pinouts in main and layout files #13 * Do not remove spaces in inline comments * Last pass with the formatter * This commit reduces the scheduler timebase from 16ms to 128ms to interfere the minimum with commutation * Refactored duplicate assignments #99 (#101) * Refactored duplicate assignments * Refactored CSEG definitions into separate file * Refactored MCU types and checks * Fixed check time bug and replaced schedule time literals by macros * Replaced Stop_Adc and Start_Adc macro calls by Reastart_Adc macro call * Updated names to more descriptive. Added parameter to select timebase depending on the needs. Scheduler cycle set to 1024ms by default. * Add 'enums' for PWM frequency and resolution #103 (#104) * Bugfix/bring back bidirectional 3d mode (#105) * Fixed temperature limiting affecting to 3D mode. * Fixed Pwm_Limit_By_Rpm to match comments concept * Implemented EDT safety arm feature * Disable temp protection by default #95 (#100) * Add ENUM for DSHOT commands (#117) * Refactored MCU specific offsets to separate module (#116) * Use a single macro for values. (#120) * Renamed functions to improve readability * Update melody size and tag count to be a single macro * Implemented lock and unlock flash macros (#121) * Refactored 16 bit division by two into seperate macro #112 (#119) * Initial draft for supporter section (#89) * Bump version number and mark as DEV version #122 (#124) * Add TuneRC and T-Motor to the supporter section #85 * Add functionality to support two char layouts. Add OA layout. #113 (#114) * Removed relative jumps (#126) * Added/improved a couple of comments to better explain what is going on. (#107) * Improved documentation, unified formatting, added explanations where necessary. * Disable dithering by default (#130) * Allow version string to be passed to Makefile via buildscript (for naming resulting files) (#128) * RC7 tag (#132) * Apply version also if building all files * Fixed RC version * Further reducing motor speed before changing direction in 3D bidirectional mode (#138) * RC10 * Add happymodel and weebleed to the 'Supporters' section #139 (#140)
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.