sync with upstream #2
Open
Conversation
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
Reservation of an MCU_ID area to simplificate adding of upcoming BLHeli_32 MCU
Co-authored-by: nerdCopter <[email protected]>
add target HOWI_HOBBYWING_XROTORF7CONV
* make angle feel more responsive * Update pid.c Co-authored-by: ctzsnooze <[email protected]>
due to many users complaining of issues with this latest release and seeing a handful fix this by just raising their IMUF q, I'm pulling these changes. I think we should just replace the current 0.4.0 hex files with new ones that contain these defaults.
* IFRC_IFLIGHT_F745_AIO * IFRC_IFLIGHT_F745_AIO LED * IFRC_IFLIGHT_F745_AIO full fixes Co-authored-by: Kaiowarez <[email protected]> Co-authored-by: nerdCopter <[email protected]>
Co-authored-by: Kaiowarez <[email protected]>
Co-authored-by: Kaiowarez <[email protected]>
* add df_yaw to BB header * add df_angle_high/low to BB header
* LUXMINIF7 BlackBox Fix Co-authored-by: BeauBrewski <[email protected]>
Co-authored-by: BeauBrewski <[email protected]>
With beginning of GCC 9 a great deal of warnings of "unaligned pointer value" are introduced. For dropping the warning: "taking address of packed member of 'struct accDev_s' may result in an unaligned pointer value [-Waddress-of-packed-member]" simply remove the 'packing' similar to commit: 5b09b39 The __attribute__((packed)) means the smallest possible alignment (1 byte), usually used for implementing binary network protocols.
The function cast in macro PG_REGISTER_ARRAY_WITH_RESET_FN wants (pgResetFunc)(void * /* base */) rather than (pgResetFunc)(void * /* base */, int /* size */); This commit fixes the warning caused by incompatible function types.
This commit fixes warning: statement may fall through [-Wimplicit-fallthrough=]
Fix out of bounds buffer problems caused by employing incorrect sizes VOL_LABEL_LEN != 12 in bs->volume_label FILE_NAME_SHRT_LEN + FILE_NAME_EXTN_LEN != 8 in entry->name
Co-authored-by: nerdCopter <[email protected]>
Co-authored-by: nerdCopter <[email protected]>
Co-authored-by: nerdCopter <[email protected]>
* [target] add FLYCOLORF7_AIO; needs GYRO_2 SPI * [target] mod FLYCOLORF7_AIO; remove GYRO2
removed motor 7 & 8 as no associated timers defined.
* [github] workflow updates
* [targets] TMOTORF7 / TMOTORF7_AIO - fix motor3 timer/channel * fix commas
[target] add FLYSPARKF4
* github-actions: update artifacts upload/download to v4 * remove old stuff * fix overwrite
- `set prearm_allow_rearm = ON`, default is `OFF`. - This allows re-arming mid-air, still **_requiring_** preArm, but without need for PreArm AUX _resetting_. - Arming still requires PreArm switch. - Required for mid-air rearm: `set small_angle = 180` (when using ACC) - **WARNING** - it is very hard to coordinate 0 throttle (**_required_** for arming) in mid-air.
Move applySmithPredictor function declaration inside USE_SMITH_PREDICTOR guard and guard the function call in gyro_filter_impl.h to prevent compilation errors when the Smith Predictor feature is disabled.
Replace unsafe direct void* to int cast with intptr_t intermediate type. This is platform-safe and prevents compiler warnings about casting pointer to smaller integer type. Uses C99 standard intptr_t for guaranteed portability across 32-bit and 64-bit architectures. Aligns with Betaflight implementation (proven production code). Fixes: cms_unittest compilation error Line: src/main/cms/cms.c:631 Type: Single-line change, zero functional impact
… prevent multiple definition errors (#1098) * fix: move global variable definitions from headers to source files to prevent multiple definition errors Global variables in cms.h and ledstrip.h were defined directly in headers, causing multiple definition linker errors when included in multiple translation units (e.g., unit tests). This commit changes those to 'extern' declarations in the headers and adds the actual definitions to their respective .c files, following proper C linkage rules. Fixes #1096. * refactor: make ledstrip variables static to match Betaflight pattern and improve encapsulation Remove extern declarations of colors, modeColors, and specialColors from ledstrip.h and make them static in ledstrip.c. These variables are only used internally within ledstrip.c and should not be exposed globally. This matches Betaflight's approach exactly and improves code encapsulation by reducing header pollution.
… of 4 same as uint32 (#986)
* BMI270 - 6k code ; but does not properly work * fix: sampleRate * Fix BMI270 experimental mode initialization order The BMI270 case in gyroSetSampleRate() checks gyro->hardware_lpf to determine whether to use experimental 6.4kHz or standard 3.2kHz mode. This check was happening before the struct member was initialized, causing it to always evaluate to 0 (GYRO_HARDWARE_LPF_NORMAL) instead of the configured GYRO_HARDWARE_LPF_EXPERIMENTAL value. Other IMU sensors are unaffected as they don't reference hardware_lpf during the sample rate setup phase.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
16 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.
need to set
masteras default-branch in settings too:Then continuously sync with upstream master moving forward.