search

mcu-debug / peripheral-viewer

Standalone Peripheral(SVD) Viewer extension extracted from cortex-debug, now works with any debugger
MIT License
10 stars 5 forks source link

fix read-only peripheral #30

Open kossnikita opened 8 months ago

kossnikita commented 8 months ago

fix !19

PhilippHaefele commented 8 months ago

After a quick look into SVD specifications (https://www.keil.com/pack/doc/CMSIS/SVD/html/elem_registers.html & https://www.keil.com/pack/doc/CMSIS/SVD/html/elem_special.html#registerPropertiesGroup_gr), I suspect that access rights are not writable by default and there's only an inheritance model.

So changing the default here mist likely will not fix it in an appropriate way.

I didn't look into code to be precise on this, so not really sure

kossnikita commented 8 months ago

UPD: Good catch! Please give me feedback on the new solution

kossnikita commented 8 months ago

I suspect that access rights are not writable by default

I can't find any informtion about this. Therefore, I believe that it should have read-write based on my reading of svd files, in which often access is not specified at all, although read-write is supposed