search

efabless / caravel

Caravel is a standard SoC harness with on chip resources to control and read/write operations from a user-dedicated space.
https://caravel-harness.readthedocs.io/
Apache License 2.0
265 stars 64 forks source link

caravel_openframe_wrapper `gpio_loopback_one*` and `gpio_loopback_zero*` shifted relative to chip_io_openframe #470

Open kareefardi opened 1 year ago

kareefardi commented 1 year ago

West, North and some of East pins are shifted. See screenshots taken from caravel_openframe gds view:

2023-06-04_14-06

2023-06-04_14-07_1 2023-06-04_14-07

RTimothyEdwards commented 1 year ago

@kareefardi : I don't see any such offset. Are you looking at the .mag views of the cells? Those are the views used for tapeouts.

RTimothyEdwards commented 1 year ago

Also, are you using the openframe_development branch?

There is no GDS view of caravel_openframe, so how are you generating it?

RTimothyEdwards commented 1 year ago

@kareefardi : Okay, with a little investigation, I think I've sorted out the main issue, which is that the problem lies in my openframe_project_example repository and not in caravel, and the issue should never have been reported here because it is not a caravel issue. Also, the issue appears to be that the DEF file generated for the wrapper and added to the openlane directory in the example project is outdated with respect to the latest version of caravel. I have updated the openframe_project_example repository with a DEF file that matches the layout in caravel.

RTimothyEdwards commented 1 year ago

@kareefardi : Should be fixed by PR #473 once it's merged into main.