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

Unify/Simplify SystolicArrayConfig #2

Closed
vighneshiyer opened this issue Mar 8, 2019 · 1 comment
Closed

Unify/Simplify SystolicArrayConfig #2

vighneshiyer opened this issue Mar 8, 2019 · 1 comment

Comments

@vighneshiyer
Copy link
Collaborator

Currently some config parameters (like block_size, sp_bank_entries, sp_width) in SystolicArrayConfig are dependent on prior parameters and can be computed internally. Remove the unnecessary assertions in Controller.scala once this is done.

Also, the datatype (inner_type) of the systolic array is being passed in to the top-level; could we move that to the Config object if possible?

@hngenc
Copy link
Member

hngenc commented Dec 6, 2019

Parameters like block_size, sp_bank_entries, and sp_width, are no longer inputs to the config class, and all datatypes have now been folded into it.

@hngenc hngenc closed this as completed Dec 6, 2019
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

2 participants