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.

Sign up for GitHub

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

Jump to bottom

Rp2040 jlink #19

Closed
wants to merge 2 commits into from
Closed

Rp2040 jlink #19

wants to merge 2 commits into from

Conversation

kilograham
Copy link

Seems to work for people in the wild

@czak
Copy link

czak commented May 28, 2021 •
edited
Loading

I can confirm this branch is working with a J-Link EDU 11.0 using the following wiring:

jlink-pico

And the following configuration script:

source [find interface/jlink.cfg]
transport select swd
adapter speed 1000

source [find target/rp2040.cfg]
targets rp2040.core0
program build/blink.elf verify reset exit

The line targets rp2040.core0 is (currently, as of 7e5ea18) needed because this branch is 1 commit behind rp2040 and missing f8e14ec in target/rp2040.cfg.

lurch
lurch reviewed May 28, 2021
View reviewed changes
src/jtag/drivers/jlink.c
@@ -2002,6 +2002,8 @@ struct pending_scan_result {
void *buffer;
/** Offset in the destination buffer */
unsigned buffer_offset;
/** true if the command has nmo acknowledgement */
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nmo -> no?

@kilograham kilograham closed this Oct 26, 2021
@kilograham kilograham deleted the rp2040_jlink branch October 26, 2021 22:27
@kilograham
Copy link
Author

this was already merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lurch lurch lurch left review comments

@benjacam benjacam benjacam approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@kilograham @czak @lurch @benjacam @liamfraser