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

Operating conditions in corners are wrong #107

Closed
mguthaus opened this issue Jan 4, 2021 · 5 comments
Closed

Operating conditions in corners are wrong #107

mguthaus opened this issue Jan 4, 2021 · 5 comments
Assignees

Comments

@mguthaus
Copy link
Collaborator

mguthaus commented Jan 4, 2021

Regarding liberty opcond, the problem is that some models use 1.9 in operating condition, but that is different to the voltage map, in order to support multi-voltage implementation, some tools rely on pg-pin voltage for linking, while others non MV-aware just assume single voltage by using operating condition alone. To avoid linking inconsistency, these should match in libraries modeling pg-pins.
sram_1rw1r_32_256_8_sky130_TT_1p9V_25C.lib

operating_conditions(OC){
process : 1.0 ;
voltage : 1.9 ;
temperature : 25;
}
...
nom_voltage : 1.8;
nom_temperature : 25;
nom_process : 1.0;
...
voltage_map ( VDD, 1.8 );
voltage_map ( GND, 0 );

@mguthaus
Copy link
Collaborator Author

Any update on this?

@hznichol
Copy link
Contributor

I removed the nominal voltage/temp/process and made the voltage map the same as the operating conditions.
Was going to test at least the syntax before pushing the change.

@mguthaus
Copy link
Collaborator Author

Go ahead and push. You can assume the previous syntax was correct and just the values were wrong.

@mguthaus
Copy link
Collaborator Author

What is the status on this?

@hznichol
Copy link
Contributor

Change has been added to which now has "operating_conditions" and "nom_voltage, nom_temperature, nom_process" as the same values in the lib file.

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