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

CMSIS DAP documentation has incorrect multidrop example #1608

Open
swetland opened this issue Feb 28, 2023 · 0 comments
Open

CMSIS DAP documentation has incorrect multidrop example #1608

swetland opened this issue Feb 28, 2023 · 0 comments
Assignees

Comments

@swetland
Copy link

swetland commented Feb 28, 2023

The documentation for the DAP_SWD_Sequence command provides a diagram of multi-drop target selection as an example of its use:
https://arm-software.github.io/CMSIS_5/DAP/html/group__DAP__SWD__Sequence.html

incorrect-sequence

Unfortunately, this sequence is incorrect. It claims to be a DP TARGETSEL register write (10011001) but it is actually a register read (10111101). The ARM Debug Interface Architecture Spec (IHI0031G) provides the correct sequence on page 124 in section B4.3. A copy is attached.

correct-sequence

I'm filing this bug in hopes that the next person to foolishly use the diagram in the CMSIS DAP docs as a guide when writing debug probe control software might avoid spending a bunch of time wondering why things were not working... as I did.

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

No branches or pull requests

3 participants