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

Update BKMN-NERO.config #1291

Merged
merged 2 commits into from
Apr 2, 2024
Merged

Update BKMN-NERO.config #1291

merged 2 commits into from
Apr 2, 2024

Conversation

ahmad-kassem
Copy link
Contributor

@ahmad-kassem ahmad-kassem commented Apr 2, 2024

The old BKMN NERO F7 Rev1 target contains many incorrect resources.

I have tested all pins from the pins header back to the MCU and corrected the target accordingly.

I have added the missing resources for motors 5 to 8 I have corrected the GYRO_EXTI 1 pin.

And many more.
I have tested this target and it is fully working.

Please merge into master.

The old BKMN NERO F7 Rev1 target contains many incorrect resources. 

I have tested all pins from the pins header back to the MCU and corrected the target accordingly.

I have added the missing resources for motors 5 to 8
I have corrected the GYRO_EXTI 1 pin.

And many more.
I have tested this target and it is fully working.

Please merge into master.
@ahmad-kassem ahmad-kassem requested review from blckmn and a team as code owners April 2, 2024 05:24
configs/default/BKMN-NERO.config Outdated Show resolved Hide resolved
@haslinghuis haslinghuis merged commit ca3a659 into betaflight:master Apr 2, 2024
1 check passed
@nerdCopter
Copy link
Member

needs sync to https://github.com/betaflight/config

@nerdCopter
Copy link
Member

i was looking at this, and realized in older firmware (3.5.x), that ICM20602 is actually detected by MPU6500 code. unsure if this is still as such.
therefore wondering if

#define USE_ACC_SPI_MPU6500
#define USE_GYRO_SPI_MPU6500

could have remained. 🤷‍♂️

@haslinghuis
Copy link
Member

Think that should work. Have a clone MATEKF411 where I have to add ICM20602 (as it has the MPU6000 defined). Seems either work - only difference would be gyroHasOverflowProtection

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants