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] FOXEERF722V4 - MPU6000, ICM42688P #906

Closed
wants to merge 2 commits into from

Conversation

nerdCopter
Copy link
Member

@nerdCopter nerdCopter commented Jun 1, 2023

  • FOXEERF722V4 - created by @BeauBrewski (modified after the fact)
  • MPU6000
  • ICM42688P

closes #876

Co-authored-by: nerdCopter 56646290+nerdCopter@users.noreply.github.com

@nerdCopter nerdCopter changed the title FOXEERF722V4 - MPU6000, ICM42688P [Target] FOXEERF722V4 - MPU6000, ICM42688P Jun 29, 2023
@nerdCopter nerdCopter added target/new in-testing-needs-testing needs testing before merge consideration labels Jun 29, 2023
@nerdCopter
Copy link
Member Author

@nerdCopter nerdCopter force-pushed the 20230601_FOXEERF722V4_ICM42688P branch from cf56e34 to 93b191e Compare July 18, 2023 13:58
@nerdCopter
Copy link
Member Author

@mokshfpv report:
✅ .hex Flash
✅ Gyro
✅ Accelerometer
✅ Motors
untested Onboard ADC
untested OSD (analog) (due to HD/djiO3 setup)
untested Smart Audio controls (OSD and CLI)
untested Alarm switch
no Barometer on FC
✅ Blackbox flash-chip
untested LED
untested GPS

@nerdCopter
Copy link
Member Author

nerdCopter commented Jul 18, 2023

dev-talk, re #922:

  • 922 did not have #define ENSURE_MPU_DATA_READY_IS_LOW
  • 922 did have #define DEFAULT_FEATURES (FEATURE_OSD | FEATURE_TELEMETRY | FEATURE_AIRMODE | FEATURE_RX_SERIAL) and #define DEFAULT_RX_FEATURE FEATURE_RX_SERIAL
  • 922 did not seem to have other major differences, other than baro/i2c.
  • unsure why 922 failed

edits:

  • 922's .mk had F7X5XG_TARGETS += $(TARGET), but this one has F7X2RE_TARGETS += $(TARGET) <-- this was the problem that made 922 fail

@nerdCopter nerdCopter force-pushed the 20230601_FOXEERF722V4_ICM42688P branch from fcecf2b to f93cbf3 Compare August 24, 2023 15:22
@nerdCopter
Copy link
Member Author

closing this in favor of #922 since it was fixed and scripted conversion

@nerdCopter nerdCopter closed this Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in-testing-needs-testing needs testing before merge consideration target/new
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TARGET REQUEST] FOXEERF722V4 (ICM42688P)
2 participants