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

RFC 53: Low-level I/O primitives. #53

Merged
merged 1 commit into from Mar 11, 2024
Merged

Conversation

wanda-phi
Copy link
Member

@whitequark whitequark added meta:nominated Nominated for discussion on the next relevant meeting area:core RFC affecting APIs in amaranth-lang/amaranth labels Mar 6, 2024
@whitequark
Copy link
Member

Is a deprecation period even necessary for Values as io ports? Arguably, all well-behaved code will continue to just work given that platform.request(dir='-') returns an IOPort.

After sleeping on it: we can probably do this right away. I don't think we have any uncursed way of obtaining a toplevel inout without a platform or the newly introduced undocumented ports= alternate API, do we?

Should we forbid mixing various ways of port creation?

Eventually probably, but I feel like this can be left until a later point when we improve the backend interface as a whole.

Traditional name bikeshedding.

I'm happy with IOValue/IOPort.

@wanda-phi
Copy link
Member Author

Updated to make metadata per-bit.

@wanda-phi wanda-phi force-pushed the ioport branch 4 times, most recently from a04aee5 to a6946a5 Compare March 10, 2024 07:00
@whitequark
Copy link
Member

We have discussed this RFC on the 2024-03-11 Amaranth core meeting. The disposition was to merge.

@whitequark whitequark removed the meta:nominated Nominated for discussion on the next relevant meeting label Mar 11, 2024
text/0053-ioport.md Outdated Show resolved Hide resolved
@whitequark whitequark merged commit 2c7ad0a into amaranth-lang:main Mar 11, 2024
@wanda-phi wanda-phi deleted the ioport branch March 11, 2024 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:core RFC affecting APIs in amaranth-lang/amaranth
2 participants