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

close missing DRC rules in klayout deck #521

Open
jeffdi opened this issue Jan 10, 2024 · 14 comments
Open

close missing DRC rules in klayout deck #521

jeffdi opened this issue Jan 10, 2024 · 14 comments
Assignees

Comments

@jeffdi
Copy link
Collaborator

jeffdi commented Jan 10, 2024

No description provided.

@jeffdi
Copy link
Collaborator Author

jeffdi commented Jan 10, 2024

issue surfaced on 2311 projects for hvi and nwell

@jeffdi
Copy link
Collaborator Author

jeffdi commented Jan 10, 2024

received violator cells from SWT, but they don't cover all the rules and we are missing Calibre reports to benchmark results. The reported are numerous.

@RTimothyEdwards
Copy link
Contributor

We can implement rules for ourselves based on the s8 calibre rules and the derived (generated) layer tables. But the generated layer tables are not part of the public documentation, so they would be for internal use only.

However, there are various rules in the public documentation that affect the generated layers. In some cases, these rules effectively become MR rules in the absence of any direct check of the mask layer itself. The rule nwell.9 (when HVI intersects nwell, the nwell must be completely covered by HVI) is one such rule. We need to add that rule to our own MR deck because we don't have MR rules for the generated layers. There are probably other similar rules that cover other generated layers.

@DavidRLindley
Copy link
Contributor

The current ruleset incorrectly checks MR_capm.SP.2 rule. The attached PDF shows how the rule should be applied.

capm_rules.pdf

@marwaneltoukhy
Copy link
Member

marwaneltoukhy commented Apr 14, 2024

The current ruleset incorrectly checks MR_capm.SP.2 rule. The attached PDF shows how the rule should be applied.

capm_rules.pdf

@DavidRLindley there is still no explanation of what is met3_bottom_plate

@kareefardi
Copy link
Contributor

@DavidRLindley Couple of follow up questions:

  • Just to confirm Metal3 not continuous under CAPM, but electrically connected implies connectivity through other metal layers e.g.: met3 -> via2 -> met2 -> via2 -> met3
  • In scenario 3, does MR_capm.SP.2 apply to spacing between Met3_bot_plate and a met3 patch that doesn't enclose capm? There is such a case in the violator cells.

@DavidRLindley
Copy link
Contributor

DavidRLindley commented Apr 15, 2024 via email

@DavidRLindley
Copy link
Contributor

Marwan, Kareem,

I verified the following with Skywater.

  1. met3_bot_plate is defined as (capm:dg AND met3:dg) sized by 0.14um
  2. the capm spacing (MR_capm.SP.1) needs to be satisfied regardless of how the metal3 geoms are connected.
  3. MR_capm.SP.2 applies to any met3 that is not connected to the met3_bot_plate.

David

@kareefardi
Copy link
Contributor

Thanks David. Should we assume that similar rules apply for cap2m?

@DavidRLindley
Copy link
Contributor

DavidRLindley commented Apr 16, 2024 via email

@DavidRLindley
Copy link
Contributor

Two psdm rules are being flagged on some proprietary memory macro IP.

  1. psdm.4 : min. spacing across areaid:ce : 0.38um
    I think this rule is not applied correctly since both psdm geoms are inside areaid:ce.

  2. psdm.6a : min. psdm spacing on parallel edges in areaid:ce : 0.38um
    FAB looked at the attached PDF and agreed that our psdm.6a rule did not apply because the edges were not parallel. The correct
    rule to apply in this scenario is the lesser core rule 0.29um.

We can have a Zoom meeting if you want to view the errors.

psdm_rules.pdf

@DavidRLindley
Copy link
Contributor

As of 2024/May/08 these issues exist in the rules

  1. mcon straddling the areaid:core is truncated by the rules
  2. I have a couple examples in Chipalooza_1 where the a couple of licon shapes are being incorrectly flagged
  3. We need an global li density check.
  4. We need to settle on the final set of capm rules.
  5. The switches need to be address. They didn't work correctly in my last run of the most recent rules (core PSDM/NSDM).

@jeffdi
Copy link
Collaborator Author

jeffdi commented May 22, 2024

need to include new issues related to created layer DRC errors.

@jeffdi
Copy link
Collaborator Author

jeffdi commented May 22, 2024

remaining issues

  • licon
  • mcon
  • created layers rules - need to generate from reversed engineer work by Tim as well as violator

verify if new rule deck has created layer rules - then check for violator cells

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

5 participants