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

[Bug] Boards with no PX4IO MCU play error tune on startup #22356

Closed
AlexKlimaj opened this issue Nov 13, 2023 · 4 comments · Fixed by #22368
Closed

[Bug] Boards with no PX4IO MCU play error tune on startup #22356

AlexKlimaj opened this issue Nov 13, 2023 · 4 comments · Fixed by #22368

Comments

@AlexKlimaj
Copy link
Member

AlexKlimaj commented Nov 13, 2023

Describe the bug

I am getting a constant error tune playing without a PX4IO carrier board. The error tune will play until a new tune overrides it such as the home position tune.

Caused by #22243

ERROR [px4io] PX4IO Not Supported
INFO  [tune_control] Stopping playback...
ERROR [px4io] PX4IO Not Supported
PX4IO start failed

To Reproduce

Upload current main to a flight controller that doesn't include a PX4IO MCU.

Expected behavior

No error tune on boot without a PX4IO.

Screenshot / Media

No response

Flight Log

No Flight Log

Software Version

Current main

Flight controller

ARKV6X

Vehicle type

None

How are the different components wired up (including port information)

No response

Additional context

No response

@AlexKlimaj AlexKlimaj changed the title [Bug] [Bug] Boards with no PX4IO Processor play error tune on startup Nov 13, 2023
@AlexKlimaj AlexKlimaj changed the title [Bug] Boards with no PX4IO Processor play error tune on startup [Bug] Boards with no PX4IO MCU play error tune on startup Nov 13, 2023
@julianoes
Copy link
Contributor

julianoes commented Nov 13, 2023

Which target is this? I didn't know you could have the ARKv6X without IO.

The check is based on whether the IO update file is included:

@AlexKlimaj
Copy link
Member Author

Which target is this? I didn't know you could have the ARKv6X without IO.

The check is based on whether the IO update file is included:

The IO MCU is on the carrier board. You can use any of the PAB form factor flight controllers on carrier boards with or without the IO. The ARK PAB Carrier and ARK Jetson PAB Carrier don't include the IO MCU. But you can use the ARKV6X on any of the Holybro or CUAV carriers with an IO MCU.

@julianoes
Copy link
Contributor

Right, so essentially the IO seems to be an option. So that means we need to make it "optional" which also means we can't raise an alarm if IO is not discovered when it should be there. We can't have both, bit messy.

@AlexKlimaj
Copy link
Member Author

We could extend the ver command to specially be able to query if the IO should be started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants