COMM: Review - round 1

modules/Communication/mcu_comm.kicad_sch

@@ -1343,14 +1343,23 @@
   (junction (at 260.35 150.495) (diameter 0.9144) (color 0 0 0 0))
   (junction (at 260.35 160.655) (diameter 0.9144) (color 0 0 0 0))
 
+  (no_connect (at 17.78 168.91) (uuid 5dcc0194-6451-4b0e-89d8-ff4be5c612f3))
+  (no_connect (at 17.78 172.72) (uuid 216d93fe-403b-4581-a39c-3bc7b3edb777))
+  (no_connect (at 17.78 193.04) (uuid 2bb93dcf-3fb8-41e9-a2aa-cf32ba54c7ba))
+  (no_connect (at 33.02 132.08) (uuid e9fa4ef4-6ae1-4259-ae77-713cffe01c05))
   (no_connect (at 33.655 44.45) (uuid c20b32bc-16b5-4a14-abcf-ed25ee2b1b4a))
+  (no_connect (at 81.28 151.13) (uuid 2205b73b-410a-4c1b-b01b-242e0f185eff))
   (no_connect (at 95.885 43.815) (uuid 58aa9bf0-2b8d-4f09-9852-144033041c8b))
   (no_connect (at 113.665 90.805) (uuid f8dfa690-166a-483b-b0e7-099c24dfd9cc))
   (no_connect (at 113.665 93.345) (uuid f8dfa690-166a-483b-b0e7-099c24dfd9cc))
   (no_connect (at 113.665 103.505) (uuid e7c1b842-5cee-4c9d-b4d6-63b5af7fc3ae))
   (no_connect (at 123.825 78.105) (uuid f8dfa690-166a-483b-b0e7-099c24dfd9cc))
+  (no_connect (at 132.08 72.39) (uuid 0846652a-932c-43a0-90cb-b326d89ae2f0))
   (no_connect (at 133.985 78.105) (uuid f8dfa690-166a-483b-b0e7-099c24dfd9cc))
   (no_connect (at 146.685 32.385) (uuid 8740f14b-f9b3-44fc-bc70-0ab09c84cf20))
+  (no_connect (at 152.4 21.59) (uuid eb09c624-7097-402f-ba1e-09aacdde77a5))
+  (no_connect (at 160.02 137.16) (uuid 61e217e3-f875-4369-b3db-768ec349b53a))
+  (no_connect (at 226.06 39.37) (uuid 290eeba7-3b69-4d3e-831b-508c61b337ce))
 
   (wire (pts (xy 17.145 180.34) (xy 32.385 180.34))
     (stroke (width 0) (type solid) (color 0 0 0 0))
@@ -2031,10 +2040,26 @@
     (effects (font (size 1.27 1.27)) (justify left bottom))
     (uuid eaa855c8-32a4-4450-9050-9d86f5fa408a)
   )
+  (text "Invalid USB connection" (at 19.05 170.18 0)
+    (effects (font (size 1.27 1.27)) (justify left bottom))
+    (uuid 5b35d8bd-2642-4e63-9d91-a60f975db0c0)
+  )
+  (text "Missing line inmpedance adjustment" (at 19.05 173.99 0)
+    (effects (font (size 1.27 1.27)) (justify left bottom))
+    (uuid 0c39b69a-9c5e-4ad6-adfc-8ea6575a3326)
+  )
+  (text "Missing transient voltage protection" (at 19.05 194.31 0)
+    (effects (font (size 1.27 1.27)) (justify left bottom))
+    (uuid 64f3828c-2401-4779-9686-c7c124c8d79a)
+  )
   (text "WATCHDOG" (at 34.925 19.05 0)
     (effects (font (size 2 2) (thickness 0.4) bold) (justify left bottom))
     (uuid a8c311b3-ed81-4119-b589-93a5c63e6751)
   )
+  (text "Intetionally empty?" (at 35.56 133.35 0)
+    (effects (font (size 1.27 1.27)) (justify left bottom))
+    (uuid 3d7011df-095e-4c4e-9a53-5ac80e79de8c)
+  )
   (text "Reset" (at 45.085 71.12 0)
     (effects (font (size 2 2) (thickness 0.4) bold) (justify left bottom))
     (uuid 480c3375-4523-48e7-98c1-d9b31b518a4c)
@@ -2047,6 +2072,10 @@
     (effects (font (size 1.27 1.27)) (justify left bottom))
     (uuid f5dcdd11-ce97-4b1f-ba59-c27cf847f283)
   )
+  (text "Inconsistent SPI Chip_select naming" (at 82.55 152.4 0)
+    (effects (font (size 1.27 1.27)) (justify left bottom))
+    (uuid a06ccb96-1253-46a8-bcbb-23359c1cab19)
+  )
   (text "High-Speed oscillator" (at 85.725 171.45 0)
     (effects (font (size 2 2) (thickness 0.4) bold) (justify left bottom))
     (uuid 7cd06a7f-06fa-4cd5-a879-1226b343440f)
@@ -2059,6 +2088,10 @@
     (effects (font (size 2 2) (thickness 0.4) bold) (justify left bottom))
     (uuid 7071d21f-a340-4b29-b9ad-54e06225002e)
   )
+  (text "VDDUSB not connected\nto USB voltage" (at 133.35 74.93 0)
+    (effects (font (size 1.27 1.27)) (justify left bottom))
+    (uuid da49d7a0-d07b-4258-a5d9-1c7f98ff11b7)
+  )
   (text "Low-Speed oscillator" (at 136.525 171.45 0)
     (effects (font (size 2 2) (thickness 0.4) bold) (justify left bottom))
     (uuid 4713ba73-7475-43e7-94f7-c9f882301c03)
@@ -2067,6 +2100,14 @@
     (effects (font (size 1.27 1.27)) (justify left bottom))
     (uuid f7d9b2ec-c0df-4662-90c8-9909c84b451c)
   )
+  (text "Missing power flag" (at 153.67 22.86 0)
+    (effects (font (size 1.27 1.27)) (justify left bottom))
+    (uuid 766d22bb-807b-492d-a247-75e552fc8513)
+  )
+  (text "USB should be\ndifferential" (at 161.29 139.7 0)
+    (effects (font (size 1.27 1.27)) (justify left bottom))
+    (uuid 46e5f71d-34ce-43f8-ba05-1401f11d9e0d)
+  )
   (text "Decoupling" (at 189.23 149.225 90)
     (effects (font (size 1.27 1.27)) (justify left bottom))
     (uuid 4d08a862-84cc-419f-9ca6-1667b50a9fe4)
@@ -2085,6 +2126,10 @@
     (effects (font (size 1.27 1.27)) (justify left bottom))
     (uuid b241b1f5-8a83-4767-948f-d4633a553b85)
   )
+  (text "Insufficient decoupling capacitors" (at 227.33 40.64 0)
+    (effects (font (size 1.27 1.27)) (justify left bottom))
+    (uuid 30ca5fa1-f582-4220-bbd1-2b26bb54cee9)
+  )
   (text "Decoupled MCU capacitors" (at 228.6 24.13 0)
     (effects (font (size 1.5 1.5)) (justify left bottom))
     (uuid 7b7658d3-0141-4ca6-bd5f-273daa58cfc3)