Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ignition Controller: Repurpose PCIE_HOST_TO_FPGA_PWREN to PCIE_FPGA_TO_HOST_ALERT_L #75

Open
Aaron-Hartwig opened this issue Feb 3, 2023 · 2 comments

Comments

@Aaron-Hartwig
Copy link
Collaborator

This means renaming, swapping the input to an output, and adding a weak internal pull-up on the D5 pin.

@Aaron-Hartwig
Copy link
Collaborator Author

Note that this change needs to be released with care. On current hardware, this pin is meant to be an input and is being driven by a buffer, meaning if we were to just swap the FPGA pin here it would cause the pin and buffer to fight, ultimately ending in sadness. This can only happen for Build D hardware (does not exist yet) or previous builds which have been modified. When I write the MCN I will reference it here.

@Aaron-Hartwig
Copy link
Collaborator Author

See updates and notes to https://github.com/oxidecomputer/hardware-sidecar/issues/765, but the action here going forward is just to remove the PCIE_HOST_TO_FPGA_PWREN pin altogether.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant