edalize icon indicating copy to clipboard operation
edalize copied to clipboard

Add support for official gowin tool

Open flaminggoat opened this issue 1 year ago • 3 comments

Example core:

  tang_nano_20k:
    default_tool: gowin
    description : Sipeed Tang Nano 20k
    filesets: [rtl, tang_nano_20k]
    flow: gowin
    flow_options:
      tool: gowin
      part: GW2AR-LV18QN88C8/I7
      gowin_options:
        - -multi_boot 1
    toplevel: top

flaminggoat avatar Jun 24 '24 17:06 flaminggoat

Thanks for adding this. Bonus points for also adding tests. Unfortunately this implementation is written for the Tool API that is about to be deprecated, and all new backends must instead support the Flow API. In practice this means adding a file in the tools subdirectory and then probably a short file in the flows subdirectory as well. I am fully aware that I haven't communicated this change very clearly and I apologize for that. Hopefully it's not too much work to change this. I think the efinity files can be useful as examples for how to write these files.

Let me know if you need assistance.

olofk avatar Jun 25 '24 21:06 olofk

Very good. I think I found a typo, but looks fine otherwise. Appreciate if you could throw in some tests too. Doesn't have to be anything advanced. Check e.g. test_tool_verilator.py for reference and let me know if you have any questions.

olofk avatar Jul 05 '24 11:07 olofk

@olofk I think that should be resolved now

flaminggoat avatar Jul 12 '24 12:07 flaminggoat

Sorry for taking so long to get back to this. It looks all fine and dandy. Picked and pushed. Thanks!

olofk avatar Nov 04 '24 22:11 olofk