search

The-OpenROAD-Project / OpenROAD-flow-scripts

OpenROAD's scripts implementing an RTL-to-GDS Flow. Documentation at https://openroad-flow-scripts.readthedocs.io/en/latest/
https://theopenroadproject.org/
Other
320 stars 275 forks source link

flow: platforms: ihp-sg13g2: Add IO files #2090

Closed dnltz closed 1 month ago

dnltz commented 3 months ago

Add all IO related lib, lef, and gds files when 'HAS_IO_RING' is set to 1. Those files will be appended to the 'ADDITIONAL_x' variables.

dnltz commented 3 months ago

@maliberty any concerns about adding HAS_IO_RING as optional environment variable? Should I document that somewhere?

maliberty commented 3 months ago

@maliberty any concerns about adding HAS_IO_RING as optional environment variable? Should I document that somewhere?

In order to generate a pad ring the user will need to set FOOTPRINT_TCL (or the obsolete FOOTPRINT). HAS_IO_RING seems redundant.

dnltz commented 3 months ago

FOOTPRINT_TCL

That's even better, thanks!

dnltz commented 3 months ago

@KrzysztofHerman fyi

maliberty commented 3 months ago

This is marked as a draft - should I hold off on review?

dnltz commented 2 months ago

This is marked as a draft - should I hold off on review?

yeah, let's start when the other PR got merged since this fails anyway

dnltz commented 1 month ago

@maliberty since all required files are on master now, this PR is ready for review :)

dnltz commented 1 month ago

@maliberty lib files for IO cells are only available in the dev branch: https://github.com/IHP-GmbH/IHP-Open-PDK/tree/dev/ihp-sg13g2/libs.ref/sg13g2_io/lib

So, should be available with the next merge to main.

maliberty commented 1 month ago

@dnltz will you make a PR once they are ready?

dnltz commented 1 month ago

sure, can do that.