-
Notifications
You must be signed in to change notification settings - Fork 150
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
various fixes for 030-iob18 #2072
Conversation
Signed-off-by: Hans Baier <hansfbaier@gmail.com>
Signed-off-by: Hans Baier <hansfbaier@gmail.com>
One of the results of these fixes is the discovery of two new features:
|
…duces specimens to 40 Signed-off-by: Hans Baier <hansfbaier@gmail.com>
cf8b6c6
to
8ce3922
Compare
@tmichalak Ready for review. The Vref changes get rid of a lot of warning of mismatching Vref, but don't change the outcome. The IBUF_LOW_PWR_SUPPORTED change finds a new feature. |
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 guess that the fact that this fuzzer is kintex specific could make up for some hacks that were used. However, we should not lose track of those and at least open an issue. The write_io_banks.tcl
could use some generalization when it comes to the high performance banks. However, I like the fact that now only 40 specimens are enough and most of all that it works on hardware. Good work!!!
…e banks Signed-off-by: Hans Baier <hansfbaier@gmail.com>
@tmichalak I created the issue: #2073 |
Signed-off-by: Hans Baier <hansfbaier@gmail.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.
LGTM, but let's wait for green CI just in case.
Signed-off-by: Hans Baier hansfbaier@gmail.com