but I have contents in the magic_ext2spice.feedbac...
# mpw-one-clean-short
m
but I have contents in the magic_ext2spice.feedback.txt file: feedback add "Illegal overlap (types do not connect)" medium
a
@Matt Venn: Are you still using the old Fake Diodes?
m
yes, just looking at metal5 short for now
ah, this could be to do with that as well.
good call, I'll update and run again
a
@Matt Venn: Yeah that's probably it. The old fake diodes would flag the illegal overlaps as well
j
hi @Matt Venn any update? should we re-pull your project and run again?
m
Not yet
I have made some progress but not pushed anything
with @Ahmed Ghazy help I have fixed the overlap and shorts errors
but I'm concerned about antenna issues with my aggregated designs
I didn't realise this before but antenna rules aren't checked at the macro level. So I have 8 blocks inside the user project area, all connected with long wires
If this is going to be a show stopper for all the designs then I'd like to fix this somehow, but need some help to understand how serious an issue it is and how to manually place antenna diodes at the inputs to all the modules
j
@Matt Venn we have not been chasing antenna errors and are choosing waive them since they only affect yield. is there a particular concern you have relative to your design(s)?
m
Here's my user project wrapper:
long interconnecting wires
j
@Tim Edwards any concern? ^^^
t
@jeffdi: Well, I can't see squat from the screenshot. Tolerance of antenna violations is up to the user; I've been recommending users to do something about antenna violations that are more than 2x the recommended ratio limit, but I have no actual yield data, so it's up to the user to assume the risk.
m
I had a chat with @Ahmed Ghazy about it and it seems that with the current tools the best I can do is use diode strategy 1 to make sure antenna diodes are on all the inputs
but my point is that the distance between the macros is greater than the distance inside the macros
and afaik we are not checking antenna rules at user_project_wrapper. so a long wire there could cause an issue that isn't ever checked
am I making any sense?
btw @jeffdi I am making good progress, should have it pushed tomorrow
@jeffdi my repo is updated: https://github.com/mattvenn/caravel-mph
mpw-check is failing with 75 DRC
as expected