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

Create and use AP_PERIPH_IMU_ENABLED #29232

Merged
merged 2 commits into from
Feb 7, 2025

Conversation

shiv-tyagi
Copy link
Contributor

@shiv-tyagi shiv-tyagi commented Feb 5, 2025

Part of HAL_PERIPH_ENABLE_XX renaming series.

@peterbarker
Copy link
Contributor

.... still not compiling...

@shiv-tyagi
Copy link
Contributor Author

.... still not compiling...

It's compiling now. Had to make a small change in CubeNode-ETH's hwdef.

@@ -4,9 +4,12 @@ include ../CubeNode/hwdef.dat
undef PE0
undef PE1
undef PC12
undef HAL_PERIPH_ENABLE_IMU
undef AP_PERIPH_IMU_ENABLED
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Usually we'd group this with the enable, but this is fine

@peterbarker peterbarker merged commit f62e65f into ArduPilot:master Feb 7, 2025
100 checks passed
@peterbarker
Copy link
Contributor

Merged, thanks!

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.

2 participants