d-ronin / dRonin

The dRonin flight controller software.
http://dronin.org
Other
289 stars 167 forks source link

Enabling FrSKY sPort Non Inverted on seppuku causes bootloop when battery monitoring is not enabled #2214

Closed LitterBugs closed 6 years ago

LitterBugs commented 6 years ago

Issue details: Bootloop occurs when enabling "FrSKY sPort Non Inverted" on uart 1 or 4. When enabled, and R-XSR RX telemetry is plugged into uart1/4, Seppuku will bootloop. Unplugging telemetry pin stops the bootloop, but does not allow telemetry. Reverting back to wired stops the bootloop and telemetry works again. Tested with two different R-XSR RXes and two different Seppukus. (beta and released)

LitterBugs commented 6 years ago

Upgraded Screech build, and it works fine. Exported Screech settings and imported into the F450 build and that build now works. Reset F450 back to defaults, and reconfigured, does not work again. I've also tried all UARTs now, and it affects them all

Trying to isolate difference in UAVO between defaults and Screech settings that cause the bootloop when enabling sPort Non Inverted.

LitterBugs commented 6 years ago

Simplified testing down to 1) Flash Wired.2, enable s.bus and sPort un-inverted.
2) Upgraded to pre-inconceivable. 3) still works 4) deleted settings partition 5) enabled s.bus and sPort un-inverted. 6) doesn't work Comparing uavo exports from both....

LitterBugs commented 6 years ago

Hmmm... not quite working either....

yds commented 6 years ago

Reproduced on the FrSky XSRF4O -- an OmnibusF4SD clone with integrated XSR internally wired like so:

on Wired.2 both of the above settings work.

setting FrSky SPort Non Inverted results in a bootloop on the Pre-Inconceivable 4a4727f http://jar.lyle.org:8080/job/dronin/1800/Nodes=master/

glowtape commented 6 years ago

@LitterBugs Can you upload both UAVOs? This kind of smells like an ID mismatch somewhere.

LitterBugs commented 6 years ago

So narrowed it down to having battery monitoring enabled or not. If FrSKY telem is enabled without battery monitoring enabled, the bootloop occurs.

tracernz commented 6 years ago

Fixed in #2215