search

ARM-software / CMSIS_5

CMSIS Version 5 Development Repository
http://arm-software.github.io/CMSIS_5/index.html
Apache License 2.0
1.33k stars 1.08k forks source link

CMSIS/Utilities/CMSIS-SVD.xsd bitRangeType pattern #570

Open balokir opened 5 years ago

balokir commented 5 years ago

bitRangeType declared as the following:

Comment tells that it should accept values from 0 - 69 but pattern accepts [0-49:0-49] - is it typo in comment, or bug in pattern, or I miss something?
JonatanAntoni commented 5 years ago

Hi @balokir,

Thanks for highlighting this. This looks clearly like a misalignment to me. I'd rather expect values between 0 and 31, here. But perhaps we had already 64-bit registers in mind which would need a value range up to 63. Allowing values up to 69, effectively, would result from using a trivial pattern instead of a more complex one.

I'll take this on and report back as soon as it got fixed. @thorstendb-ARM, can you comment on this?

Cheers, Jonatan