microsoft / vscode-embedded-tools

Official issue tracking repository for Microsoft's Embedded Tools VS Code extension.
MIT License
43 stars 3 forks source link

Peripheral View fails to load #6

Closed sullivanmj closed 2 years ago

sullivanmj commented 2 years ago

Type: Bug Report

Describe the bug

The problem is that the embedded register "peripheral view" is not initializing correctly.

To Reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior

The peripheral view should initialize.

Code sample and logs

Screenshots

Additional context

Attached file is the contents of max32665.svd: max32665.txt

aleun commented 2 years ago

Thanks for the bug report, @sullivanmj! I took a look at your SVD file, and it looks like it's got two places where it's not following the CMSIS-SVD spec, so the SVD parser is rejecting them.

I've manually fixed up the SVD, but I don't have your hardware on hand to test it out. Can you try it and let me know if it's working for you? Here's the patch: max32665.patch.txt

sullivanmj commented 2 years ago

That fixed it. Thanks a million for the quick response and for fixing that file for me. If there's anything I'd request at this point in terms of a code change, it would be an error message telling me essentially what you've just told me.

Thanks again.

aleun commented 2 years ago

Glad that worked!

If there's anything I'd request at this point in terms of a code change, it would be an error message telling me essentially what you've just told me.

I agree that we should be exposing this better. We'll make sure to add better error messages to our backlog.