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

ONI spec section #6

Open
aacuevas opened this issue Feb 24, 2024 · 1 comment
Open

ONI spec section #6

aacuevas opened this issue Feb 24, 2024 · 1 comment

Comments

@aacuevas
Copy link

Ported from jonnew/ONI#6 and related to #5

Original text:

ONI compliance registers
Since the spec might evolve in the future, with new capabilities that might have hardware-dependent parameters, the second register region is reserved for information about the hardware capabilities themselves.
Address 0x00100000 will contain the specification version. Since new capabilities will always be specified in a revision, this is enough to inform of all the capabilities.
Following addresses will specify optional or hardware-dependent parameters as described in the relevant specification version, such as buffer sizes for specific functions.

@aacuevas
Copy link
Author

Since I originally wrote this, I have a slightly different opinion.
The base idea is the same: We need a field that informs to which spec revision the hardware is compliant. And we might need fields for some values whose meaning might be defined by the spec but actual values are hardware-dependent.

However, I am now inclined to think that a revision might not always just be a full set of fixed capabilities. I believe we should be able to define capabilities as "optional" sections of the spec, which is a common practice. This section should, then, accommodate ways to inform which optional capabilities (as defined by a spec version) are enabled.

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

No branches or pull requests

1 participant