Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AUX1 TRACE output broken for DEBUG build. #5360

Closed
1 task done
philmoz opened this issue Aug 1, 2024 · 0 comments · Fixed by #5377
Closed
1 task done

AUX1 TRACE output broken for DEBUG build. #5360

philmoz opened this issue Aug 1, 2024 · 0 comments · Fixed by #5377
Assignees
Labels
bug/regression ↩️ A new version of EdgeTX broke something compilation Related to compiling the firmware and firmware options

Comments

@philmoz
Copy link
Collaborator

philmoz commented Aug 1, 2024

Is there an existing issue for this problem?

  • I have searched the existing issues

What part of EdgeTX is the focus of this bug?

Transmitter firmware

Current Behavior

No TRACE output is being sent to AUX1 on TX16S when built with DEBUG=YES
AUX1 serial port is no longer being initialised at startup.

Broken in PR #5232

Expected Behavior

TRACE output should be available at startup.

Steps To Reproduce

Build with DEBUG=YES on TX16S.

Version

Nightly (Please give date/commit below)

Transmitter

RadioMaster TX16S / TX16SMK2

Operating System (OS)

macOS

OS Version

No response

Anything else?

It seems that ENABLE_BOOTLOADER_DEBUG is now required to enable DEBUG code in the bootloader; but adding this prevents building due to missing functions.

Adding the missing files to the bootloader build still does not work as the DEBUG flag is not passed when compiling board.cpp.

@philmoz philmoz added bug 🪲 Something isn't working triage Bug report awaiting review / sorting labels Aug 1, 2024
@pfeerick pfeerick added compilation Related to compiling the firmware and firmware options bug/regression ↩️ A new version of EdgeTX broke something and removed triage Bug report awaiting review / sorting bug 🪲 Something isn't working labels Aug 2, 2024
@philmoz philmoz closed this as completed Aug 5, 2024
@pfeerick pfeerick linked a pull request Aug 5, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/regression ↩️ A new version of EdgeTX broke something compilation Related to compiling the firmware and firmware options
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants