- Notifications
You must be signed in to change notification settings - Fork8
Usedefcell for clock delay buffers#7
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Rylan12 commentedDec 29, 2024
It's also worth noting that this is the case for For the latches, it's less noticeable since their bodies only contain For the clock buffers, there are no nested cells, so the following production-rule-based cells are created and (assuming I understand correctly) can be placed apart from each other:
Here is an exceprt from a |
Uh oh!
There was an error while loading.Please reload this page.
This PR changes the
CLKBUF{1,2,3}standard definitions to usedefcellinstead ofdefprocwhich matches the rest of the cells defined here. Also, my basic understanding tells me that usingdefproccan lead to inconsistent delay timing since it will depend on where and how the place-and-route tool puts each of the buffer components.I tested this briefly, and it seems to generate a pretty compact transistor layout with
sky130l(below is a sampleCLKBUF1layout I did quickly—it seems to work as expected in IRSim).Obviously, feel free to disregard if this is intentional, I just noticed that it didn't match other cells and figured I'd point it out in case it was a mistake.