Hi I m having XOR fail in precheck can anyone guess why open-source-silicon.dev #gf180mcu

Join Slack

Hi, I'm having XOR fail in precheck, can anyone gu...

# gf180mcu

Cra2yPierr0t

12/04/2022, 5:51 PM

Hi, I'm having XOR fail in precheck, can anyone guess why?

Cra2yPierr0t

12/04/2022, 5:59 PM

I'm starting to think it's the IO area.

Mitch Bailey

12/04/2022, 6:50 PM

What layers?

Cra2yPierr0t

12/04/2022, 6:53 PM

I don't know, I just looked at user_project_wrapper.xor.gds in klayout. Hope this is helpful.

Mitch Bailey

12/04/2022, 6:56 PM

Can you load that on top of your design in klayout?

Cra2yPierr0t

12/04/2022, 7:02 PM

Is it like this? There is certainly something growing there that I don't understand.

Mitch Bailey

12/04/2022, 7:06 PM

Right. There’s a golden version of the wrapper that has the port positions for the connections to caravel. The XOR check makes sure everything’s in the right place.

Cra2yPierr0t

12/04/2022, 7:12 PM

So something is wrong at the

make user_project_wrapper

stage. I'm stumped....

Cra2yPierr0t

12/04/2022, 7:17 PM

golden version of the wrapper is wrong?

Mitch Bailey

12/04/2022, 7:27 PM

Either that or the pin positions in the config file. You could try opening the caravel gds and see where the actual pins should go.

Janani Aravind

12/04/2022, 10:37 PM

@Mitch Bailey, I get the exact same issue! At metal layer 2 and Metal 3 I see the XOR differences. Attaching the reference image.

Cra2yPierr0t

12/04/2022, 11:07 PM

I'm not sure of the conditions, but it seems that some designs have different pin placements, which causes the precheck XOR to fail. user_proj_example did not have this problem, even though I changed the die size. I think that most of the designs currently submitted will not pass the XOR precheck.

Janani Aravind

12/04/2022, 11:12 PM

I agree, my user_proj_example doesn't have the issue either!

Rolf Widenfelt

12/04/2022, 11:35 PM

Is this failing in the efabless submit stage or the pre-check on github?

Cra2yPierr0t

12/04/2022, 11:37 PM

It is failing in the efabless submit stage and local precheck.

👍 1

Rolf Widenfelt

12/04/2022, 11:49 PM

So, this may or may not be helpful, but it appears that the caravel_user_project repo is currently failing its tests (https://github.com/efabless/caravel_user_project). It seems that some people have forked (or cloned) that one.

👀 1

Janani Aravind

12/04/2022, 11:53 PM

@jeffdi, @Matt Venn, is this an issue with Caravel_user_project? What could be the best solution?

Cra2yPierr0t

12/05/2022, 12:14 AM

https://github.com/efabless/caravel_user_project/issues/220

Mitch Bailey

12/05/2022, 2:25 AM

FWIW, the XOR check is only on

user_project_wrapper

and

user_analog_project_wrapper

user_proj_example

is unaffected.

👍 1

Vijayan Krishnan

12/05/2022, 6:05 AM

@Cra2yPierr0t can you share your repo?

Cra2yPierr0t

12/05/2022, 6:07 AM

@Vijayan Krishnan This https://github.com/cpu-dev/caravel_jacaranda-8_GF180

proppy

12/05/2022, 6:20 AM

@Cra2yPierr0t I think you may have a problem with your power pins: power pin が合わない気がする https://github.com/cpu-dev/caravel_jacaranda-8_GF180/blob/main/openlane/computer/config.json#L17-L18 https://github.com/cpu-dev/caravel_jacaranda-8_GF180/blob/main/verilog/rtl/jacaranda-8/computer.v#L19-L20

Vijayan Krishnan

12/05/2022, 6:28 AM

@proppy Good catch. Its good to move power pins inside respective PDK_FAMILY will be permanent fix. new user will not check those pins until they understand the flow.

Cra2yPierr0t

12/05/2022, 6:32 AM

@proppy If both config.json and config.tcl are present, config.tcl is used, so power pin does not seem to be the cause. https://github.com/cpu-dev/caravel_jacaranda-8_GF180/blob/main/openlane/computer/config.tcl#L73-L74

proppy

12/05/2022, 6:43 AM

ah sorry!

😄 1

proppy

12/05/2022, 6:43 AM

it would be best to remove the json file then 😛

jeffdi

12/05/2022, 6:52 AM

@Cra2yPierr0t I posted a reply to the issue - your project looks like it is based on gfmpw-0a. You need to patch more files to get to gfmpw-0d. Your user_project_wrapper is out of date. See the patches in the release notes… https://github.com/efabless/caravel_user_project/releases/tag/gfmpw-0d

Cra2yPierr0t

12/05/2022, 6:56 AM

@jeffdi Wow! Thanks for the answer! I'll fix it and give it a try!

proppy

12/05/2022, 6:57 AM

@Cra2yPierr0t sent https://github.com/cpu-dev/caravel_jacaranda-8_GF180/pull/1 to your fork

proppy

12/05/2022, 6:57 AM

let me know if it fixes it.

Cra2yPierr0t

12/05/2022, 6:59 AM

@proppy Merged!! I'll get back to you shortly! Thank you👍👍👍👍👍