ExpressLRS / Backpack

Firmware for ExpressLRS compatible backpacks
GNU General Public License v3.0
100 stars 73 forks source link

Orqa FPV.Connect support #70

Closed jonas-koeritz closed 2 years ago

jonas-koeritz commented 2 years ago

Adds the option to use an ESP based receiver in place of the Ghost Atto as a backpack on the FPV.Connect module. Configure the Goggles as if the Ghost integration is used (enable SPI mode for RapidFire, Enable Ghost Integration, Bridge solder pads on the back of FPV.Connect etc.).

The Pinout doesn't match and the Rx might need 5V to function. I will upload pictures and some documentation on how to connect the Rx correctly. I didn't want to include a SoftwareSerial dependency for something that needs soldering anyways.

Special thanks to @Igor-Misic for the captures of the original Ghost integration!

jonas-koeritz commented 2 years ago

Added some documentation https://github.com/ExpressLRS/Backpack/wiki/Orqa-FPV.Connect-Wiring

jonas-koeritz commented 2 years ago

Accompanying blog post: https://blog.jonaskoeritz.de/2022/06/20/expresslrs-integration-for-orqa-fpv-connect/

JyeSmith commented 2 years ago

Very cool and thanks for the write up! and you to @Igor-Misic!

I asked Orqa about this a while back but they didnt want to play with us LOL

Does ghost/connect have the ability to start/stop DVR yet?

JyeSmith commented 2 years ago

@jonas-koeritz have you looked at the pinout on the ribbon cable, and do the Backpack pins trace straight back? If you are on discord please ping me. Itll be easier to chat there.

JyeSmith commented 2 years ago

And now we wait...

https://orqafpv.freshdesk.com/support/discussions/topics/48000564048

jonas-koeritz commented 2 years ago

They would need to add this to Ghost. Gonna probe the module later, I think the Connect thing isn't even playing a role in this.

nightflasher commented 2 years ago

the picture in the wiki isn't clear for me, maybe for others too. can you do another picture where the RX and the fpv.connect are connected with colored lines? instead of using a rx, you could have an HM EP82 or an ESP-M3 (needs 3v3)?

jonas-koeritz commented 2 years ago

Do you know of a good software to create a clearer graphical representation of the connections? It would certainly be possible to use another module for that purpose, I used the Rx because I had it at hand and didn't want to buy additional hardware.

JyeSmith commented 2 years ago

I have a Connect arriving in a couple of days. I'll grab some more pics.

nightflasher commented 2 years ago

I've made one for the ttgo variants with gimp, m$ paint is fine too.

nightflasher commented 2 years ago

as i understand the wiring diagram, i've created a quick'n'dirty version - delete it, if it's wrong. which backpack target should be flashed? The ESP-M3 in this case is powered by 3v3. orqa_connect_esp-m3_elrsbp

jonas-koeritz commented 2 years ago

@nightflasher That diagram is correct, as long as the module runs on 3.3V. The additional wire on my photo is used to get +5V to the Rx. The targets I added to the codebase are called "Orqa_ESP_RXBackpack*", I don't know if changes are needed to accomodate the ESP-M3 as I don't own that module.

JyeSmith commented 2 years ago

Wiring of a Jumper Rx.

image

JyeSmith commented 2 years ago

Working well with the Connect. Itll be super nice once there is DVR control!

Thanks for the PR 😃

nightflasher commented 2 years ago

What about DVR start/stop?

jonas-koeritz commented 2 years ago

DVR control isn't supported by the Ghost integration by Orqa. So it is currently not possible through ELRS either.

nightflasher commented 2 years ago

Damn it! Why? What are they doing over there at orqa? That was the main reason I've initially bought into ghost and orqa...so still waiting.

Nice work anyway!

JyeSmith commented 2 years ago

@jonas-koeritz lets get this rebased, Wezs comments addressed and merged!

JyeSmith commented 2 years ago

@jonas-koeritz do you have time to finalise this PR?

jonas-koeritz commented 2 years ago

This should be mergeable now :)

jonas-koeritz commented 2 years ago

If it turns out that the looping approach is more stable for others this can easily be added back in (removed it in https://github.com/ExpressLRS/Backpack/pull/70/commits/e7e983608d05d8a97cb1d0c3ae6e9b4f3129ae61)

JyeSmith commented 2 years ago

I did a final flash and check, looks good.