Roflin/rebbarb
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add REVIEW.md: complete hardware-review coverage record

Browse Source 
Documents everything examined in the 2026-07-17/18 review - not just
findings but full coverage with verdicts, so future sessions know what
not to re-review: per-sheet verification against datasheets and the
WIZnet reference design, the settled look-wrong-but-correct list
(VPHY/VPLL on 3.3V, D4/D5 floating VBUS + flow-through, EN at 12V,
RSET 12.4k, no CT cap, DO/DI direction evidence), PCB/JLC/BOM audit
state, explicit not-reviewed list (bench/gateware items), and a binding
rule to keep the file updated in the same commit as any change to a
reviewed circuit. Referenced from CLAUDE.md alongside TODO.md.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
Dennis Brentjes
2026-07-19 07:02:30 +00:00
committed by Dennis Brentjes
parent d0426b38fe
commit a86aa255f1
9 changed files with 10118 additions and 3936 deletions
Download Patch File Download Diff File Expand all files Collapse all files
hardware/re-bba-rb/ethernet.kicad_sch
+386 -398
View File
@@ -5,6 +5,129 @@
(uuid "a88c6b60-53c5-4c5b-a4f8-dc5df82b7e8a")
(paper "A4")
(lib_symbols
(symbol "Connector:TestPoint"
(pin_numbers
(hide yes)
)
(pin_names
(offset 0.762)
(hide yes)
)
(exclude_from_sim no)
(in_bom yes)
(on_board yes)
(in_pos_files yes)
(duplicate_pin_numbers_are_jumpers no)
(property "Reference" "TP"
(at 0 6.858 0)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Value" "TestPoint"
(at 0 5.08 0)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Footprint" ""
(at 5.08 0 0)
(show_name no)
(do_not_autoplace no)
(hide yes)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Datasheet" ""
(at 5.08 0 0)
(show_name no)
(do_not_autoplace no)
(hide yes)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Description" "test point"
(at 0 0 0)
(show_name no)
(do_not_autoplace no)
(hide yes)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "ki_keywords" "test point tp"
(at 0 0 0)
(show_name no)
(do_not_autoplace no)
(hide yes)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "ki_fp_filters" "Pin* Test*"
(at 0 0 0)
(show_name no)
(do_not_autoplace no)
(hide yes)
(effects
(font
(size 1.27 1.27)
)
)
)
(symbol "TestPoint_0_1"
(circle
(center 0 3.302)
(radius 0.762)
(stroke
(width 0)
(type default)
)
(fill
(type none)
)
)
)
(symbol "TestPoint_1_1"
(pin passive line
(at 0 0 90)
(length 2.54)
(name "1"
(effects
(font
(size 1.27 1.27)
)
)
)
(number "1"
(effects
(font
(size 1.27 1.27)
)
)
)
)
)
(embedded_fonts no)
)
(symbol "Device:C"
(pin_numbers
(hide yes)
@@ -6517,130 +6640,6 @@
)
(embedded_fonts no)
)
(symbol "Connector:TestPoint"
(pin_numbers
(hide yes)
)
(pin_names
(offset 0.762)
(hide yes)
)
(exclude_from_sim no)
(in_bom yes)
(on_board yes)
(in_pos_files yes)
(duplicate_pin_numbers_are_jumpers no)
(property "Reference" "TP"
(at 0 6.858 0)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Value" "TestPoint"
(at 0 5.08 0)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Footprint" ""
(at 5.08 0 0)
(show_name no)
(do_not_autoplace no)
(hide yes)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Datasheet" ""
(at 5.08 0 0)
(show_name no)
(do_not_autoplace no)
(hide yes)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Description" "test point"
(at 0 0 0)
(show_name no)
(do_not_autoplace no)
(hide yes)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "ki_keywords" "test point tp"
(at 0 0 0)
(show_name no)
(do_not_autoplace no)
(hide yes)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "ki_fp_filters" "Pin* Test*"
(at 0 0 0)
(show_name no)
(do_not_autoplace no)
(hide yes)
(effects
(font
(size 1.27 1.27)
)
)
)
(symbol "TestPoint_0_1"
(circle
(center 0 3.302)
(radius 0.762)
(stroke
(width 0)
(type default)
)
(fill
(type none)
)
)
)
(symbol "TestPoint_1_1"
(pin passive line
(at 0 0 90)
(length 2.54)
(name "1"
(effects
(font
(size 1.27 1.27)
)
)
)
(number "1"
(effects
(font
(size 1.27 1.27)
)
)
)
)
)
(embedded_fonts no)
)
)
(junction
(at 167.64 81.28)
@@ -6900,6 +6899,16 @@
)
(uuid "2af46450-30c0-4cff-8738-1c006a5fca5f")
)
(wire
(pts
(xy 264.16 104.14) (xy 264.16 106.68)
)
(stroke
(width 0)
(type default)
)
(uuid "2fb7a116-988f-45e1-99ca-636bfc8b3abe")
)
(wire
(pts
(xy 111.76 110.49) (xy 111.76 109.22)
@@ -7490,6 +7499,16 @@
)
(uuid "b151decd-9a01-4c5b-8dd6-750e97c62826")
)
(wire
(pts
(xy 259.08 110.49) (xy 259.08 113.03)
)
(stroke
(width 0)
(type default)
)
(uuid "b2a1faaa-31f8-4abd-bf6f-f483b2639004")
)
(wire
(pts
(xy 81.28 29.21) (xy 92.71 29.21)
@@ -7850,6 +7869,16 @@
)
(uuid "f7d021be-8913-4b26-980b-4c67ae03779f")
)
(wire
(pts
(xy 259.08 102.87) (xy 259.08 100.33)
)
(stroke
(width 0)
(type default)
)
(uuid "f8acf49d-b212-45d4-bed0-c78b73c13dbb")
)
(wire
(pts
(xy 114.3 29.21) (xy 116.84 29.21)
@@ -8290,6 +8319,16 @@
)
(uuid "d76dbeb7-a006-43f4-9ee5-e1f4df0245b6")
)
(label "ETH_3V3"
(at 259.08 100.33 90)
(effects
(font
(size 1.27 1.27)
)
(justify left bottom)
)
(uuid "d77dd782-590c-478d-af84-11fe9cce16a1")
)
(label "ETH_3V3"
(at 72.39 177.8 180)
(effects
@@ -8340,6 +8379,16 @@
)
(uuid "e10c86ed-9f67-49f9-b760-00cd79c33942")
)
(label "ETH_RST"
(at 259.08 113.03 90)
(effects
(font
(size 1.27 1.27)
)
(justify left bottom)
)
(uuid "e22758d1-05a2-4d3b-bf3b-4ddf6cc78c26")
)
(label "LED_ACT"
(at 129.54 81.28 0)
(effects
@@ -8350,6 +8399,16 @@
)
(uuid "e22d7d65-069d-4eba-a49d-f7e825017209")
)
(label "ETH_3V3"
(at 264.16 106.68 90)
(effects
(font
(size 1.27 1.27)
)
(justify left bottom)
)
(uuid "e334588c-685d-4bba-8229-40f8d701126a")
)
(label "PHY_RX_N"
(at 251.46 100.33 90)
(effects
@@ -8788,6 +8847,84 @@
)
)
)
(symbol
(lib_id "Connector:TestPoint")
(at 264.16 104.14 0)
(unit 1)
(body_style 1)
(exclude_from_sim no)
(in_bom no)
(on_board yes)
(in_pos_files yes)
(dnp no)
(uuid "1193d498-9654-409a-9ac0-66959074d36e")
(property "Reference" "TP3"
(at 266.192 102.235 0)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
(justify left)
)
)
(property "Value" "ETH_3V3"
(at 266.192 104.775 0)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
(justify left)
)
)
(property "Footprint" "TestPoint:TestPoint_Pad_D1.5mm"
(at 264.16 104.14 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Datasheet" ""
(at 264.16 104.14 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Description" "Test point"
(at 264.16 104.14 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(pin "1"
(uuid "36a33547-93a5-4f08-afe3-5b24cfdf40d0")
)
(instances
(project "re-bba-rb"
(path "/dbb182a6-d579-468e-b98b-6f0950da9e3a/c2ddb95f-8a63-4bb0-b3f2-a27a372fea3a"
(reference "TP3")
(unit 1)
)
)
)
)
(symbol
(lib_id "power:GND")
(at 129.54 99.06 90)
@@ -9212,6 +9349,131 @@
)
)
)
(symbol
(lib_id "Device:R")
(at 259.08 106.68 0)
(unit 1)
(body_style 1)
(exclude_from_sim no)
(in_bom yes)
(on_board yes)
(in_pos_files yes)
(dnp no)
(uuid "249287b8-98d2-4790-83eb-51837e4cba14")
(property "Reference" "R36"
(at 261.112 104.775 0)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
(justify left)
)
)
(property "Value" "10k"
(at 261.112 107.315 0)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
(justify left)
)
)
(property "Footprint" "Resistor_SMD:R_0402_1005Metric_Pad0.72x0.64mm_HandSolder"
(at 259.08 106.68 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Datasheet" ""
(at 259.08 106.68 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Description" "Resistor"
(at 259.08 106.68 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "MPN" "0402WGF1002TCE"
(at 0 0 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Manufacturer" "UNI-ROYAL"
(at 0 0 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "LCSC" "C25744"
(at 0 0 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Tolerance" "1%"
(at 0 0 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(pin "1"
(uuid "b1c7365e-5912-4683-9608-7ad9581f6a6e")
)
(pin "2"
(uuid "7740076c-285d-45a6-95ea-0c1b5f42cfdb")
)
(instances
(project "re-bba-rb"
(path "/dbb182a6-d579-468e-b98b-6f0950da9e3a/c2ddb95f-8a63-4bb0-b3f2-a27a372fea3a"
(reference "R36")
(unit 1)
)
)
)
)
(symbol
(lib_id "power:GND")
(at 106.68 110.49 0)
@@ -16105,278 +16367,4 @@
)
)
)
(text "W5100S RSTn pull-up: RSTn is a plain\ninput (no internal pull) and floats while\nthe FPGA configures. Pulled to ETH_3V3\n(not 3V3) so it is low when ETH rail off."
(exclude_from_sim no)
(at 254.0 95.25 0)
(effects
(font
(size 1.27 1.27)
)
(justify left)
)
(uuid "445a64bc-57fe-4f1a-bbd9-cfcaeac670df")
)
(symbol
(lib_id "Device:R")
(at 259.08 106.68 0)
(unit 1)
(body_style 1)
(exclude_from_sim no)
(in_bom yes)
(on_board yes)
(in_pos_files yes)
(dnp no)
(uuid "249287b8-98d2-4790-83eb-51837e4cba14")
(property "Reference" "R36"
(at 261.11199999999997 104.775 0)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
(justify left)
)
)
(property "Value" "10k"
(at 261.11199999999997 107.31500000000001 0)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
(justify left)
)
)
(property "Footprint" "Resistor_SMD:R_0402_1005Metric_Pad0.72x0.64mm_HandSolder"
(at 259.08 106.68 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Datasheet" ""
(at 259.08 106.68 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Description" "Resistor"
(at 259.08 106.68 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "MPN" "0402WGF1002TCE"
(at 0 0 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Manufacturer" "UNI-ROYAL"
(at 0 0 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "LCSC" "C25744"
(at 0 0 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Tolerance" "1%"
(at 0 0 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(pin "1"
(uuid "b1c7365e-5912-4683-9608-7ad9581f6a6e")
)
(pin "2"
(uuid "7740076c-285d-45a6-95ea-0c1b5f42cfdb")
)
(instances
(project "re-bba-rb"
(path "/dbb182a6-d579-468e-b98b-6f0950da9e3a/c2ddb95f-8a63-4bb0-b3f2-a27a372fea3a"
(reference "R36")
(unit 1)
)
)
)
)
(wire
(pts
(xy 259.08 102.87) (xy 259.08 100.33)
)
(stroke
(width 0)
(type default)
)
(uuid "f8acf49d-b212-45d4-bed0-c78b73c13dbb")
)
(label "ETH_3V3"
(at 259.08 100.33 90)
(effects
(font
(size 1.27 1.27)
)
(justify left bottom)
)
(uuid "d77dd782-590c-478d-af84-11fe9cce16a1")
)
(wire
(pts
(xy 259.08 110.49) (xy 259.08 113.03)
)
(stroke
(width 0)
(type default)
)
(uuid "b2a1faaa-31f8-4abd-bf6f-f483b2639004")
)
(label "ETH_RST"
(at 259.08 113.03 270)
(effects
(font
(size 1.27 1.27)
)
(justify left bottom)
)
(uuid "e22758d1-05a2-4d3b-bf3b-4ddf6cc78c26")
)
(symbol
(lib_id "Connector:TestPoint")
(at 264.16 104.14 0)
(unit 1)
(body_style 1)
(exclude_from_sim no)
(in_bom no)
(on_board yes)
(in_pos_files yes)
(dnp no)
(uuid "1193d498-9654-409a-9ac0-66959074d36e")
(property "Reference" "TP3"
(at 266.192 102.235 0)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
(justify left)
)
)
(property "Value" "ETH_3V3"
(at 266.192 104.775 0)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
(justify left)
)
)
(property "Footprint" "TestPoint:TestPoint_Pad_D1.5mm"
(at 264.16 104.14 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Datasheet" ""
(at 264.16 104.14 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(property "Description" "Test point"
(at 264.16 104.14 0)
(hide yes)
(show_name no)
(do_not_autoplace no)
(effects
(font
(size 1.27 1.27)
)
)
)
(pin "1"
(uuid "36a33547-93a5-4f08-afe3-5b24cfdf40d0")
)
(instances
(project "re-bba-rb"
(path "/dbb182a6-d579-468e-b98b-6f0950da9e3a/c2ddb95f-8a63-4bb0-b3f2-a27a372fea3a"
(reference "TP3")
(unit 1)
)
)
)
)
(wire
(pts
(xy 264.16 104.14) (xy 264.16 106.68)
)
(stroke
(width 0)
(type default)
)
(uuid "2fb7a116-988f-45e1-99ca-636bfc8b3abe")
)
(label "ETH_3V3"
(at 264.16 106.68 270)
(effects
(font
(size 1.27 1.27)
)
(justify left bottom)
)
(uuid "e334588c-685d-4bba-8229-40f8d701126a")
)
)