Fix heartbeat pin collision with clock pin

robtaylor · robtaylor · commit b7c28ce83c25 · 2025-10-28T12:48:41.000Z
- Change core_heartbeat from A2 to A8 (after JTAG pins A3-A7)
- Heartbeat is an output, clock is an input - can't share pin
- Addresses gatecat review comment on grid_array.py:220
diff --git a/chipflow_lib/packaging/grid_array.py b/chipflow_lib/packaging/grid_array.py
@@ -217,7 +217,7 @@ def bringup_pins(self) -> BringupPins:
             core_power=self._power,
             core_clock=GAPin('A', 2),
             core_reset=GAPin('A', 1),
-            core_heartbeat=GAPin('A', 2),  # Note: Same as clock in original
+            core_heartbeat=GAPin('A', 8),  # Output pin, after JTAG (A3-A7)
             core_jtag=self._jtag
         )
 

Original file line number	Diff line number	Diff line change
`@@ -217,7 +217,7 @@ def bringup_pins(self) -> BringupPins:`
`217`	`217`	`core_power=self._power,`
`218`	`218`	`core_clock=GAPin('A', 2),`
`219`	`219`	`core_reset=GAPin('A', 1),`
`220`		`- core_heartbeat=GAPin('A', 2), # Note: Same as clock in original`
	`220`	`+ core_heartbeat=GAPin('A', 8), # Output pin, after JTAG (A3-A7)`
`221`	`221`	`core_jtag=self._jtag`
`222`	`222`	`)`
`223`	`223`