-
Notifications
You must be signed in to change notification settings - Fork 108
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
A test for OSERDES routing #1261
Conversation
Signed-off-by: Maciej Kurc <mkurc@antmicro.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As in the comment, I would enable the DIFF test, and merge this once there is also the fix for the incorrect behavior.
PARENT_NAME oserdes_routing | ||
# The fasm diff test is disabled as it is expected to fail for now. Should | ||
# be re-enabled when #1258 is fixed | ||
DISABLE_DIFF_TEST |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe the DIFF test should be enabled, as the purpose of this test is to verify this behavior. This way I could take over this PR which can be merged as soon as the const issue is fixed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I've enabled the diff test. Should fail in Vivado with net conflict between <const 0> and <const 1>
Signed-off-by: Maciej Kurc <mkurc@antmicro.com>
1260a65
to
11c3721
Compare
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
@mkurc-ant I have pushed a fix to correctly calculate the ZINV and IS_INVERTED parameters. |
Signed-off-by: Alessandro Comodi <acomodi@antmicro.com>
The
Failing tests are related to other issues:
I think we can merge the fix anyway. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This is a small test that checks SymbiFlow handling of OSERDES inputs. Demonstrates the #1258 issue.
This PR also updates pack patterns for OSERDES.