tx, crsf, send device info upon ping device

[olliw42]

attempt to account for #297

not tested, just did some code

@jlpoltrack maybe you find a chance to test this out :)

[jlpoltrack]

Not working at the moment. This appears to send as a 'Broadcast Frame' but needs to be sent as a 'Extended Header Frame' with the addresses.

[jlpoltrack]

Got it working, will PR to your branch.

[olliw42]

@jlpoltrack
MANY thx for your PR :)
added it, and tidied it up
could you pl double check/test again?

It seems that EdgeTx cuts the name down to 15 chars max ... what do you see actually displayed now?
how many chars? is there some version info?

[olliw42]

I guess it would be nice if we also could get the bind button LOL

[jlpoltrack]

could you pl double check/test again?

Still working :)

It seems that EdgeTx cuts the name down to 15 chars max ... what do you see actually displayed now?

In Settings > Version > Modules I see the string 'MLRS BetaFPV Mi V40.0.0'. For the version, the closest that I can get is to send '0x05030100' which results in V1.3.5' on the radio (but can't quite see an easy way to convert the VERSION to that).

[olliw42]

thx
so, we probably want to not add the MLRS at the beginning of th ename, I guess
concerning the version, I was wondering if it is part of the name, or generated from the firmwareid field, looks like it's generated ...

[jlpoltrack]

Well IMO probably doesn't need to be perfect, it's rather buried in the settings. I had to look at the EdgeTx source to figure out where this would be...

[olliw42]

@jlpoltrack
added some code to - hopefully - do the version correctly.
also stripped the name
Could you pl test?

also did the serial_no ... EdgeTx seems to use it to identfy ELRS modules, so, we could use the same mechanism to - maybe at some point - make EdgeTx to identify mLRS, e.g., to also give us a bind button
https://github.com/EdgeTX/edgetx/blob/main/radio/src/telemetry/crossfire.cpp#L295-L296

[jlpoltrack]

added some code to - hopefully - do the version correctly.

Yea, looks good. Shows 'V1.3.5'

[olliw42]

so, ready to merge?

[jlpoltrack]

Yea, good to merge.

[olliw42]

many thx, sir