lowRISC / opentitan

OpenTitan: Open source silicon root of trust
https://www.opentitan.org
Apache License 2.0
2.6k stars 783 forks source link

[rtl,flash_ctrl] Remove PHY_ALERT_CFG CSR #25418

Open matutem opened 5 days ago

matutem commented 5 days ago

Description

The PHY_ALERT_CFG CSR's outputs are unconnected, and the CSR itself can be removed. This would cause changes in CSR offsets, so it would become an incompatible change requiring bumping up the major version of the IP.

Razer6 commented 5 days ago

Couldn't youi create a skip-to and create a hole?

vogelpi commented 4 days ago

To not require bumping the major version but avoid the coverage holes I would rather update the definition of the CSR to always read as 0, and to update the documentation inline with that.