GOWIN: Fixing locations and splitting timing report at correct locations

[tiagosr]

188 Tested with Gowin EDA, it works well 😃

Added parsing of multiple path reports Also, Gowin's .sdc format requires adding {} around clock names in [get_ports ...] - if other tools don't like this, might need to add some logic to SYN.py to select for that.

[JulianKemmerer]

Looking fantastic - glad to hear its starting to work well :)

Ill be making some minor comments on certain things...

[JulianKemmerer]

Btw Ill run some tests regarding {} around clock names :+1: ...

[tiagosr]

Question: I see there's some files that were committed to the path_delay_cache and pipeline_min_period_cache directories for other parts, should I commit the ones generated for the parts I'm testing it with?

[JulianKemmerer]

Question: I see there's some files that were committed to the path_delay_cache and pipeline_min_period_cache directories for other parts, should I commit the ones generated for the parts I'm testing it with?

I think there is one last thing to change - iirc the directory is like gowin/GW2AR-LV18QN88PC8:C/pnr - critically pnr talking about. That is trying to reflect if these timing numbers are synthesis estimates or full real pnr placed and routed values...

IIRC the HTML report is post synthesis - so need to change the part of code which looks up if the tool is set to syn or pnr and return False instead of True https://github.com/JulianKemmerer/PipelineC/blob/bd55aa790e761ddb15051247dcab172189d02ccd/src/SYN.py#L195

Minor thing syntax thing: I feel like I once saw the python is compare syntax is prefered for some reason here? for consistency stick with that when checking tool ex. if SYN_TOOL is VIVADO: etc, change in LOGIC_IS_ZERO_DELAY too

once the directory is producing syn paths - those path_delay_cache files should be checked in please.

I am still not certain about how to use pipeline_min_period_cache values and there tend to be alot of them over time ... so I tend to not commit that actually - little bit of a research-level feature at the moment

[JulianKemmerer]

Looking good to me. I hopefully will be installing the gowin tools and trying to get through the full flow myself soon :crossed_fingers:

Thanks again for all the strong work :muscle:

[JulianKemmerer]

@tiagosr are you on the Discord by chance? come by and chat :nerd_face: btw when I do get this working locally - I want to credit you gowin tool support on with mastodon/twitter post and such so if you have any links for yourself to share feel free