Skip to content

Xilinx constraints fix#937

Open
Ludblanc wants to merge 7 commits into
x-heep:mainfrom
Ludblanc:xilinx_constraints_fix
Open

Xilinx constraints fix#937
Ludblanc wants to merge 7 commits into
x-heep:mainfrom
Ludblanc:xilinx_constraints_fix

Conversation

@Ludblanc

Copy link
Copy Markdown

The FPGA synthesis constraints file includes the waveform parameter, which defines the clock duty cycle. By default, if this parameter is not explicitly specified, the duty cycle is assumed to be (50%) of the clock period.

The waveform parameter is expressed as two values:

$$(t_1; t_2)$$

where (t_1) represents the time of the first transition (typically (0)), and ($t_2$) represents the time of the second transition in nanoseconds. In this case, it is defined for all signals as:

$$(0; 5,\text{ns})$$

Please note that when both clock edges are used, it may be necessary to introduce clock uncertainties to provide adequate setup and hold timing margins.

@LuigiGiuffrida98 LuigiGiuffrida98 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Ludblanc, thank you for your PR, since, the DC is assumed to be 50%, do we have any specific reason to have the waveform parameter in the create_clock command?

I would suggest to simply remove it.

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