Skip to content

Orqa FPV.Connect support#70

Merged
JyeSmith merged 8 commits into
ExpressLRS:masterfrom
jonas-koeritz:orqa
Oct 28, 2022
Merged

Orqa FPV.Connect support#70
JyeSmith merged 8 commits into
ExpressLRS:masterfrom
jonas-koeritz:orqa

Conversation

@jonas-koeritz

@jonas-koeritz jonas-koeritz commented Jun 20, 2022

Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Contributor Author

@jonas-koeritz

Copy link
Copy Markdown
Contributor Author

@JyeSmith

Copy link
Copy Markdown
Member

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

Copy link
Copy Markdown
Member

@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

Copy link
Copy Markdown
Member

And now we wait...

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

@jonas-koeritz

Copy link
Copy Markdown
Contributor Author

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.

@km-coded

Copy link
Copy Markdown

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

Copy link
Copy Markdown
Contributor Author

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

Copy link
Copy Markdown
Member

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

@km-coded

Copy link
Copy Markdown

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

@km-coded

km-coded commented Jul 28, 2022

Copy link
Copy Markdown

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

Copy link
Copy Markdown
Contributor Author

@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_RX_Backpack_*", I don't know if changes are needed to accomodate the ESP-M3 as I don't own that module.

@JyeSmith

JyeSmith commented Aug 1, 2022

Copy link
Copy Markdown
Member

Wiring of a Jumper Rx.

image

Comment thread src/orqa.cpp
void Orqa::SendGHSTUpdate(uint16_t freq, uint8_t ghstChannel)
{
uint8_t packet[] = {
0x82, 0x0C, 0x20, // Header

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

My only comment is to make some of these header bytes defines. But Im not fussed until we start to have multiple packets e.g. DVR start/stop.

@JyeSmith

JyeSmith commented Aug 1, 2022

Copy link
Copy Markdown
Member

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

Thanks for the PR 😃

Comment thread src/Vrx_main.cpp
Comment thread src/orqa.cpp Outdated

GENERIC_CRC8 ghst_crc(GHST_CRC_POLY);

Orqa::Orqa()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Constructor does nothing... can it be removed?

Comment thread src/orqa.cpp Outdated

@wvarty wvarty left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Changes requested as per inline comments

@km-coded

km-coded commented Sep 3, 2022

Copy link
Copy Markdown

What about DVR start/stop?

@jonas-koeritz

Copy link
Copy Markdown
Contributor Author

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

@km-coded

km-coded commented Sep 5, 2022

Copy link
Copy Markdown

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

Copy link
Copy Markdown
Member

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

@JyeSmith

Copy link
Copy Markdown
Member

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

@jonas-koeritz

Copy link
Copy Markdown
Contributor Author

This should be mergeable now :)

@jonas-koeritz

Copy link
Copy Markdown
Contributor Author

If it turns out that the looping approach is more stable for others this can easily be added back in (removed it in e7e9836)

@JyeSmith

Copy link
Copy Markdown
Member

I did a final flash and check, looks good.

@JyeSmith JyeSmith merged commit 5eeab17 into ExpressLRS:master Oct 28, 2022
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.

4 participants