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

hwdef: GEPRCF745BTHD: rely on custom builds for non-BMP280 baros #27999

Merged

Conversation

peterbarker
Copy link
Contributor

this is overflowing - it has an onboard baro, that will have to do..

@andyp1per @Hwurzburg @YI-BOYANG this board did not compile for ArduPlane when support was merged in #27491

@peterbarker peterbarker force-pushed the pr/GEPRC745THD-compilation branch from 384c1b8 to 83879f0 Compare September 4, 2024 02:59
@peterbarker peterbarker changed the title hwdef: GEPRCF745BTHD: remove support for external Baro hwdef: GEPRCF745BTHD: rely on custom builds for non-BMP280 baros Sep 4, 2024
@YI-BOYANG
Copy link
Contributor

Is it possible to add DPS310 and MS5611 drivers? Because our GPS products integrate these barometers.

this is overflowing - it has an onboard baro, that will have to do...
@peterbarker peterbarker force-pushed the pr/GEPRC745THD-compilation branch from 83879f0 to 7ebfb48 Compare September 4, 2024 05:07
@peterbarker
Copy link
Contributor Author

Is it possible to add DPS310 and MS5611 drivers? Because our GPS products integrate these barometers.

Yes! I've added lines to include those and adjusted the comment.

@YI-BOYANG
Copy link
Contributor

Is it possible to add DPS310 and MS5611 drivers? Because our GPS products integrate these barometers.

Yes! I've added lines to include those and adjusted the comment.

Thanks!

@andyp1per
Copy link
Collaborator

I have one of these so can test now.

@peterbarker peterbarker merged commit bfcac18 into ArduPilot:master Sep 5, 2024
41 checks passed
@peterbarker peterbarker deleted the pr/GEPRC745THD-compilation branch September 5, 2024 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants