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

initial BMI270 support - NORMAL config file 8192 bytes #890

Closed

Conversation

nerdCopter
Copy link
Member

@nerdCopter nerdCopter commented May 15, 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
  • NORMAL config file (8192 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 "CRT/IOC/CAS" 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.

Closes #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

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

This comment was marked as outdated.

@nerdCopter nerdCopter force-pushed the 20230515_rebase_master_Peck07_20220726_BMI270_targets branch from aaac692 to 0720a81 Compare May 18, 2023 16:39
@nerdCopter nerdCopter force-pushed the 20230515_rebase_master_Peck07_20220726_BMI270_targets branch from 0720a81 to 353be17 Compare May 18, 2023 16:44
@DusKing1
Copy link

I'll look into this once I got some spare time...

@nerdCopter nerdCopter force-pushed the 20230515_rebase_master_Peck07_20220726_BMI270_targets branch from b0b8d1b to 748035a Compare June 27, 2023 18:47
@nerdCopter nerdCopter force-pushed the 20230515_rebase_master_Peck07_20220726_BMI270_targets branch from 646acf5 to 4234039 Compare July 11, 2023 19:23
@nerdCopter
Copy link
Member Author

BMI270_INT_EXTI is not a valid define.

  • could add parameter at src/main/sensors/gyro.c:688 or use existing GYRO_1_EXTI_PIN or MPU_INT_EXTI

off-subject, same is true for non-existent ICM42688P_EXTI_PIN

@nerdCopter
Copy link
Member Author

  • found that mpu6000 drivers and defines are required for bm270i-only target to work.
  • will need to rectify in firmware source-code.

@nerdCopter
Copy link
Member Author

  • defining USE_GYRO_EXTI (and USE_MPU_DATA_READY_SIGNAL) breaks linking stage.
  • have not found a need for it yet, however. - needs investigation.

@nerdCopter nerdCopter force-pushed the 20230515_rebase_master_Peck07_20220726_BMI270_targets branch from 8975d51 to 136f035 Compare August 28, 2023 19:10
@nerdCopter nerdCopter changed the title initial BMI270 support initial BMI270 support - NORMAL config file 8192 bytes Sep 27, 2023
@nerdCopter
Copy link
Member Author

Closing this in favor of FIFO. #930

@nerdCopter nerdCopter closed this Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feature request] BMI270 support
4 participants