Skip to content

Reset new data available after reading it#69

Open
atinm wants to merge 2 commits intoqqqlab:mainfrom
atinm:atinm/ibus-bugfixes
Open

Reset new data available after reading it#69
atinm wants to merge 2 commits intoqqqlab:mainfrom
atinm:atinm/ibus-bugfixes

Conversation

@atinm
Copy link
Contributor

@atinm atinm commented Jul 4, 2025

Without the reset, we don't notice if the radio is disconnected.

Without the reset, we don't notice if the radio is disconnected.
@atinm
Copy link
Contributor Author

atinm commented Jul 4, 2025

@qqqlab found the issue with IBUS - I need to reset the data available flag after reading it, otherwise we never notice if the radio is disconnected.

@atinm
Copy link
Contributor Author

atinm commented Jul 4, 2025

Hold off on this ... still debugging ...

@atinm
Copy link
Contributor Author

atinm commented Jul 5, 2025

@qqqlab life sucks ... IBUS failsafe. I have the FlySky i6 controller running the modded firmware and I'm finding out that setting its failsafe doesn't actually set the channel to the values set in the failsafe and the receiver just keeps sending the last values to the microcontroller when it loses connection to the receiver. It does change channels 9 thru 12 to > 1500 when normally they are 1500 when things are working. So I could just hardcode that, but I hate doing that as what if someone is actually using channel 9 or 12 . . . I might have to go figure out the bug in the FS-i6 firmware first.

With the FS-i6 firmware from https://github.com/qba667/FlySkyI6,
failsafe doesn't actually change channels 1-8, it continues sending
the current values even on lost radio link. But channels 9-12 all
become > 1500 when they are 1500 before losing radio link so
hardcode checking channels 9-12 > 1500 for failsafe for radio link
lost. Hopefully we are never in a situation where channel 9-12 are
all > 1500 when radio link is lost...
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.

1 participant