Page MenuHomeFreeBSD
Differential D17642

Update of FPGA toolchain for iCE40 FPGAs.
ClosedPublic
Actions

Authored by jsorocil_gmail.com on Oct 21 2018, 9:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 16, 10:33 AM
Unknown Object (File)
Fri, Mar 22, 11:37 PM
Unknown Object (File)
Fri, Mar 22, 11:37 PM
Unknown Object (File)
Fri, Mar 22, 11:37 PM
Unknown Object (File)
Fri, Mar 22, 11:36 PM
Unknown Object (File)
Fri, Mar 22, 11:36 PM
Unknown Object (File)
Fri, Mar 22, 7:07 PM
Unknown Object (File)
Mar 9 2024, 3:51 PM
View All 48 Files
Subscribers
rigoletto
tobik

Details

Reviewers
mat
tobik
Group Reviewers
Ports Committers
Commits
rP484933: devel/lattice-ice40-tools: Bump version to match the rest of the iCE40 FPGA…
rP484932: devel/arachne-pnr: Update to newer snapshot
rP484931: devel/icestorm: Update to newer snapshot
rP484926: devel/yosys: Update to 0.8
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

jsorocil_gmail.com created this revision.Oct 21 2018, 9:43 PM
Harbormaster completed remote builds in B20324: Diff 49417.Oct 21 2018, 9:43 PM
jsorocil_gmail.com updated this revision to Diff 49418.Oct 21 2018, 9:49 PM

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
Harbormaster completed remote builds in B20325: Diff 49418.Oct 21 2018, 9:49 PM
rigoletto added a reviewer: mat.Oct 22 2018, 5:10 PM
rigoletto added a subscriber: rigoletto.Oct 22 2018, 5:17 PM
rigoletto added a reviewer: Ports Committers.Oct 22 2018, 5:20 PM
rigoletto added inline comments.
devel/icestorm/Makefile
22 ↗(On Diff #49418)

You can use the short version. :-)

tobik accepted this revision.Oct 22 2018, 5:41 PM
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
rigoletto added inline comments.Oct 22 2018, 6:15 PM
devel/icestorm/Makefile
22 ↗(On Diff #49418)

I was not criticizing but informing. ;-)

jsorocil_gmail.com updated this revision to Diff 49492.Oct 23 2018, 7:14 AM
  • Update of FPGA toolchain for iCE40 FPGAs.
This revision now requires review to proceed.Oct 23 2018, 7:14 AM
Harbormaster completed remote builds in B20351: Diff 49492.Oct 23 2018, 7:14 AM
jsorocil_gmail.com marked an inline comment as done.Oct 23 2018, 7:17 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.

tobik added inline comments.Oct 23 2018, 8:59 AM
devel/yosys/Makefile
27 ↗(On Diff #49418)

Set DISTVERSIONPREFIX=yosys- instead.

jsorocil_gmail.com updated this revision to Diff 49511.Oct 23 2018, 3:51 PM
jsorocil_gmail.com marked an inline comment as done.

Update of FPGA toolchain for iCE40 FPGAs

Harbormaster completed remote builds in B20357: Diff 49511.Oct 23 2018, 3:51 PM
tobik accepted this revision.Oct 23 2018, 3:53 PM
This revision is now accepted and ready to land.Oct 23 2018, 3:53 PM
jsorocil_gmail.com added a comment.Nov 6 2018, 5:01 PM

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

tobik added a comment.Nov 7 2018, 9:59 PM
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.

Closed by commit rP484926: devel/yosys: Update to 0.8 (authored by tobik). · Explain WhyNov 14 2018, 2:43 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
devel/
yosys/
5 lines
6 lines
15 lines

Diff 50408

View Options

head/devel/yosys/Makefile

Loading...
View Options

head/devel/yosys/distinfo

Loading...
View Options

head/devel/yosys/pkg-plist

Loading...