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

adi_board.tcl: add some optional parameters to ad_xcvrcon and ad_cpu_interconnect to support hierarchal designs #1070

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

aholtzma-am
Copy link
Contributor

ad_xcvrcon

The port_prefix parameter allows an optional prefix to be applied to
port names created by ad_xcvrcon. This is useful when created large
hierarchical designs where all ports are prefixed by their hierarchical
name to avoid name clashes at the top level (for example: slot_a,
slot_b, slot_c, etc).

The resetn parameter sets the net name to be connected to ext_reset_in
on the reset generator. This parameter defaults to 'sys_cpu_resetn'.

ad_cpu_interconnect

Add an optional prefix, segment_prefix, to be applied to the segment to avoid name clashes. This is used when creating a block design in the root, and then moved to a hierarchical cell using move_bd_cells. Without the prefix, a second instance of this block design will have segment name conflicts because it uses the instance name before moving.

The port_prefix parameter allows an optional prefix to be applied to
port names created by ad_xcvrcon. This is useful when created large
hierarchical designs where all ports are prefixed by their hierarchical
name to avoid name clashes at the top level (for example: slot_a,
slot_b, slot_c, etc).

The resetn parameter sets the net name to be connected to ext_reset_in
on the reset generator. This parameter defaults to 'sys_cpu_resetn'.
This adds an optional prefix to be applied to the segment to avoid names clashes. This is used when creating a block design in the root, and then moved to a hierarchical cell using move_bd_cells. Without the prefix, a second instance of this block design will have segment name conflicts because it uses the instance name before moving.
@acostina acostina requested review from sarpadi and FilipG24 January 26, 2023 08:26
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.

1 participant