Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31219 )
Change subject: src/drivers/intel/wifi/Kconfig: add NO_INTEL_WIFI option ......................................................................
Patch Set 1: Code-Review-1
(1 comment)
I would like to know the reasoning behind this change.
AFAIUI, the current default is meant to make Intel wireless cards work by default, since different people have different hardware configurations. It does no harm to those without such hardware, and makes the default configuration for a board more likely to work well. And anyone who doesn't want to use the driver can disable it in Kconfig.
https://review.coreboot.org/#/c/31219/1/src/drivers/intel/wifi/Kconfig File src/drivers/intel/wifi/Kconfig:
https://review.coreboot.org/#/c/31219/1/src/drivers/intel/wifi/Kconfig@4 PS1, Line 4: default y if PCIEXP_PLUGIN_SUPPORT && !NO_INTEL_WIFI
iirc, the decision to default to `y` was because you can never […]
I agree, negative Kconfig options are usually troublesome.