Rework peripheral read strategy and `peripheral-inspector.svdAddrGapThreshold`

[jreineckearm]

Type: Bug Report

Describe the bug

• OS and Version: Windows and macOS
• VS Code Version: 1.98.2
• Extension Version: 1.7.0
• Target Device: NXP FRDM-K32L3A6 Board
• Other extensions you installed (and if the issue persists after disabling them): Arm Debugger Extension, Arm Embedded Debugger Extension
• A clear and concise description of what the bug is.

Peripheral Inspector runs into a series of errors when reading from certain peripherals.

It turns out that the current SVD-based register read implementation doesn't make use all useful information to efficiently read registers from the target device. That includes access widths (currently not supported in Debug Adapter Protocol), but also a better analysis of how to bundle memory accesses into larger ones which can bring performance improvements.

As part of that, we may want to review the need for the peripheral-inspector.svdAddrGapThreshold option. It looks like it was a way to achieve similar bundling of memory accesses without more detailed analysis of the SVD nodes.

To Reproduce
Steps to reproduce the behavior:

1. Connect to the device with one of the mentioned debuggers.
2. Explore the peripheral list with different svdAddrGapThreshold values.
3. You'll see a varying number of errors of which many can be explained by not honoring such additional information from the SVD file.

Expected behavior

Better use of SVD file information and less errors.
Acceptable performance without using the svdAddrGapThreshold.
An updated svdAddrGapThreshold default value that turns it off.

Code sample and logs
N/A

Screenshots

N/A

Additional context

N/A

[jreineckearm]

@asimgunes , @markgoodchild , @thegecko , does any of you or your users have a strong opinion on keeping the svdAddrGapThreshold setting? I believe this may become obsolete depending on the outcome of properly using all available SVD information. And it might be less confusing to remove if we see this isn't useful anymore.

Any opinion is welcomed.

[jreineckearm]

Assigning to myself while @thorstendb-ARM is working on this.