Clean up option descriptions and unused code

src/Harp.Toolkit/Generate/GenerateRegisterMetadataCommand.cs

@@ -15,7 +15,7 @@ public GenerateRegisterMetadataCommand()
         Argument<FileInfo> registerWorksheetPathArgument = ArgumentValidation.AcceptExistingOnly(
             new Argument<FileInfo>("registers.xls")
             {
-                Description = "The path to the file describing the device registers.",
+                Description = "The path to the file that describes the device registers.",
                 Arity = ArgumentArity.ExactlyOne
             });
 

src/Harp.Toolkit/Generate/GeneratorHelper.cs

@@ -1,18 +1,10 @@
 using System.CodeDom.Compiler;
 using System.Text;
-using Harp.Generators;
 
 namespace Harp.Toolkit.Generate;
 
 public static class GeneratorHelper
 {
-    public static IEnumerable<KeyValuePair<string, T>> GetPortPinsOfType<T>(IDictionary<string, PortPinInfo> portPins) where T : PortPinInfo
-    {
-        return from item in portPins
-               where item.Value is T
-               select new KeyValuePair<string, T>(item.Key, (T)item.Value);
-    }
-
     public static bool AssertNoGeneratorErrors(CompilerErrorCollection errors)
     {
         if (errors.Count > 0)

src/Harp.Toolkit/Generate/IOMetadataPathOption.cs

@@ -8,7 +8,7 @@ public IOMetadataPathOption()
         : base("--ios")
     {
         OptionValidation.AcceptExistingOnly(this);
-        Description = "The path to the file describing the device IO pins.";
+        Description = "The path to the file that describes the device IO pins.";
         DefaultValueFactory = result => result.AcceptExistingOnly(new FileInfo("ios.yml"));
     }
 }

src/Harp.Toolkit/Generate/MetadataPathArgument.cs

@@ -8,7 +8,7 @@ public MetadataPathArgument()
         : base("metadata")
     {
         ArgumentValidation.AcceptExistingOnly(this);
-        Description = "The path to the file describing the device registers.";
+        Description = "The path to the file that describes the device registers.";
         DefaultValueFactory = result => result.AcceptExistingOnly(new FileInfo("device.yml"));
         Arity = ArgumentArity.ZeroOrOne;
     }

src/Harp.Toolkit/Verify/VerifyCommand.cs

@@ -33,7 +33,7 @@ public VerifyCommand()
 
         Option<int?> ppsEventOption = new("--pps-event")
         {
-            Description = "Address of the register on the tested device (--port) that reports the incoming PPS pulse from the reference clock device. Enables the PPS alignment test, which also requires --clock-port.",
+            Description = "Address of the register on the tested device, given by --port, that reports the incoming PPS pulse from the reference clock device. Enables the PPS alignment test, which also requires --clock-port.",
             Required = false,
         };
 
@@ -51,7 +51,7 @@ public VerifyCommand()
 
         Option<FileInfo> metadataOption = new("--metadata")
         {
-            Description = "The path to the file describing the device registers. Enables validation of the generated interface against a live read of every declared register, and cross-checks the WhoAmI, firmware and hardware versions.",
+            Description = "The path to the file that describes the device registers. Enables validation of the generated interface against a live read of every declared register, and cross-checks the WhoAmI, firmware and hardware versions.",
             Required = false,
         };
         OptionValidation.AcceptExistingOnly(metadataOption);
@@ -92,8 +92,8 @@ static async Task<int> RunVerification(string portName, FileInfo? reportFile, bo
         if (metadataPath is not null)
         {
             AnsiConsole.Markup($"Loading device metadata from [bold]{metadataPath.FullName}[/]...");
-            deviceMetadata = DeviceMetadata.Load(metadataPath.FullName);
             deviceRawYaml = await File.ReadAllTextAsync(metadataPath.FullName, cancellationToken);
+            deviceMetadata = DeviceMetadata.Parse(deviceRawYaml);
             AnsiConsole.MarkupLine($" [green]Done![/] ({deviceMetadata.Registers.Count} registers)");
         }