Page MenuHomeFreeBSD

sysutils/rpi-firmware: Update to 1.20180619 and add rpi0-w config.txt
ClosedPublic

Authored by manu on Jul 12 2018, 10:02 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 17, 4:17 AM
Unknown Object (File)
Mon, Apr 14, 8:07 AM
Unknown Object (File)
Mon, Apr 14, 4:51 AM
Unknown Object (File)
Sun, Apr 13, 10:34 PM
Unknown Object (File)
Sun, Apr 13, 10:23 PM
Unknown Object (File)
Sun, Apr 13, 9:25 PM
Unknown Object (File)
Sun, Mar 23, 11:52 PM
Unknown Object (File)
Feb 23 2025, 4:32 AM
Subscribers

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

linimon retitled this revision from rpi-firmware: Update to 1.20180619 and add rpi0-w config.txt to sysutils/rpi-firmware: Update to 1.20180619 and add rpi0-w config.txt.Jul 12 2018, 10:47 AM
linimon added a subscriber: linimon.
linimon added inline comments.
sysutils/rpi-firmware/Makefile
30

If we expect this list to grow, I might suggest adding something like

CONFIG_TXT_FILES=a b c

above, and then adding an iterator here.

This revision now requires changes to proceed.Jul 12 2018, 11:24 AM
sysutils/rpi-firmware/Makefile
30

e.g.

+CONFIG_TXT_FILES= config.txt config_rpi_0_w.txt config_rpi3.txt
+
do-patch:

  • ${CP} ${FILESDIR}/config.txt ${WRKSRC}/boot/
  • ${CP} ${FILESDIR}/config_rpi3.txt ${WRKSRC}/boot/

+.for i in ${CONFIG_TXT_FILES}
+ ${CP} ${FILESDIR}/${i} ${WRKSRC}/boot/
+.endfor

sysutils/rpi-firmware/Makefile
30

phab handily removed my '^-' from the two existing CP lines.

sysutils/rpi-firmware/Makefile
30

The list won't grow.

imp added inline comments.
sysutils/rpi-firmware/Makefile
30

famous last words.

sysutils/rpi-firmware/Makefile
30

I guess it will when the RPI4 will be out.
I'll do the for loop, this is not interesting to even talk about this :)

sysutils/rpi-firmware/Makefile
30

Unless there are more than 3 files in the very close future, doing it this way is better/shorter. (Also, you could run CP only one with all the files, which would be even better.)

manu marked 7 inline comments as done.
This revision was not accepted when it landed; it landed in state Needs Review.Jul 22 2018, 12:00 PM
This revision was automatically updated to reflect the committed changes.