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

J-Link Thread Awareness Support in Zephyr/ZMK #788

Open
schengnz opened this issue May 10, 2021 · 3 comments
Open

J-Link Thread Awareness Support in Zephyr/ZMK #788

schengnz opened this issue May 10, 2021 · 3 comments
Labels
core Core functionality/behavior of ZMK enhancement New feature or request

Comments

@schengnz
Copy link

Zephyr has recently added thread awareness support for J-Link (zephyrproject-rtos/zephyr#34660). Since J-Link is the industry standard for on-device probe, it would be great if we could roll in this support into ZMK as part of #736.

@joelspadin
Copy link
Collaborator

It looks like that just updates the west debug command to run J-Link with the correct arguments to enable the Zephyr plugin. If you are debugging through VS Code, something like this should be close to what that change uses:

{
  "debugServerPath": "C:\\Program Files (x86)\\SEGGER\\JLink\\JLinkGDBServerCL.exe",
  "debugServerArgs": "-select usb -port 2331 -if SWD -speed auto -device nRF52840_xxAA -silent -singlerun -nogui -rtos GDBServer\\RTOSPlugin_Zephyr",
}

@schengnz
Copy link
Author

We also need support for CONFIG_DEBUG_THREAD_INFO=y, which is not in 2.4 branch.

@joelspadin
Copy link
Collaborator

It looks like CONFIG_DEBUG_THREAD_INFO was added after v2.5, so we'll need to wait for 2.6 to get that.

@joelspadin joelspadin added core Core functionality/behavior of ZMK enhancement New feature or request labels Jul 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Core functionality/behavior of ZMK enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants