Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37768 )
Change subject: util/superiotool: alter Makefile to build the binary on FreeBSD ......................................................................
Patch Set 2:
the pciutils target is to detect the presence of the libpci headers and print a message which libraries are probably missing; with this patch the detection/message doesn't get called any more and leaves dead code in the makefile. There is some BSD-specific case in the test tool that's embedded in the makefile (why?!), so without having really looked into that issue, I'd guess that the proper fix would either be fixing the test tool for your platform without or removing the now dead code in the makefile and adding the information to the readme. i'd be ok with both of those