Page MenuHomeFreeBSD

sysutils/flashrom: Enable buspirate interface by default
ClosedPublic

Authored by manu on Sep 28 2018, 7:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 18 2025, 12:49 AM
Unknown Object (File)
Dec 13 2024, 5:15 PM
Unknown Object (File)
Nov 19 2024, 4:33 AM
Unknown Object (File)
Oct 21 2024, 11:46 PM
Unknown Object (File)
Oct 7 2024, 12:59 PM
Unknown Object (File)
Oct 2 2024, 5:28 PM
Unknown Object (File)
Oct 2 2024, 1:43 PM
Unknown Object (File)
Sep 19 2024, 3:32 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.