OpenROAD-flow-scripts icon indicating copy to clipboard operation
OpenROAD-flow-scripts copied to clipboard
verified publisher icon The-OpenROAD-Project

flow: platforms: ihp-sg13g2: Add IO files

Open dnltz opened this issue 1 year ago • 6 comments

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.

  • [ ] blocked by #2084

dnltz avatar Jun 27 '24 09:06 dnltz

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

dnltz avatar Jun 27 '24 09:06 dnltz

@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.

maliberty avatar Jun 27 '24 17:06 maliberty

FOOTPRINT_TCL

That's even better, thanks!

dnltz avatar Jul 01 '24 08:07 dnltz

@KrzysztofHerman fyi

dnltz avatar Jul 01 '24 08:07 dnltz

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

maliberty avatar Jul 01 '24 15:07 maliberty

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 avatar Jul 03 '24 15:07 dnltz

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

dnltz avatar Aug 06 '24 10:08 dnltz

@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.

dnltz avatar Aug 12 '24 07:08 dnltz

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

maliberty avatar Aug 12 '24 13:08 maliberty

sure, can do that.

dnltz avatar Aug 12 '24 13:08 dnltz