In verilog model of standard cells, fix "notifier" register definition

efabless / caravel_mgmt_soc_litex

https://caravel-mgmt-soc-litex.readthedocs.io/en/latest/

Apache License 2.0

25 stars 15 forks source link

Open derekcom17 opened 2 years ago

derekcom17 commented 2 years ago

Make these fixes in several places:

In some cases, the notifier was defined as type "wire" which is invalid for some tools.
Add the defenition of notifier in the cases where it was absent.
Fix an issue where a wire was defined twice in the same line, which the VCS simulator didn't like.

These issues I had may have been tool-specific, but all these changes should be backwards compatible.

RTimothyEdwards commented 1 year ago

We should not have PDK files in this repository---they should be fixed in the PDK.