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

various fixes for 030-iob18 #2072

Merged
merged 5 commits into from
Dec 19, 2022
Merged

various fixes for 030-iob18 #2072

merged 5 commits into from
Dec 19, 2022

Conversation

hansfbaier
Copy link
Collaborator

Signed-off-by: Hans Baier hansfbaier@gmail.com

Signed-off-by: Hans Baier <hansfbaier@gmail.com>
Signed-off-by: Hans Baier <hansfbaier@gmail.com>
@hansfbaier
Copy link
Collaborator Author

hansfbaier commented Dec 14, 2022

One of the results of these fixes is the discovery of two new features:

RIOB18.IOB_Y0.ZIBUF_LOW_PWR origin:030-iob18 39_125
RIOB18.IOB_Y1.ZIBUF_LOW_PWR origin:030-iob18 38_02

…duces specimens to 40

Signed-off-by: Hans Baier <hansfbaier@gmail.com>
@hansfbaier hansfbaier changed the title WIP: various fixes for 030-iob18 various fixes for 030-iob18 Dec 16, 2022
@hansfbaier
Copy link
Collaborator Author

@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.

Copy link
Contributor

@tmichalak tmichalak left a 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!!!

fuzzers/030-iob18/Makefile Show resolved Hide resolved
fuzzers/030-iob18/Makefile Show resolved Hide resolved
fuzzers/030-iob18/write_io_banks.tcl Outdated Show resolved Hide resolved
…e banks

Signed-off-by: Hans Baier <hansfbaier@gmail.com>
@hansfbaier
Copy link
Collaborator Author

@tmichalak I created the issue: #2073

Signed-off-by: Hans Baier <hansfbaier@gmail.com>
Copy link
Contributor

@tmichalak tmichalak left a 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.

@tmichalak tmichalak merged commit fa3a727 into f4pga:master Dec 19, 2022
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.

2 participants