From 31f6320afeedfe622b0af51360078be28d26e378 Mon Sep 17 00:00:00 2001 From: Furquan Shaikh Date: Sat, 13 Mar 2021 22:34:57 -0800 Subject: drivers/wifi, mb/google: Drop config `WIFI_SAR_CBFS` Now that SAR support in VPD is deprecated in coreboot, there is no need for a separate Kconfig `WIFI_SAR_CBFS` as the SAR table is only supported as a CBFS file. This change drops the config `WIFI_SAR_CBFS` from drivers/wifi/generic/Kconfig and its selection in mb/google/.../Kconfig. wifi_sar_defaults.hex is added to CBFS only if CONFIG_WIFI_SAR_CBFS_FILEPATH is not empty because current mainboards do not provide a default SAR file in coreboot. Thus, CONFIG_WIFI_SAR_CBFS_FILEPATH is updated to have a default value of "". BUG=b:173465272 Cq-Depend: chromium:2757781 Change-Id: I0bb8f6e2511596e4503fe4d8c34439228ceaa3c7 Signed-off-by: Furquan Shaikh Reviewed-on: https://review.coreboot.org/c/coreboot/+/51484 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi Reviewed-by: Tim Wawrzynczak --- src/drivers/wifi/generic/Kconfig | 17 ++--------------- src/drivers/wifi/generic/Makefile.inc | 10 ++++++++-- 2 files changed, 10 insertions(+), 17 deletions(-) (limited to 'src/drivers/wifi') diff --git a/src/drivers/wifi/generic/Kconfig b/src/drivers/wifi/generic/Kconfig index 43c7d9ee1b..1d0e19f005 100644 --- a/src/drivers/wifi/generic/Kconfig +++ b/src/drivers/wifi/generic/Kconfig @@ -21,9 +21,6 @@ config USE_SAR default n help Enable it when wifi driver uses SAR configuration feature. - VPD entry "wifi_sar" is read to get SAR settings, if its - not found driver may look into CBFS for default settigs. - WIFI_SAR_CBFS is option to enable CBFS lookup. config SAR_ENABLE bool @@ -40,20 +37,10 @@ config GEO_SAR_ENABLE default n depends on USE_SAR -config WIFI_SAR_CBFS - bool "Enable SAR table addition to CBFS" - default n - depends on USE_SAR - help - wifi driver would look for "wifi_sar" vpd key and load SAR settings from - it, if the vpd key is not found then the driver tries to look for sar - settings from CBFS with file name wifi_sar_defaults.hex. - So OEM/ODM can override wifi sar with VPD. - config WIFI_SAR_CBFS_FILEPATH string "The cbfs file which has WIFI SAR defaults" - depends on WIFI_SAR_CBFS - default "src/mainboard/\$(MAINBOARDDIR)/wifi_sar_defaults.hex" + depends on USE_SAR + default "" config DSAR_SET_NUM hex "Number of SAR sets when D-SAR is enabled" diff --git a/src/drivers/wifi/generic/Makefile.inc b/src/drivers/wifi/generic/Makefile.inc index c17844993d..5650d8ab6f 100644 --- a/src/drivers/wifi/generic/Makefile.inc +++ b/src/drivers/wifi/generic/Makefile.inc @@ -5,8 +5,14 @@ ramstage-y += generic.c ramstage-$(CONFIG_GENERATE_SMBIOS_TABLES) += smbios.c ramstage-$(CONFIG_HAVE_ACPI_TABLES) += acpi.c -cbfs-files-$(CONFIG_WIFI_SAR_CBFS) += wifi_sar_defaults.hex -wifi_sar_defaults.hex-file := $(call strip_quotes,$(CONFIG_WIFI_SAR_CBFS_FILEPATH)) +CONFIG_WIFI_SAR_CBFS_FILEPATH := $(call strip_quotes,$(CONFIG_WIFI_SAR_CBFS_FILEPATH)) + +ifneq ($(CONFIG_WIFI_SAR_CBFS_FILEPATH),) + +cbfs-files-$(CONFIG_USE_SAR) += wifi_sar_defaults.hex +wifi_sar_defaults.hex-file := $(CONFIG_WIFI_SAR_CBFS_FILEPATH) wifi_sar_defaults.hex-type := raw endif + +endif -- cgit v1.2.3