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

Use of register macros within common_cells #68

Open
stmach opened this issue Feb 19, 2020 · 1 comment
Open

Use of register macros within common_cells #68

stmach opened this issue Feb 19, 2020 · 1 comment
Labels
lint question Further information is requested

Comments

@stmach
Copy link
Contributor

stmach commented Feb 19, 2020

What's the stance on the usage of register macros from registers.svh in general and especially within this repo itself?

  • Are we advocating for "safe" use of always_ff blocks by only using them through these macros?
  • If so, should we replace always_ff blocks with the appropriate macros for IPs within this repo?
@stmach stmach added question Further information is requested lint labels Feb 19, 2020
@zarubaf
Copy link
Contributor

zarubaf commented Feb 22, 2020

We do not have a recommendation on the usage of registers.svh and I think the "easiest" way would be for the maintainer of the repository to decide. We can try to start a discussion on this but I am afraid that this will result in a lot of bikeshedding.

I am in favor of using FF macros in common cells IPs, although I am not advocating to re-write everything, merely start a migration process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lint question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants