chipsalliance / silicon-notebooks

Apache License 2.0
161 stars 28 forks source link

add serv notebook #47

Closed proppy closed 1 year ago

proppy commented 1 year ago

TEST=https://colab.research.google.com/github/proppy/silicon-notebooks/blob/serv/serv-openlane-gf180mcu.ipynb

@mithro @olofk fyi @gregdavill @xobs were you successful to plug SERV to the SRAMs macros present in the GF180MCU PDK https://github.com/google/globalfoundries-pdk-ip-gf180mcu_fd_ip_sram/tree/9c411928870ce15226228fa52ddb6ecc0ea4ffbe

mithro commented 1 year ago

You can view the notebook at https://github.com/chipsalliance/silicon-notebooks/blob/74579085cd72205f3f0b7849aad31257638fa5fa/serv-openlane-gf180mcu.ipynb ?

mithro commented 1 year ago

Title / description of the first cell seems to still be Digital inverter with OpenLane?

olofk commented 1 year ago

Cool! As you mentioned, it would be great to have this running with SRAM instead. I suspect that would make it like ten times smaller :)

Also, would be cool to see if we can run this through FuseSoC for better scalability

proppy commented 1 year ago

You can view the notebook at https://github.com/chipsalliance/silicon-notebooks/blob/74579085cd72205f3f0b7849aad31257638fa5fa/serv-openlane-gf180mcu.ipynb ?

https://colab.research.google.com/github/proppy/silicon-notebooks/blob/serv/serv-openlane-gf180mcu.ipynb should give you the current HEAD of the pr branch.

proppy commented 1 year ago

Title / description of the first cell seems to still be Digital inverter with OpenLane?

Fixed with 3478d5602966721fe315e88eaa6bfea1e56645cb.

proppy commented 1 year ago

it would be great to have this running with SRAM instead. I suspect that would make it like ten times smaller :)

Do you think this should be done in this PR, or should be file an issue about it?

Also, would be cool to see if we can run this through FuseSoC for better scalability

Same question!

proppy commented 1 year ago

Also, would be cool to see if we can run this through FuseSoC for better scalability

We might even want a different notebook? serv-fusesoc-gf180mcu.ipynb.

mithro commented 1 year ago

it would be great to have this running with SRAM instead. I suspect that would make it like ten times smaller :)

Do you think this should be done in this PR, or should be file an issue about it?

File an issue.

Also, would be cool to see if we can run this through FuseSoC for better scalability

Same question!

File an issue.

proppy commented 1 year ago

File an issue.

Done

File an issue.

Done

mithro commented 1 year ago

Merge away?