Mathew King has uploaded this change for review.

View Change

mb/google/zork: Remove unsused code

Remove unused code that appears to be left over from grunt.

Signed-off-by: Mathew King <mathewk@chromium.org>
Change-Id: Id5bdb1c957342d55c5e6378c503b8d90da050601
---
M src/mainboard/google/zork/mainboard.c
M src/mainboard/google/zork/variants/baseboard/gpio_baseboard_dalboz.c
M src/mainboard/google/zork/variants/baseboard/gpio_baseboard_trembyle.c
M src/mainboard/google/zork/variants/baseboard/include/baseboard/variants.h
4 files changed, 0 insertions(+), 36 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/05/48505/1

diff --git a/src/mainboard/google/zork/mainboard.c b/src/mainboard/google/zork/mainboard.c
index 5ab90cf..413f34a 100644
--- a/src/mainboard/google/zork/mainboard.c
+++ b/src/mainboard/google/zork/mainboard.c
@@ -162,17 +162,6 @@
 
 	/* Update DUT configuration */
 	mainboard_devtree_update();
-
-	/*
-	 * Some platforms use SCI not generated by a GPIO pin (event above 23).
-	 * For these boards, gpe_configure_sci() is still needed, but all GPIO
-	 * generated events (23-0) must be removed from gpe_table[].
-	 * For boards that only have GPIO generated events, table gpe_table[]
-	 * must be removed, and get_gpe_table() should return NULL.
-	 */
-	gpes = variant_gpe_table(&num);
-	if (gpes != NULL)
-		gpe_configure_sci(gpes, num);
 }
 
 void mainboard_get_dxio_ddi_descriptors(const fsp_dxio_descriptor **dxio_descs,
diff --git a/src/mainboard/google/zork/variants/baseboard/gpio_baseboard_dalboz.c b/src/mainboard/google/zork/variants/baseboard/gpio_baseboard_dalboz.c
index bf27abf..12d2890 100644
--- a/src/mainboard/google/zork/variants/baseboard/gpio_baseboard_dalboz.c
+++ b/src/mainboard/google/zork/variants/baseboard/gpio_baseboard_dalboz.c
@@ -169,18 +169,6 @@
 	return gpio_set_stage_ram;
 }
 
-/*
- * This function is still needed for boards that sets gevents above 23
- * that will generate SCI or SMI. Normally this function
- * points to a table of gevents and what needs to be set. The code that
- * calls it was modified so that when this function returns NULL then the
- * caller does nothing.
- */
-const __weak struct sci_source *variant_gpe_table(size_t *num)
-{
-	return NULL;
-}
-
 static void wifi_power_reset_configure_active_low_power(void)
 {
 	/*
diff --git a/src/mainboard/google/zork/variants/baseboard/gpio_baseboard_trembyle.c b/src/mainboard/google/zork/variants/baseboard/gpio_baseboard_trembyle.c
index 0745eda..a2ad517 100644
--- a/src/mainboard/google/zork/variants/baseboard/gpio_baseboard_trembyle.c
+++ b/src/mainboard/google/zork/variants/baseboard/gpio_baseboard_trembyle.c
@@ -181,18 +181,6 @@
 	return gpio_set_stage_ram;
 }
 
-/*
- * This function is still needed for boards that sets gevents above 23
- * that will generate SCI or SMI. Normally this function
- * points to a table of gevents and what needs to be set. The code that
- * calls it was modified so that when this function returns NULL then the
- * caller does nothing.
- */
-const __weak struct sci_source *variant_gpe_table(size_t *num)
-{
-	return NULL;
-}
-
 static void wifi_power_reset_configure_active_low_power(void)
 {
 	/*
diff --git a/src/mainboard/google/zork/variants/baseboard/include/baseboard/variants.h b/src/mainboard/google/zork/variants/baseboard/include/baseboard/variants.h
index 4ec6add..f6e7e7c 100644
--- a/src/mainboard/google/zork/variants/baseboard/include/baseboard/variants.h
+++ b/src/mainboard/google/zork/variants/baseboard/include/baseboard/variants.h
@@ -9,7 +9,6 @@
 #include <soc/platform_descriptors.h>
 #include "chip.h"
 
-const struct sci_source *variant_gpe_table(size_t *num);
 const struct soc_amd_gpio *variant_early_gpio_table(size_t *size);
 /*
  * This function provides base GPIO configuration table. It is typically provided by

To view, visit change 48505. To unsubscribe, or for help writing mail filters, visit settings.