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

[Target Request] DIAT_MAMBAF405_2022B (icm42688p) #892

Closed
DutchFPV opened this issue May 19, 2023 · 16 comments · Fixed by #908
Closed

[Target Request] DIAT_MAMBAF405_2022B (icm42688p) #892

DutchFPV opened this issue May 19, 2023 · 16 comments · Fixed by #908
Assignees
Labels
feature-request in-testing-needs-testing needs testing before merge consideration target/new

Comments

@DutchFPV
Copy link

Vendor/Model

  • Please provide the target name: DIAT-MAMBAF405_2022B
  • Please provide vendor and model of the FC: Diatone MAMBAF405 MK IV 2022B (ICM42688)
  • Please provide a link to the vendor specifications.

https://www.getfpv.com/diatone-mamba-mk4-f405-mini-flight-controller-icm42688p.html

CLI dump or Unified Target

https://github.com/betaflight/unified-targets/blob/master/configs/default/DIAT-MAMBAF405_2022B.config

This is for a teammate who has the FC and is ready to test and feedback asap. Thanks!

@DutchFPV DutchFPV changed the title DIAT-MAMABAF405_2022B Target Request: DIAT-MAMABAF405_2022B May 19, 2023
@BeauBrewski BeauBrewski self-assigned this May 21, 2023
@BeauBrewski BeauBrewski changed the title Target Request: DIAT-MAMABAF405_2022B Target Request: DIAT-MAMBAF405_2022B May 21, 2023
@BeauBrewski BeauBrewski changed the title Target Request: DIAT-MAMBAF405_2022B Target Request: DIAT-MAMBAF405_2022B (icm42688p) May 21, 2023
@BeauBrewski
Copy link
Collaborator

Working on test target
#891

@nerdCopter
Copy link
Member

nerdCopter commented May 24, 2023

needs thorough testing with focus on safety

please submit blackbox with

set debug_mode = GYRO_SCALED
save

please feedback

Gyro
Accelerometer
Motors
Beeper
Onboard ADC
OSD (analog)
Smart Audio
Barometer -- Baro will not work, no DSP310 driver in Emu
Blackbox
LED
GPS

EmuFlight_0.4.2_DIAT_MAMBAF405_2022B_Build_c634207ab_ICM42688P_Analog.hex.zip

@BeauBrewski BeauBrewski added the in-testing-needs-testing needs testing before merge consideration label May 25, 2023
@nerdCopter
Copy link
Member

i have a feeling that target DIAT_MAMBAF405_2022B will not work, i've learned more about target making, but much more to learn. would still like feedback to know the status.

@nerdCopter nerdCopter changed the title Target Request: DIAT-MAMBAF405_2022B (icm42688p) Target Request: DIAT_MAMBAF405_2022B (icm42688p) May 27, 2023
@nerdCopter nerdCopter changed the title Target Request: DIAT_MAMBAF405_2022B (icm42688p) [Target Request] DIAT_MAMBAF405_2022B (icm42688p) May 27, 2023
@DutchFPV
Copy link
Author

Your hunch is correct. Target is not showing any gyro in the configurator and 0% cpu. Lemme know if you need any more details to give this another go.

@BeauBrewski
Copy link
Collaborator

@nerdCopter I see your issue. The target.h file has the gyro named as Gyro_1 in all your defines. This will not work since there is no definitions for a gyro_2 or use of dual gyros. When single gyro, have to just name it what the gyro is by type like GYRO_MPU6000 etc.

@nerdCopter
Copy link
Member

#908

@nerdCopter
Copy link
Member

nerdCopter commented Jun 1, 2023

@DutchFPV
Copy link
Author

DutchFPV commented Jun 1, 2023

Will do. This is for a teammate but he's eager to get it and test it. I'll report back with BB asap. Thanks!

@DutchFPV
Copy link
Author

DutchFPV commented Jun 5, 2023

Got an update this morning that the target is still not showing active gyro in the configurator. Let me know if I can gather any further details for you.

@BeauBrewski
Copy link
Collaborator

BeauBrewski commented Jun 30, 2023

@DutchFPV

With BF installed, in CLI can your friend type status and copy/paste results back.

Other thing that would be useful is with that last Emuflight hex, to send a copy of the dump all so we can compare the hex results to the BF config.

@nerdCopter
Copy link
Member

@DutchFPV
Copy link
Author

Is that an upgraded/updated hex? I'll see if I can get you BB from my buddy. Sorry we had both been traveling, but should have time to do this soon. I'll update asap

@nerdCopter
Copy link
Member

Is that an upgraded/updated hex?

yes, new from scratch. i've learned a bit more since first attempts.

@DutchFPV
Copy link
Author

Good news, he is reporting that it is working and detecting gyro. He's going to get BB for us at practice tomorrow night.

@DutchFPV
Copy link
Author

The official word is it "flew beautifully". Here's the bb he was able to capture...
mamba_f4_ICM.zip

@nerdCopter
Copy link
Member

The official word is it "flew beautifully". Here's the bb he was able to capture... mamba_f4_ICM.zip

Thank you both!! discord for any further needs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request in-testing-needs-testing needs testing before merge consideration target/new
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants