EdgeTX / edgetx

EdgeTX is the cutting edge open source firmware for your R/C radio
https://edgetx.org
GNU General Public License v2.0
1.59k stars 338 forks source link

[NV14] problem with external module update (ExpressLRS - R9M) #1584

Closed nightflasher closed 1 year ago

nightflasher commented 2 years ago

i've switched over to edgeTX today and wanted to update the R9M module with latest and greatest ExpressLRS 2.2.0 - but there was this "no sync" error message and the module can't be updated.

yhomega commented 2 years ago

I used a st link. Wasted too much time trying with opentx and nv14. Seems with edge tx it doesnt work too

nightflasher commented 2 years ago

@yhomega this is not correct, it worked fine with the latest version of openTX by qba667 and stefan73.

stefan73 commented 2 years ago

Yes, it works fine on the OpenFlySky branch. I guess the reason is that the ext module driver is vastly incomplete.

Look at src/targets/nv14/extmodule_helper.cpp And compare this to https://github.com/open-flysky/Flysky-OpenTX-Test-Branch/blob/pulses23/radio/src/targets/nv14/extmodule_driver.cpp

This looks mergable to me. It also means that no module flashing should currently work at all.

raphaelcoeffic commented 2 years ago

@stefan73 do you want to give it a shot? (PR?) You should know however that the external module driver has been completely re-written for horus & taranis target family, so that you might have to implement a completely different interface now. Also it should be converted to HAL/LL. Let me know if you need guidance.

bladan83 commented 1 year ago

Hi, I flashed EdgeTX 2.8 Yesterday but still the problem to flash external module... no improvements in this side? the solution seems to be in the stefan73 post, is there any developer can merge it?

raphaelcoeffic commented 1 year ago

Guys, would you mind to retest this on the latest nightly? The complete module code has been rewritten and tested quite a fair bit. At least on EL18 this work quite good for me.

raphaelcoeffic commented 1 year ago

@pfeerick should we close this one? (no feedback since Feb, and it seems to work for us)

gagarinlg commented 1 year ago

Seems to work OK for us, so it is closed. If someone still has this problem, please comment.