Fix CMSIS-DAP packet sizes for Atmel programmers - #699
Open
aWZHY0yQH81uOYvH wants to merge 1 commit into
Open
Conversation
VID/PID table from OpenOCD
aWZHY0yQH81uOYvH
force-pushed
the
fix-atmel-cmsisdap
branch
from
August 1, 2026 04:13
0352a2e to
8977189
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The CMSIS-DAP backend does not work with certain Atmel/Microchip debuggers (constant timeouts, responses for earlier queries being returned for later ones).
Apparently, some Atmel tools use a nonstandard report size, which wreaks all kinds of havoc on the CMSIS-DAP backend. OpenOCD handles this with a table of offending VID/PIDs, allowing the initial communication to proceed correctly even before packet sizes can be queried. This fix simply sets
_pkt_szin advance if the user specifies any of the known VID/PID pairs.This now allows my Atmel-ICE debugger to work with openFPGALoader.
AI disclosure: Claude found this and wrote the patch.