Page MenuHomeFreeBSD

sysutils/flashrom: Enable buspirate interface by default
ClosedPublic

Authored by manu on Sep 28 2018, 7:08 PM.
Tags
None
Referenced Files
F115253223: D17350.id48658.diff
Mon, Apr 21, 9:53 PM
Unknown Object (File)
Sat, Apr 19, 5:37 AM
Unknown Object (File)
Fri, Apr 11, 5:10 PM
Unknown Object (File)
Wed, Mar 26, 11:37 AM
Unknown Object (File)
Sun, Mar 23, 1:45 PM
Unknown Object (File)
Mar 16 2025, 4:00 PM
Unknown Object (File)
Mar 16 2025, 3:09 PM
Unknown Object (File)
Mar 15 2025, 5:01 AM
Subscribers

Details

Summary

Tested with my buspirate and two spi flash.
While here, pet portlint.

Diff Detail

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

Event Timeline

sysutils/flashrom/Makefile
35–41 ↗(On Diff #48560)
BUSPIRATE_MAKE_ENV=        CONFIG_BUSPIRATE_SPI=yes
BUSPIRATE_MAKE_ENV_OFF=    CONFIG_BUSPIRATE_SPI=no
manu marked an inline comment as done.Oct 1 2018, 1:25 PM
manu added inline comments.
sysutils/flashrom/Makefile
35–41 ↗(On Diff #48560)

Thanks ! I was looking for something like that but couldn't find it in the handbook, I see it know of course.

This revision is now accepted and ready to land.Oct 2 2018, 2:55 PM

Always enable buspirate support since it's enabled by default upstream and just use serial port.

This revision now requires review to proceed.Oct 2 2018, 8:19 PM
manu marked an inline comment as done.Oct 2 2018, 8:20 PM
manu added inline comments.
sysutils/flashrom/Makefile
35–41 ↗(On Diff #48560)

Just FYI it didn't worked, it was always compile with buspirate support despite the make config changes.
Not a problem since we want to always have support for it since it's enabled by default upstream.

sysutils/flashrom/Makefile
34 ↗(On Diff #48658)

Remove.

36 ↗(On Diff #48658)

Why += ?

manu marked 2 inline comments as done.Oct 3 2018, 9:16 AM
manu added inline comments.
sysutils/flashrom/Makefile
34 ↗(On Diff #48658)

Right, will do.

36 ↗(On Diff #48658)

Wasn't done by me, I'll fix it if necessary.

Remove bsd.ports.options.mk
Remove += for RUN_DEPEND

manu marked 3 inline comments as done.Oct 4 2018, 3:11 PM
This revision is now accepted and ready to land.Oct 4 2018, 3:22 PM
This revision was automatically updated to reflect the committed changes.