Update of FPGA toolchain for iCE40 FPGAs.

[jsorocil_gmail.com]

Summary

• yosys updated to 0.8 version
• arachne-pnr update to the newest git version (no need to have mounted /proc on FreeBSD)
• icestorm update to the newest git version
• lattice-ice40-tools metaport version update to g20181021

[jsorocil_gmail.com]

Update of FPGA toolchain for iCE40 FPGAs.Update of FPGA toolchain for iCE40 FPGAs.

• yosys updated to 0.8 version
• arachne-pnr update to the newest git version (no need to have mounted /proc on FreeBSD)
• icestorm update to the newest git version
• lattice-ice40-tools metaport version update to g20181021

[rigoletto]

devel/icestorm/Makefile

22 ↗

(On Diff #49418)

You can use the short version. :-)

[tobik]

LGTM except for the small problem in devel/yosys.

devel/icestorm/Makefile

22 ↗

(On Diff #49418)

What's the benefit? There is nothing wrong with it and changing them is just busy work.

If you shorten them at least make them longer than 7 chars to prevent collisions. But I'd just leave them as is.

devel/yosys/Makefile

27 ↗

(On Diff #49418)

If the upstream repo has a 0.8 tag now that matches 5706e90802fdf51a476e769790f6b5b526c57572 then this line can and should be removed.

[rigoletto]

devel/icestorm/Makefile

22 ↗

(On Diff #49418)

I was not criticizing but informing. ;-)

[jsorocil_gmail.com]

• Update of FPGA toolchain for iCE40 FPGAs.

[jsorocil_gmail.com]

devel/yosys/Makefile

27 ↗

(On Diff #49418)

If line is removed then fetch command won't work. So, tag name was used here.

[tobik]

devel/yosys/Makefile

27 ↗

(On Diff #49418)

Set DISTVERSIONPREFIX=yosys- instead.

[jsorocil_gmail.com]

Update of FPGA toolchain for iCE40 FPGAs

[jsorocil_gmail.com]

Do I need to do something else or this will be merged one day?

[tobik]

In D17642#381860, @jsorocil_gmail.com wrote:

Do I need to do something else or this will be merged one day?

I will be merged when a committer finds time to do QA on it. It would be good if you open a PR on https://bugs.freebsd.org for this too (if you haven't already) to get it into the patch queue.