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

[hw,otp_ctrl,rtl] Parametrize the DFT config and response port #25938

Merged
merged 1 commit into from
Jan 20, 2025

Conversation

Razer6
Copy link
Member

@Razer6 Razer6 commented Jan 19, 2025

This parametrization allows the usage of prim_otp with different DFT config and response ports. One usage is for example when using macro tiling within the prim instance and have different instances with different configuration in the design. By passing the port type as a parameter we don't need to pull in all the tiling related parameters into the design.

This parametrization allows the usage of prim_otp with different
DFT config and response ports. One usage is for example when using
macro tiling within the prim instance and have different instances
with different configuration in the design. By passing the port
type as a parameter we don't need to pull in all the tiling related
parameters into the design.

Signed-off-by: Robert Schilling <[email protected]>
@vogelpi
Copy link
Contributor

vogelpi commented Jan 20, 2025

CHANGE AUTHORIZED: hw/ip/prim_generic/rtl/prim_generic_otp.sv
CHANGE AUTHORIZED: hw/top_earlgrey/ip_autogen/otp_ctrl/rtl/otp_ctrl.sv

This PR doesn't change the functionality of OTP in Earlgrey.

@vogelpi vogelpi added the CI:Rerun Rerun failed CI jobs label Jan 20, 2025
@github-actions github-actions bot removed the CI:Rerun Rerun failed CI jobs label Jan 20, 2025
@a-will
Copy link
Contributor

a-will commented Jan 20, 2025

CHANGE AUTHORIZED: hw/ip/prim_generic/rtl/prim_generic_otp.sv
CHANGE AUTHORIZED: hw/top_earlgrey/ip_autogen/otp_ctrl/rtl/otp_ctrl.sv

@vogelpi vogelpi merged commit b717764 into lowRISC:master Jan 20, 2025
40 checks passed
@Razer6 Razer6 deleted the otp-prim-type branch January 20, 2025 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants