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

doc: Add VSCode and openOCD launcher documentation #46

Merged
merged 1 commit into from
Oct 20, 2021

Conversation

elsalahy
Copy link
Contributor

Summary

Added VSCode launcher to use openOCD in the flashing and debugging process.
Closes #9

Changes

  • Added VSCode openOCD launcher and sample images.

Notes for Reviewers

...

Checklist

  • Scope: The referenced issue is addressed, there are no unrelated changes.
  • Run Locally: Verified that the docs build using make server, posted screenshots, verified external links.
  • New Features Marked: Documentation for new features is marked using the new-in-version shortcode, according to the guidelines in CONTRIBUTING.
  • Style Guidelines: Documentation obeys style guidelines in CONTRIBUTING.
  • Commits: Commit messages follow guidelines in CONTRIBUTING, there are no fixup commits left.

@elsalahy elsalahy self-assigned this Oct 18, 2021
nejraselimovic
nejraselimovic previously approved these changes Oct 20, 2021
Comment on lines 35 to 36
1. Install [Cortex-Debug](https://marketplace.visualstudio.com/items?itemName=marus25.cortex-debug)
2. Create `launch.json` inside `.vscode` folder and add the following lines
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
1. Install [Cortex-Debug](https://marketplace.visualstudio.com/items?itemName=marus25.cortex-debug)
2. Create `launch.json` inside `.vscode` folder and add the following lines
1. Install [Cortex-Debug](https://marketplace.visualstudio.com/items?itemName=marus25.cortex-debug).
2. Create `launch.json` inside `.vscode` folder and add the following lines:

}
]
}
```
Copy link
Contributor

Choose a reason for hiding this comment

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

add a blank line after

Comment on lines 80 to 82
> **Note:** You need to update "program" and "setupCommands" fields with the application you plan to flash and debug.

> **Please also note:** You need to update "miDebuggerPath" and "debugServerPath" fields with the system paths of your tool chain gdb and openOCD binary.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
> **Note:** You need to update "program" and "setupCommands" fields with the application you plan to flash and debug.
> **Please also note:** You need to update "miDebuggerPath" and "debugServerPath" fields with the system paths of your tool chain gdb and openOCD binary.
> **Note:** You need to update `program` and `setupCommands` fields with the application you plan to flash and debug.
> **Please also note:** You need to update `miDebuggerPath` and `debugServerPath` fields with the system paths of your tool chain gdb and openOCD binary.

Copy link
Contributor

@nejraselimovic nejraselimovic Oct 20, 2021

Choose a reason for hiding this comment

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

Is it openOCD or OpenOCD? I see both usages in this doc, so please make sure to be consistent there. Also, I'm pretty sure this repo has the ability to wrap up notes (https://github.com/TheThingsIndustries/generic-node-docs/blob/master/doc/themes/generic-node/layouts/shortcodes/note.html). So instead of using:

> **Note:** ...

you should use:

{{< note >}} ... {{</ note >}}

And btw this should be replaced not just in this PR, but whole docs. Let me know if you want me to do it for you.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes you are right it's an OpenOCD.
I updated the notes in this PR and I think we can update the whole docs in another PR.
I created this issue #47

@elsalahy
Copy link
Contributor Author

Thanks @nejraselimovic for the review, can you please take anther look.

@elsalahy elsalahy merged commit e84faa5 into master Oct 20, 2021
@adriansmares adriansmares deleted the doc/openocd-debug branch August 12, 2022 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Debugging using VSCode and OpenOCD
2 participants