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

dfflibmap: support flops with enable #4652

Open
Ravenslofty opened this issue Oct 10, 2024 · 1 comment
Open

dfflibmap: support flops with enable #4652

Ravenslofty opened this issue Oct 10, 2024 · 1 comment
Assignees

Comments

@Ravenslofty
Copy link
Collaborator

Feature Description

This seems to be a notable oversight in this pass.

There are a few questions on hypotheticals though:

  • should we include scan-chain flops which can be wired into enable flops? sky130 has these, but it also has flops with normal enables too.
  • could this integrate with clockgate if a PDK does not provide an enable flop but provides a clock-gating primitive?
@povik
Copy link
Member

povik commented Oct 22, 2024

should we include scan-chain flops which can be wired into enable flops?

I expect no, scan-chain flops shouldn't be misused to implement enables in ordinary logic. As we saw when we discussed this, scan-chain pins are annotated so we can avoid using them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants