Fix up some typos and make these blocks generate .json

Author: nathanaelhuffman

hdl/ip/vhd/espi/BUCK

@@ -18,7 +18,7 @@ rdl_file(
 rdl_file(
     name = "espi_regs_rdl",
     src = "sys_regs/espi_regs.rdl",
-    outputs = ["espi_regs_pkg.vhd", "espi_regs.html"],
+    outputs = ["espi_regs_pkg.vhd", "espi_regs.html", "espi_regs.json"],
     visibility = ['PUBLIC']
 )
 

hdl/ip/vhd/espi/sys_regs/espi_regs.rdl

@@ -95,7 +95,7 @@ addrmap espi_regs {
         name = "Command FIFO Byte Count FIFO";
         desc = "";
         field {
-            desc = "store command byte size in fifo. Beacause we are sending 4bytes at a time into the fifo, but only reading
+            desc = "store command byte size in fifo. Because we are sending 4bytes at a time into the fifo, but only reading
             1 byte at a time, we need to store the number of cmd bytes in the fifo. This allows queueing up multiple commands
             if desired. left over data bytes are discarded if this fifo is empty and there are <4 bytes left in cmd_fifo";
         } fifo_data[7:0] =  0;

hdl/ip/vhd/info/BUCK

@@ -4,14 +4,15 @@ load("//tools:rdl.bzl", "rdl_file")
 rdl_file(
     name = "info_regs_rdl",
     src = "info_regs.rdl",
-    outputs = ["info_regs_pkg.vhd", "info_regs.html"],
+    outputs = ["info_regs_pkg.vhd", "info_regs.html", "info_regs.json"],
     visibility = ['PUBLIC']
 )
 
 # Janky generate git sha via genrule
 # There are a lot of better ways this might be done, but this was the simplest.
 # It does mean there's a re-build for any change to the git repo, but for now that's fine.
 # Longer-term, we might evaluate backannotating ROMs or something with this build info
+# This also breaks windows due to shell use
 genrule(
     name = "git_sha",
     out = "git_sha_pkg.vhd",

hdl/ip/vhd/info/info_2k8.vhd

@@ -109,8 +109,9 @@ begin
             hubris_compat <= rec_reset;
             scratchpad <= rec_reset;
         elsif rising_edge(clk) then
-            -- go ahead and flo this every cycle, it's external but not
-            -- changing
+            -- go ahead and flop this every cycle, it's external but not
+            -- changing.  Since we're not using this in decision logic, this should
+            -- suffice for synchronization.
             hubris_compat <= unpack(resize(hubris_compat_pins, 32));
             if wready then
                 case to_integer(awaddr) is