Skip to content

Commit

Permalink
Sidecar Mainboard FPGA: add proper c/d image (#1906)
Browse files Browse the repository at this point in the history
This fixes a problem I accidentally introduced back in
4b74549,
meaning it was part of Sidecar release v1.0.29 and R11. There's no
meaningful impact to customers from this problem.

In that update, I accidentally copied the Rev B bitstream to both
`sidecar_mainboard_rev_b.bit` **and** `sidecar_mainboard_rev_c_d.bit`.
The result is that rev C/D Sidecars are running the rev B FPGA image. In
practice this means very little, as the only meaningful change in
FPGA-land is that we added a 36th Ignition link, the link between the
controller and the target on its own board. At this time nothing is
paying attention to any of that **except** our loopback test fixture in
manufacturing (where this problem was caught).

Obviously this highlights our current manual process of getting FPGA
images into our Hubris image releases. We should certainly have a
discussion about how to add automation (or at very least checks!) to
make this process less error-prone!

@labbott @jclulow with my apologies, we will need to get this released
and deployed into manufacturing. We can coordinate that in chat. In the
meantime, I will instruct the test engineers to ignore Ignition Port 35
errors for now as that is caused by this bug.
  • Loading branch information
Aaron-Hartwig authored Oct 18, 2024
1 parent 7819a76 commit a58bc46
Showing 1 changed file with 0 additions and 0 deletions.
Binary file not shown.

0 comments on commit a58bc46

Please sign in to comment.