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

3DR radio flashing: please let user choose firmware, ask for confirmation. #4336

Open
AndKe opened this issue Dec 22, 2016 · 8 comments
Open

Comments

@AndKe
Copy link
Contributor

AndKe commented Dec 22, 2016

For consistent experience, when 3DR radio is about to be flashed: please make the behavior more like Ardupilot flashing:

Ask user for confirmation, and offer to flash custom file, instead of just flashing right away.

@DonLakeFlyer DonLakeFlyer added this to the Release V3.2 milestone Dec 23, 2016
@DonLakeFlyer DonLakeFlyer self-assigned this Dec 23, 2016
@DonLakeFlyer
Copy link
Contributor

Also when choosing Custom Firmware for PX4/ArduPilot radio buttons should disable or hide so user isn't confused.

@DonLakeFlyer DonLakeFlyer removed their assignment Mar 4, 2017
@DonLakeFlyer DonLakeFlyer modified the milestone: Release V3.2 Apr 6, 2017
@AndKe
Copy link
Contributor Author

AndKe commented Feb 24, 2018

this is still an issue, connecting a radio, means flashing it with current SiK , instead of at least confirming that the user want that, or offer to choose a firmware file..

@DonLakeFlyer DonLakeFlyer self-assigned this Mar 13, 2018
@DonLakeFlyer DonLakeFlyer added this to the Release v3.4 milestone Mar 13, 2018
@DonLakeFlyer DonLakeFlyer modified the milestones: Release v3.4, Release v3.5 Jun 20, 2018
@DonLakeFlyer DonLakeFlyer modified the milestones: Release v3.5, Release v3.6 Jan 19, 2019
@DonLakeFlyer DonLakeFlyer modified the milestones: Release v4.0, Release V4.1 Dec 11, 2019
@myself248
Copy link

Please unplug your Pixhawk and/or Radio from USB.
Plug in your device via USB to start firmware upgrade.
Found device: SiK Radio
Putting radio into command mode
Rebooting radio to bootloader
Connected to bootloader:
Version: 0
Board ID: 130
Flash size: 0
Unable to find specified firmware for board type
Upgrade cancelled

Can the check for firmware be done in advance, and refrain from even putting the radio into bootloader mode if there isn't an appropriate firmware for it? Or, if firmware isn't found, can the radio be taken back out of bootloader mode and restored to functionality? What it's doing right now is just bricking radios and I need to go dig out a Windows box with MP to unbrick them.

(The radio in the above example is an RFD900+ by the way.)

@DonLakeFlyer
Copy link
Contributor

What it's doing right now is just bricking radios and I need to go dig out a Windows box with MP to unbrick them.

You should just need to remove power from them and they go out of command mode.

@myself248
Copy link

You should just need to remove power from them and they go out of command mode.

I wish, but the red LED is on solid no matter how many times I power cycle it, until I go find something with valid firmware for it. Numerous similar tales of woe over here: https://discuss.ardupilot.org/t/3dr-radio-in-bootloader-mode-after-failed-firmware-update/14911

@DonLakeFlyer
Copy link
Contributor

DonLakeFlyer commented Jul 3, 2020

Numerous similar tales of woe over here: https://discuss.ardupilot.org/t/3dr-radio-in-bootloader-mode-after-failed-firmware-update/14911

So three years ago for the first forum post and nobody bothers to notify QGC dev of the problem. Sigh!

@DonLakeFlyer
Copy link
Contributor

Can the check for firmware be done in advance, and refrain from even putting the radio into bootloader mode if there isn't an appropriate firmware for it?

No. The id needed to know which firmware to flash comes from the bootloader.

Or, if firmware isn't found, can the radio be taken back out of bootloader mode and restored to functionality?

Yes. Working on that.

@DonLakeFlyer
Copy link
Contributor

I believe this is supported now but SiK Radio flashing in general seems busted right now. Will verify once that's fixed.

@DonLakeFlyer DonLakeFlyer removed their assignment Nov 20, 2020
@DonLakeFlyer DonLakeFlyer removed this from the Release V4.1 milestone Nov 20, 2020
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

No branches or pull requests

3 participants