[coreboot-gerrit] Change in ...coreboot[master]: drivers/intel/wifi: Select DRIVERS_INTEL_WIFI if your mainboard needs it

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33827 )

Change subject: drivers/intel/wifi: Select DRIVERS_INTEL_WIFI if your mainboard needs it
......................................................................

Patch Set 1:
> Patch Set 1:
> > Patch Set 1: Code-Review-2
>
> why Should I compile "src/drivers/intel/wifi/wifi.c" if the board I have do not have intel's wifi ?
> is it for smbios and acpi ?

The only way to determine at compile-time if that driver is needed is to have the user specify so. Default builds should work in as many cases as possible, so disabling this driver by default means NOBODY with an intel wireless card will be able to use it with a default coreboot build.

If you do not want that driver because you do not have an intel wireless card, use site-local to disable that option by default for yourself.