Skip to content

Conversation

@lucarin91
Copy link
Contributor

@lucarin91 lucarin91 commented Dec 30, 2025

Motivation

The flash grpc command needs to wait for the qdl device before starting the download.

Change description

We cannot implement a qdl read to wait for the qdl device, as the ticket says, because we don't have the firmware for the programmer, which is part of the deb image files.

The implementation uses a sort of hack that is based on the fact that qdl is waiting for a device also if we don't have the correct firmware for the programm chip in charge of the flash. So we use a dummy ELF and a valid XML file to detect when a device is connected.

We use a custom version of qdl that supports a list of edl devices.

Additional Notes

Reviewer checklist

  • PR addresses a single concern.
  • PR title and description are properly filled.
  • Changes will be merged in main.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.

@lucarin91 lucarin91 linked an issue Dec 30, 2025 that may be closed by this pull request
@lucarin91 lucarin91 marked this pull request as ready for review January 5, 2026 09:40
@lucarin91 lucarin91 requested a review from cmaglie January 5, 2026 09:41
Base automatically changed from download-grpc to main January 7, 2026 14:50
@lucarin91 lucarin91 force-pushed the add-wait-for-edl-device branch from a758d64 to 42c6ce8 Compare January 7, 2026 16:50
@lucarin91 lucarin91 force-pushed the add-wait-for-edl-device branch 2 times, most recently from 163bfb3 to 52c682e Compare January 8, 2026 17:18
@lucarin91 lucarin91 changed the base branch from main to fix-json-format-on-daemon-mode January 8, 2026 17:19
@lucarin91 lucarin91 force-pushed the fix-json-format-on-daemon-mode branch from 5c0ed73 to ed8613c Compare January 9, 2026 08:43
Base automatically changed from fix-json-format-on-daemon-mode to main January 9, 2026 09:23
@lucarin91 lucarin91 force-pushed the add-wait-for-edl-device branch from 52c682e to e166686 Compare January 9, 2026 09:24
@lucarin91 lucarin91 merged commit 9c8789a into main Jan 13, 2026
7 checks passed
@lucarin91 lucarin91 deleted the add-wait-for-edl-device branch January 13, 2026 15:52
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.

[App Lab integration] Add detection step during flash

4 participants