-
Notifications
You must be signed in to change notification settings - Fork 620
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
Check and add Hwacha tests #284
Conversation
@a0u previously used a cleaner way to query the Hwacha parameters. I am using that instead. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This works fine for now, although there should be a more scalable approach of injecting tests as part of the accelerator mix-in.
This scheme also doesn't support heterogenous RoCC configurations, but that is a broader issue with the assembly tests themselves.
I agree this mechanism is just to make things work. We need to think more about how to inject tests based on tiles, rocc modules, etc. |
Basic example of checking for a Hwacha parameter to add its tests.