Merge pull request #265 from romancardenas/master

Add `riscv` section for RISC-V targets

Author: burrbull

svd-encoder/CHANGELOG.md

@@ -7,6 +7,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ## Unreleased
 
+- Add `riscv` element for configuration parameters related to RISC-V targets.
+  You must use the `unstable-riscv` feature to enable this exeperimental element.
 - Bump MSRV to 1.65.0
 
 ## [v0.14.3] - 2023-11-15

svd-encoder/Cargo.toml

@@ -11,6 +11,9 @@ rust-version = "1.65.0"
 version = "0.14.4"
 readme = "README.md"
 
+[features]
+unstable-riscv = ["svd-rs/unstable-riscv"]
+
 [dependencies]
 convert_case = "0.6.0"
 svd-rs = { version = "0.14.7", path = "../svd-rs" }

svd-encoder/src/device.rs

@@ -34,6 +34,12 @@ impl Encode for Device {
             elem.children.push(new_node("licenseText", v.clone()));
         }
 
+        #[cfg(feature = "unstable-riscv")]
+        if let Some(v) = &self.riscv {
+            elem.children
+                .push(XMLNode::Element(v.encode_with_config(config)?));
+        }
+
         if let Some(v) = &self.cpu {
             elem.children
                 .push(XMLNode::Element(v.encode_with_config(config)?));

svd-encoder/src/lib.rs

@@ -103,5 +103,7 @@ mod readaction;
 mod register;
 mod registercluster;
 mod registerproperties;
+#[cfg(feature = "unstable-riscv")]
+mod riscv;
 mod usage;
 mod writeconstraint;

svd-encoder/src/riscv.rs

@@ -0,0 +1,70 @@
+use super::{new_node, Config, Element, Encode, EncodeError, XMLNode};
+use crate::svd::riscv::{Hart, Priority, Riscv};
+
+impl Encode for Riscv {
+    type Error = EncodeError;
+
+    fn encode_with_config(&self, config: &Config) -> Result<Element, EncodeError> {
+        let mut elem = Element::new("riscv");
+
+        if !self.core_interrupts.is_empty() {
+            let mut interrupts = Element::new("coreInterrupts");
+            for interrupt in &self.core_interrupts {
+                interrupts
+                    .children
+                    .push(interrupt.encode_node_with_config(config)?);
+            }
+            elem.children.push(XMLNode::Element(interrupts));
+        }
+        if !self.priorities.is_empty() {
+            let mut priorities = Element::new("priorities");
+            for priority in &self.priorities {
+                priorities
+                    .children
+                    .push(priority.encode_node_with_config(config)?);
+            }
+            elem.children.push(XMLNode::Element(priorities));
+        }
+        if !self.harts.is_empty() {
+            let mut harts = Element::new("harts");
+            for hart in &self.harts {
+                harts.children.push(hart.encode_node_with_config(config)?);
+            }
+            elem.children.push(XMLNode::Element(harts));
+        }
+
+        Ok(elem)
+    }
+}
+
+impl Encode for Priority {
+    type Error = EncodeError;
+
+    fn encode_with_config(&self, _config: &Config) -> Result<Element, EncodeError> {
+        let mut children = vec![new_node("name", self.name.clone())];
+        if let Some(desc) = &self.description {
+            children.push(new_node("description", desc.clone()));
+        }
+        children.push(new_node("value", format!("{}", self.value)));
+
+        let mut elem = Element::new("priority");
+        elem.children = children;
+        Ok(elem)
+    }
+}
+
+impl Encode for Hart {
+    type Error = EncodeError;
+
+    fn encode_with_config(&self, _config: &Config) -> Result<Element, EncodeError> {
+        let mut children = vec![new_node("name", self.name.clone())];
+        if let Some(desc) = &self.description {
+            children.push(new_node("description", desc.clone()));
+        }
+        children.push(new_node("value", format!("{}", self.value)));
+
+        let mut elem = Element::new("hart");
+        elem.children = children;
+        Ok(elem)
+    }
+}

svd-parser/CHANGELOG.md

@@ -7,6 +7,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ## Unreleased
 
+- Add `riscv` element for configuration parameters related to RISC-V targets.
+  You must use the `unstable-riscv` feature to enable this exeperimental element.
 - Bump MSRV to 1.65.0
 
 ## [v0.14.5] - 2024-01-03

svd-parser/Cargo.toml

@@ -17,6 +17,7 @@ readme = "README.md"
 [features]
 derive-from = ["svd-rs/derive-from"]
 expand = ["derive-from"]
+unstable-riscv = ["svd-rs/unstable-riscv"]
 
 [dependencies]
 svd-rs = { version = "0.14.7", path = "../svd-rs" }

svd-parser/src/device.rs

@@ -1,4 +1,6 @@
 use super::*;
+#[cfg(feature = "unstable-riscv")]
+use crate::svd::riscv::Riscv;
 use crate::svd::{cpu::Cpu, peripheral::Peripheral, registerproperties::RegisterProperties};
 
 /// Parses a SVD file
@@ -31,6 +33,10 @@ impl Parse for Device {
                     .collect();
                 ps?
             });
+        #[cfg(feature = "unstable-riscv")]
+        if let Some(riscv) = optional::<Riscv>("riscv", tree, config)? {
+            device = device.riscv(riscv);
+        }
         if let Some(version) = tree.get_child_text_opt("version")? {
             device = device.version(version)
         }

svd-parser/src/lib.rs

@@ -211,6 +211,8 @@ mod readaction;
 mod register;
 mod registercluster;
 mod registerproperties;
+#[cfg(feature = "unstable-riscv")]
+mod riscv;
 mod usage;
 mod writeconstraint;
 

svd-parser/src/riscv.rs

@@ -0,0 +1,85 @@
+use super::*;
+use crate::svd::riscv::{Hart, Interrupt, Priority, Riscv};
+
+impl Parse for Riscv {
+    type Object = Self;
+    type Error = SVDErrorAt;
+    type Config = Config;
+
+    fn parse(tree: &Node, config: &Config) -> Result<Self, Self::Error> {
+        if !tree.has_tag_name("riscv") {
+            return Err(SVDError::NotExpectedTag("riscv".to_string()).at(tree.id()));
+        }
+
+        let mut builder = Riscv::builder();
+
+        if let Some(interrupts) = tree.get_child("coreInterrupts") {
+            let interrupts: Result<Vec<_>, _> = interrupts
+                .children()
+                .filter(|t| t.is_element() && t.has_tag_name("interrupt"))
+                .map(|i| Interrupt::parse(&i, config))
+                .collect();
+            builder = builder.core_interrupts(interrupts?);
+        }
+
+        if let Some(priorities) = tree.get_child("priorities") {
+            let priorities: Result<Vec<_>, _> = priorities
+                .children()
+                .filter(|t| t.is_element() && t.has_tag_name("priority"))
+                .map(|i| Priority::parse(&i, config))
+                .collect();
+            builder = builder.priorities(priorities?);
+        };
+
+        if let Some(harts) = tree.get_child("harts") {
+            let harts: Result<Vec<_>, _> = harts
+                .children()
+                .filter(|t| t.is_element() && t.has_tag_name("hart"))
+                .map(|i| Hart::parse(&i, config))
+                .collect();
+            builder = builder.harts(harts?);
+        };
+
+        builder
+            .build(config.validate_level)
+            .map_err(|e| SVDError::from(e).at(tree.id()))
+    }
+}
+
+impl Parse for Priority {
+    type Object = Self;
+    type Error = SVDErrorAt;
+    type Config = Config;
+
+    fn parse(tree: &Node, config: &Config) -> Result<Self, Self::Error> {
+        if !tree.has_tag_name("priority") {
+            return Err(SVDError::NotExpectedTag("priority".to_string()).at(tree.id()));
+        }
+
+        Priority::builder()
+            .name(tree.get_child_text("name")?)
+            .description(tree.get_child_text_opt("description")?)
+            .value(tree.get_child_u32("value")?)
+            .build(config.validate_level)
+            .map_err(|e| SVDError::from(e).at(tree.id()))
+    }
+}
+
+impl Parse for Hart {
+    type Object = Self;
+    type Error = SVDErrorAt;
+    type Config = Config;
+
+    fn parse(tree: &Node, config: &Config) -> Result<Self, Self::Error> {
+        if !tree.has_tag_name("hart") {
+            return Err(SVDError::NotExpectedTag("hart".to_string()).at(tree.id()));
+        }
+
+        Hart::builder()
+            .name(tree.get_child_text("name")?)
+            .description(tree.get_child_text_opt("description")?)
+            .value(tree.get_child_u32("value")?)
+            .build(config.validate_level)
+            .map_err(|e| SVDError::from(e).at(tree.id()))
+    }
+}