data-gen: verify pin names with dot do not exist

Author: i509VCB

mspm0-data-gen/src/generate.rs

@@ -65,7 +65,9 @@ pub fn generate(
             dma_channels,
         };
 
-        verify::verify(&chip, &name)?;
+        if let Err(err) = verify::verify(&chip, &name) {
+            eprintln!("{err}");
+        };
 
         let _ = fs::write(
             format!("./build/data/{name}.json"),
@@ -258,6 +260,10 @@ fn generate_peripherals2(
                                 .unwrap_or_else(|| &device_pin_name)
                                 .to_string();
 
+                            if skip_peripheral_pin(device_pin_name, chip_name) {
+                                continue;
+                            }
+
                             peri.pins.push(PeripheralPin {
                                 pin,
                                 signal: String::from(signal),
@@ -684,3 +690,15 @@ fn generate_dma_channels(
 
     Ok(channels)
 }
+
+fn skip_peripheral_pin(pin_name: &String, chip_name: &str) -> bool {
+    // L130X and L134X defines some device pins that only contain `OPAx.IN0-`, which is one of the symbols. Not the pin
+    // itself.
+    if (chip_name == "MSPM0L130X" || chip_name == "MSPM0L134X")
+        && (pin_name == "OPA0.IN0-" || pin_name == "OPA1.IN0-")
+    {
+        return true;
+    }
+
+    false
+}

mspm0-data-gen/src/verify.rs

@@ -70,12 +70,22 @@ fn core_peripherals(chip: &Chip, name: &str) -> anyhow::Result<()> {
 fn pin_names(chip: &Chip, name: &str) -> anyhow::Result<()> {
     for peripheral in chip.peripherals.values() {
         for pin in peripheral.pins.iter() {
+            // `+` and `-` are allowed
+            //
+            // `/` and `.` are not allowed, as these are likely bugs in generation.
             if pin.pin.contains('/') {
                 let peripheral_name = &peripheral.name;
                 let pin = &pin.pin;
 
                 bail!("{name}, {peripheral_name}: pin {pin} contains a '/'");
             }
+
+            if pin.pin.contains('.') {
+                let peripheral_name = &peripheral.name;
+                let pin = &pin.pin;
+
+                bail!("{name}, {peripheral_name}: pin {pin} contains invalid characters");
+            }
         }
     }