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

[Bug Report] RUSHBLADEF7/RUSHBLADEF7HD not detecting BMP280 Barometer. #672

Open
BeauBrewski opened this issue Aug 13, 2021 · 23 comments
Open

Comments

@BeauBrewski
Copy link
Collaborator

BeauBrewski commented Aug 13, 2021

Describe the bug
Flight controller supports BMP280 in Betaflight, but will not turn on in Emuflight. See bug report #415

To Reproduce
Attempt to turn on BMP280 in Emuflight.

Setup / Versions

  • RUSHFPV RUSH BLADE F722 FLIGHT CONTROLLER FOR DJI DIGITAL FPV SYSTEM
  • BMP280 barometer
    -Affects RUSHBLADEF7 and possibly RUSHBLADEF7_HD test fw.

Additional context
See #415
See below image where barometer does not show connected.
image

@JulioCesarMatias

This comment has been minimized.

@JulioCesarMatias

This comment has been minimized.

@nerdCopter
Copy link
Member

@JulioCesarMatias , this issue ticket for RUSHBLADEF7, i think you meant to post in #415

@JulioCesarMatias
Copy link
Contributor

@JulioCesarMatias , this issue ticket for RUSHBLADEF7, i think you meant to post in #415

sorry

@BeauBrewski
Copy link
Collaborator Author

@JulioCesarMatias any chance of getting a RUSHBLADEF7 fix for BMP280 too? 🙏

@JulioCesarMatias
Copy link
Contributor

@JulioCesarMatias alguma chance de conseguir uma RUSHBLADEF7correção para BMP280 também?🙏

EmuFlight_0.3.7_RUSHBLADEF7_Build_2019_2088e6f5a.zip

@nerdCopter
Copy link
Member

  • @BeauBrewski , please let me know as soon as RUSHBLADEF7 BMP280 is validated.
  • the code for both FC are in a single PR
  • i'd like to merge before setting up for 0.4.0 release so that i can "freeze" from merging for a bit.

@nerdCopter
Copy link
Member

@BeauBrewski , here is 0.3.4 version for public-testers
TESTING_BMP280_EmuFlight_0.3.4_RUSHBLADEF7_Build_52dfc2642.hex.zip

@JulioCesarMatias
Copy link
Contributor

@BeauBrewski can you tell if it worked for a RUSHBLADEF7?

@BeauBrewski
Copy link
Collaborator Author

@JulioCesarMatias I still haven't heard back from the guy who asked for it.

@BeauBrewski
Copy link
Collaborator Author

Got reply that test file 0.3.4 and new release 0 4.0 did not show the baro for RushBladeF7 and BMP280.

@JulioCesarMatias
Copy link
Contributor

Got reply that test file 0.3.4 and new release 0 4.0 did not show the baro for RushBladeF7 and BMP280.

this is weird.

1 - Is FC looking for the BMP280 automatically on the I2C bus?

2 - Is the SDA and SCL connection correct?

@BeauBrewski
Copy link
Collaborator Author

1 - I'm not sure but can try to ask.
2 - I am assuming so as it works fine with BF.
Here is screen cap of it not working on Emu w/ the 0.3.4 test build:
234296305_299578001960792_6316470820827957796_n

@JulioCesarMatias
Copy link
Contributor

1 - I'm not sure but can try to ask.
2 - I am assuming so as it works fine with BF.
Here is screen cap of it not working on Emu w/ the 0.3.4 test build:
234296305_299578001960792_6316470820827957796_n

I've also seen some boards only work the barometer with the battery connected, I don't know if this is the case

@BeauBrewski
Copy link
Collaborator Author

BeauBrewski commented Aug 23, 2021

Not showing up with battery connected also.

@JulioCesarMatias
Copy link
Contributor

Not showing up with battery connected also.

How sad, I don't know why it didn't work, I did everything correctly in PR #671

@JulioCesarMatias
Copy link
Contributor

which .hex file did you send the guy to test? Mine or NerdCopter?

@JulioCesarMatias
Copy link
Contributor

JulioCesarMatias commented Aug 24, 2021

I was downloaded and looked at NerdCopter .hex file, and the one he uploaded is smaller than mine. Are you sure you sent mine?
Captura de tela 2021-08-23 225947

@BeauBrewski
Copy link
Collaborator Author

I had sent nerdcopters .hex. Did the code make it into 0.4.0 for this fw? The tester also had tried 0.4.0 without baro showing.

@nerdCopter
Copy link
Member

nerdCopter commented Aug 24, 2021

@JulioCesarMatias , i cherry-picked your mods into older branch, because i did not want the pilot to flash something that the 0.3.6 GUI was not ready to set all features. now that 0.4.0 is out, you can merge master into your branch and send out new .hex.

however, the cherry-pick used your target-definition, so it's still an unsolved problem irregardless.

@JulioCesarMatias
Copy link
Contributor

ok @BeauBrewski and @nerdCopter , so i don't know anymore why it doesn't work.

@BeauBrewski
Copy link
Collaborator Author

BeauBrewski commented Aug 24, 2021

Crap. I had an idea/realization. This has been RushBladeF7 HD flight controller that it is being tested on. I don't know if there is any real difference between the RUSHBLADEF7 and RUSHBLADEF7HD, but that might be part of the issue.

@BeauBrewski BeauBrewski changed the title [Bug Report] RUSHBLADEF7 not detecting BMP280 Barometer. [Bug Report] RUSHBLADEF7/RUSHBLADEF7HD not detecting BMP280 Barometer. Aug 27, 2021
@xpertcrush
Copy link

I have rush blade f7 hd, if you need me to test something let me know

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

No branches or pull requests

4 participants