oxidecomputer / oxidecomputer/quartz

Need `PcieEndpointController.bsv` to follow the PCIe spec around PERST

Open
#226 0 comments 0 reactions 1 assignee View on GitHub

@Aaron-Hartwig is already working on this.

Since Oct 8, 2024.

Sidecar Sequencer
Dominant language
VHDL
Stars
22
Forks
2
Avg merge
9h 38m
Merged PRs (30d)
1

Description

Right now the PcieEndpointController block is used to proxy PERST# from a server to the Tofino 2. That block should honor the various timings around PERST# defined by the PCIe spec:

  • Tpvperl - PERST# must remain active at least this long after power becomes valid
  • Tperst - When asserted, PERST# must remain asserted at least this long
  • Tfail - When power becomes invalid, PERST# must be asserted within this time
  • Tperst-clk - PERST# must remain active at least this long after any supplied reference clock is
    stable

There is a discussion of this in RFD 123. Right now we don't have any software controls around enforcing this behavior, things are working just due to the nature of boot. We can enforce the right behavior in the FPGA, so we should. The CEM spec is the worst case scenario around PERST#, so we should implement to that.

  • Tpvperl - 100 ms (min)
  • Tperst - 100 us (min)
  • Tfail - 500 ns (max)
  • Tperst-clk - 100 us (min)

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.