pyocd / cmsis-pack-manager

A Rust and Python module for handling CMSIS Pack files
Apache License 2.0
27 stars 34 forks source link

No support for attribute `Pname` on `Memory` element for multi-processor devices. #191

Closed noppej closed 1 year ago

noppej commented 2 years ago

Hi,

According to https://open-cmsis-pack.github.io/Open-CMSIS-Pack-Spec/main/html/pdsc_family_pg.html#element_memory, there is an attribute Pname on the Memory element, with description Processor identifier. This attribute is mandatory for devices that embed multiple processors. Only alphabetical characters, decimal digits, '-' and '_' are allowed.

As far as I can tell, this attribute is currently not represented when processing a device's memories.

Would you be willing to accept a PR that adds this into the rust cmsis_pack::pdsc::device::Memory struct?