Page MenuHomeFreeBSD

Update of FPGA toolchain for iCE40 FPGAs.
ClosedPublic

Authored by jsorocil_gmail.com on Oct 21 2018, 9:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jun 15, 4:52 AM
Unknown Object (File)
Sat, Jun 8, 10:42 AM
Unknown Object (File)
Fri, Jun 7, 4:05 AM
Unknown Object (File)
Thu, Jun 6, 8:26 AM
Unknown Object (File)
Tue, Jun 4, 5:48 PM
Unknown Object (File)
Tue, Jun 4, 5:48 PM
Unknown Object (File)
Tue, Jun 4, 5:48 PM
Unknown Object (File)
Tue, Jun 4, 5:48 PM
Subscribers

Details

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

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

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 added inline comments.
devel/icestorm/Makefile
22 ↗(On Diff #49418)

You can use the short version. :-)

tobik added a subscriber: 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.

This revision is now accepted and ready to land.Oct 22 2018, 5:41 PM
devel/icestorm/Makefile
22 ↗(On Diff #49418)

I was not criticizing but informing. ;-)

  • Update of FPGA toolchain for iCE40 FPGAs.
This revision now requires review to proceed.Oct 23 2018, 7:14 AM
jsorocil_gmail.com added inline comments.
devel/yosys/Makefile
27 ↗(On Diff #49418)

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

devel/yosys/Makefile
27 ↗(On Diff #49418)

Set DISTVERSIONPREFIX=yosys- instead.

jsorocil_gmail.com marked an inline comment as done.

Update of FPGA toolchain for iCE40 FPGAs

This revision is now accepted and ready to land.Oct 23 2018, 3:53 PM

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

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.

This revision was automatically updated to reflect the committed changes.