mb/google/nissa: Remove runtime descriptor updates and VBT selection

The infrastructure for selecting an appropriate firmware image to use
the right descriptor is now ready so runtime descriptor updates are no
longer necessary. Since the different descriptor builds split along
HDMI/USB-C lines for nereid, a single VBT file can be used for each,
removing the need for runtime VBT selection as well.

BUG=b:229022567
TEST=Nereid type-C and HDMI outputs work as expected

Signed-off-by: Sam McNally <sammc@chromium.org>
Change-Id: Idf1fbd6c26203adbda002dec3f11e54a7b9f9b82
Reviewed-on: https://review.coreboot.org/c/coreboot/+/66826
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Reka Norman <rekanorman@chromium.org>

3 files changed, 0 insertions, 82 deletions

diff --git a/src/mainboard/google/brya/variants/joxer/variant.c b/src/mainboard/google/brya/variants/joxer/variant.c
index d51927c631..3a0522078d 100644
--- a/src/mainboard/google/brya/variants/joxer/variant.c
+++ b/src/mainboard/google/brya/variants/joxer/variant.c
@@ -10,41 +10,3 @@ const char *get_wifi_sar_cbfs_filename(void)
 {
 	return "wifi_sar_0.hex";
 }
-
-/*
- * TODO(b/229022567): This is a workaround which will be removed once we
- * implement a proper solution for configuring the descriptor differently for
- * different SKUs.
- */
-void variant_update_descriptor(void)
-{
-	/*
-	 * UfsCont1Config = "Disabled"
-	 * IshSupported = "No"
-	 */
-	struct descriptor_byte emmc_bytes[] = {
-		{ 0x1f8, 0x55 },
-		{ 0x1f9, 0x55 },
-		{ 0xc18, 0x89 },
-		{ 0xc1d, 0xb8 },
-	};
-
-	/*
-	 * UfsCont1Config = "X2"
-	 * IshSupported = "Yes"
-	 */
-	struct descriptor_byte ufs_bytes[] = {
-		{ 0x1f8, 0x95 },
-		{ 0x1f9, 0x59 },
-		{ 0xc18, 0x09 },
-		{ 0xc1d, 0x28 },
-	};
-
-	if (fw_config_probe(FW_CONFIG(STORAGE, STORAGE_UFS))) {
-		printk(BIOS_INFO, "Configuring descriptor for UFS\n");
-		configure_descriptor(ufs_bytes, ARRAY_SIZE(ufs_bytes));
-	} else {
-		printk(BIOS_INFO, "Configuring descriptor for eMMC\n");
-		configure_descriptor(emmc_bytes, ARRAY_SIZE(emmc_bytes));
-	}
-}
diff --git a/src/mainboard/google/brya/variants/nereid/Makefile.inc b/src/mainboard/google/brya/variants/nereid/Makefile.inc
index 16c9748225..defb592f2f 100644
--- a/src/mainboard/google/brya/variants/nereid/Makefile.inc
+++ b/src/mainboard/google/brya/variants/nereid/Makefile.inc
@@ -1,9 +1,7 @@
 # SPDX-License-Identifier: GPL-2.0-only
 bootblock-y += gpio.c
-bootblock-y += variant.c
 
 romstage-y += gpio.c
 romstage-y += memory.c
 
 ramstage-y += gpio.c
-ramstage-y += variant.c
diff --git a/src/mainboard/google/brya/variants/nereid/variant.c b/src/mainboard/google/brya/variants/nereid/variant.c
deleted file mode 100644
index 74afb2702b..0000000000
--- a/src/mainboard/google/brya/variants/nereid/variant.c
+++ /dev/null
@@ -1,42 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-or-later */
-
-#include <baseboard/variants.h>
-#include <console/console.h>
-#include <drivers/intel/gma/opregion.h>
-#include <fw_config.h>
-#include <soc/bootblock.h>
-
-const char *mainboard_vbt_filename(void)
-{
-	if (fw_config_probe(FW_CONFIG(DB_USB, DB_1A_HDMI)))
-		return "vbt-nereid_hdmi.bin";
-
-	return "vbt.bin";
-}
-
-void variant_update_descriptor(void)
-{
-	/* TypeCPort2Config = "No Thunderbolt" */
-	struct descriptor_byte typec_bytes[] = {
-		{ 0xc76, 0xb7 },
-		{ 0xc77, 0xb6 },
-		{ 0xc7c, 0xee },
-		{ 0xca0, 0x0c },
-	};
-
-	/* TypeCPort2Config = "DP Fixed Connection" */
-	struct descriptor_byte hdmi_bytes[] = {
-		{ 0xc76, 0x75 },
-		{ 0xc77, 0xc4 },
-		{ 0xc7c, 0x1e },
-		{ 0xca0, 0x0e },
-	};
-
-	if (fw_config_probe(FW_CONFIG(DB_USB, DB_1A_HDMI))) {
-		printk(BIOS_INFO, "Configuring descriptor for HDMI\n");
-		configure_descriptor(hdmi_bytes, ARRAY_SIZE(hdmi_bytes));
-	} else {
-		printk(BIOS_INFO, "Configuring descriptor for Type-C\n");
-		configure_descriptor(typec_bytes, ARRAY_SIZE(typec_bytes));
-	}
-}