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

BMI270 Support - FIFO config file 328 bytes #930

Conversation

nerdCopter
Copy link
Member

@nerdCopter nerdCopter commented Sep 27, 2023

  • WIP
  • Initial BMI270 support
  • tested as flyable
  • needs Configurator synchronization (Config wrongly shows 8k, 4k, etc. loops)
  • intend to remove target definitions from this PR
  • may need fixes
  • FIFO config file (328 bytes)
  • OSR4 mode is default

This code is ported mostly from Betaflight and minimally iNav. Thanks to project teams and GPL licensed opensource.
Intention of porting some changes from FlightNG/ATBetaflight if possible. Portability unknown at this time.
https://github.com/betaflight/betaflight/blob/master/LICENSE
https://github.com/iNavFlight/inav/blob/master/LICENSE
https://github.com/flightng/atbetaflight/blob/4.3-bsp-rebase/LICENSE

Massive thanks to @Peck07 for investigating and fixing some pre-existing failed works.
Massive thanks to @BeauBrewski for target definitions and help.

Replaces #890 / Closes #890
Resolves #861

Further notes. @Peck07 said he made it work by picking BF's "pre-SPI" changes. So we need to verify if newer code exists to make bmi270 more efficient. Should also investigate @DusKing1's bmi270 updates from https://github.com/flightng/atbetaflight or at least the more complete registers. iNav show that accgyro_bmi270_maximum_fifo.h not needed. i also moved the bmi270_maximum_fifo_config_file constant into accgyro_spi_bmi270.c file removing the need for accgyro_bmi270_maximum_fifo.c.

Co-authored-by: KarateBrot [email protected]
Co-authored-by: Steve Evans [email protected]
Co-authored-by: chrisrosser [email protected]
Co-authored-by: Dominic Clifton [email protected]
Co-authored-by: mikeller [email protected]
Co-authored-by: Bruce Luckcuck [email protected]
Co-authored-by: Martin Luessi [email protected]
Co-authored-by: Peck07 [email protected]
Co-authored-by: nerdCopter [email protected]
Co-authored-by: Dusking1 [email protected]

nerdCopter and others added 30 commits June 20, 2023 14:33
fixes requirement for mpu6000 defines
@nerdCopter nerdCopter changed the title BMI270 Support using FIFO config file BMI270 Support - FIFO config file 321 bytes Sep 27, 2023
@nerdCopter nerdCopter changed the title BMI270 Support - FIFO config file 321 bytes BMI270 Support - FIFO config file 328 bytes Sep 27, 2023
@nerdCopter nerdCopter force-pushed the 20230620_rebase_PR890_BMI270_as_FIFO_without_normal_driver branch from 3770dfa to aaf935d Compare September 28, 2023 14:26
@nerdCopter nerdCopter force-pushed the 20230620_rebase_PR890_BMI270_as_FIFO_without_normal_driver branch from c44a2b8 to dc51cc9 Compare September 29, 2023 16:31
@nerdCopter nerdCopter requested a review from a team October 27, 2023 17:58
@nerdCopter nerdCopter marked this pull request as ready for review November 1, 2023 12:48
@nerdCopter nerdCopter merged commit b1cf5d0 into emuflight:master Nov 1, 2023
6 checks passed
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.

[feature request] BMI270 support
4 participants