Skip to content

Fixed Bug, Channel 1 was being mixed with the header byte#30

Merged
Witty-Wizard merged 1 commit into
26-bug-ibus-not-working-with-inverted-rx-on-rp2040-mcufrom
29-bug-ibus-header-field-mixed-with-first-channel
Dec 6, 2024
Merged

Fixed Bug, Channel 1 was being mixed with the header byte#30
Witty-Wizard merged 1 commit into
26-bug-ibus-not-working-with-inverted-rx-on-rp2040-mcufrom
29-bug-ibus-header-field-mixed-with-first-channel

Conversation

@Witty-Wizard
Copy link
Copy Markdown
Owner

There was a mistake in code where header was assigned the LSB of channel 1

Original Code

_channelData.header = (_rxData[2] << 8) | _rxData[0];

Fixed COde

_channelData.header = (_rxData[1] << 8) | _rxData[0];

@Witty-Wizard Witty-Wizard added the bugfixes 🐞 Address bugs label Dec 6, 2024
@Witty-Wizard Witty-Wizard self-assigned this Dec 6, 2024
@Witty-Wizard Witty-Wizard linked an issue Dec 6, 2024 that may be closed by this pull request
@Witty-Wizard Witty-Wizard changed the base branch from main to 26-bug-ibus-not-working-with-inverted-rx-on-rp2040-mcu December 6, 2024 10:47
@Witty-Wizard Witty-Wizard merged commit b84230e into 26-bug-ibus-not-working-with-inverted-rx-on-rp2040-mcu Dec 6, 2024
@Witty-Wizard Witty-Wizard deleted the 29-bug-ibus-header-field-mixed-with-first-channel branch December 6, 2024 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfixes 🐞 Address bugs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🪲Bug: ibus header field mixed with first channel

1 participant