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

Fix OPEN_DRAIN IO attribute #194

Merged
merged 1 commit into from
Aug 10, 2023
Merged

Fix OPEN_DRAIN IO attribute #194

merged 1 commit into from
Aug 10, 2023

Conversation

yrabbit
Copy link
Collaborator

@yrabbit yrabbit commented Aug 10, 2023

The coding of the OPEN_DRAIN attribute turned out to be not so unambiguous. It looks like there is ON / OFF and one more default state.

As a temporary measure, setting OPEN_DRAIN=OFF is ignored, but you use OPENDRAIN=ON at your own peril and risk.

The coding of the OPEN_DRAIN attribute turned out to be not so
unambiguous. It looks like there is ON / OFF and one more default state.

As a temporary measure, setting OPEN_DRAIN=OFF is ignored, but you use
OPENDRAIN=ON at your own peril and risk.

Signed-off-by: YRabbit <[email protected]>
@yrabbit yrabbit merged commit 46ecf0c into YosysHQ:master Aug 10, 2023
11 of 13 checks passed
@yrabbit yrabbit deleted the opendrain branch August 10, 2023 07:34
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.

1 participant