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

Intel altiobuf_ requires oe with the same with as data lines. #300

Merged
merged 3 commits into from Jan 9, 2020

Conversation

schwigi
Copy link
Contributor

@schwigi schwigi commented Jan 8, 2020

Ah ok missunderstood the concept.

This should fix #297, with your suggestion.

Simple test looks fine on MAX10 FPGA.

@codecov
Copy link

codecov bot commented Jan 8, 2020

Codecov Report

Merging #300 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #300   +/-   ##
=======================================
  Coverage   82.13%   82.13%           
=======================================
  Files          34       34           
  Lines        5647     5647           
  Branches     1160     1160           
=======================================
  Hits         4638     4638           
  Misses        864      864           
  Partials      145      145

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 63902dd...8fed4d9. Read the comment docs.

@whitequark whitequark closed this Jan 9, 2020
@whitequark
Copy link
Contributor

I apologize, I misunderstood your patch here. This is, in fact, the right fix.

@whitequark whitequark reopened this Jan 9, 2020
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.

Output enable for Intel platform on multi I/O pins
2 participants