mb/google/auron: Use `get_gpios` function

Change-Id: I91424a45ae67186987630b7686102f467f57e7ee
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/49779
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Michael Niewöhner <foss@mniewoehner.de>

5 files changed, 20 insertions, 79 deletions

diff --git a/src/mainboard/google/auron/variants/auron_paine/spd/spd.c b/src/mainboard/google/auron/variants/auron_paine/spd/spd.c
index ba829d0a78..88487d5a5a 100644
--- a/src/mainboard/google/auron/variants/auron_paine/spd/spd.c
+++ b/src/mainboard/google/auron/variants/auron_paine/spd/spd.c
@@ -19,25 +19,14 @@
 /* Copy SPD data for on-board memory */
 void mainboard_fill_spd_data(struct pei_data *pei_data)
 {
-	int spd_bits[3] = {
+	const int gpio_vector[] = {
 		SPD_GPIO_BIT0,
 		SPD_GPIO_BIT1,
-		SPD_GPIO_BIT2
+		SPD_GPIO_BIT2,
+		-1,
 	};
-	int spd_gpio[3];
-	int spd_index;
 
-	spd_gpio[0] = get_gpio(SPD_GPIO_BIT0);
-	spd_gpio[1] = get_gpio(SPD_GPIO_BIT1);
-	spd_gpio[2] = get_gpio(SPD_GPIO_BIT2);
-
-	spd_index = spd_gpio[2] << 2 | spd_gpio[1] << 1 | spd_gpio[0];
-
-	printk(BIOS_DEBUG, "SPD: index %d (GPIO%d=%d GPIO%d=%d GPIO%d=%d)\n",
-	       spd_index,
-	       spd_bits[2], spd_gpio[2],
-	       spd_bits[1], spd_gpio[1],
-	       spd_bits[0], spd_gpio[0]);
+	const unsigned int spd_index = get_gpios(gpio_vector);
 
 	fill_spd_for_index(pei_data->spd_data[0][0], spd_index);
 
diff --git a/src/mainboard/google/auron/variants/auron_yuna/spd/spd.c b/src/mainboard/google/auron/variants/auron_yuna/spd/spd.c
index ba829d0a78..88487d5a5a 100644
--- a/src/mainboard/google/auron/variants/auron_yuna/spd/spd.c
+++ b/src/mainboard/google/auron/variants/auron_yuna/spd/spd.c
@@ -19,25 +19,14 @@
 /* Copy SPD data for on-board memory */
 void mainboard_fill_spd_data(struct pei_data *pei_data)
 {
-	int spd_bits[3] = {
+	const int gpio_vector[] = {
 		SPD_GPIO_BIT0,
 		SPD_GPIO_BIT1,
-		SPD_GPIO_BIT2
+		SPD_GPIO_BIT2,
+		-1,
 	};
-	int spd_gpio[3];
-	int spd_index;
 
-	spd_gpio[0] = get_gpio(SPD_GPIO_BIT0);
-	spd_gpio[1] = get_gpio(SPD_GPIO_BIT1);
-	spd_gpio[2] = get_gpio(SPD_GPIO_BIT2);
-
-	spd_index = spd_gpio[2] << 2 | spd_gpio[1] << 1 | spd_gpio[0];
-
-	printk(BIOS_DEBUG, "SPD: index %d (GPIO%d=%d GPIO%d=%d GPIO%d=%d)\n",
-	       spd_index,
-	       spd_bits[2], spd_gpio[2],
-	       spd_bits[1], spd_gpio[1],
-	       spd_bits[0], spd_gpio[0]);
+	const unsigned int spd_index = get_gpios(gpio_vector);
 
 	fill_spd_for_index(pei_data->spd_data[0][0], spd_index);
 
diff --git a/src/mainboard/google/auron/variants/gandof/spd/spd.c b/src/mainboard/google/auron/variants/gandof/spd/spd.c
index 5cd3ddcb15..9c323144e4 100644
--- a/src/mainboard/google/auron/variants/gandof/spd/spd.c
+++ b/src/mainboard/google/auron/variants/gandof/spd/spd.c
@@ -19,25 +19,14 @@
 /* Copy SPD data for on-board memory */
 void mainboard_fill_spd_data(struct pei_data *pei_data)
 {
-	int spd_bits[3] = {
+	const int gpio_vector[] = {
 		SPD_GPIO_BIT0,
 		SPD_GPIO_BIT1,
-		SPD_GPIO_BIT2
+		SPD_GPIO_BIT2,
+		-1,
 	};
-	int spd_gpio[3];
-	int spd_index;
 
-	spd_gpio[0] = get_gpio(SPD_GPIO_BIT0);
-	spd_gpio[1] = get_gpio(SPD_GPIO_BIT1);
-	spd_gpio[2] = get_gpio(SPD_GPIO_BIT2);
-
-	spd_index = spd_gpio[2] << 2 | spd_gpio[1] << 1 | spd_gpio[0];
-
-	printk(BIOS_DEBUG, "SPD: index %d (GPIO%d=%d GPIO%d=%d GPIO%d=%d)\n",
-	       spd_index,
-	       spd_bits[2], spd_gpio[2],
-	       spd_bits[1], spd_gpio[1],
-	       spd_bits[0], spd_gpio[0]);
+	const unsigned int spd_index = get_gpios(gpio_vector);
 
 	fill_spd_for_index(pei_data->spd_data[0][0], spd_index);
 
diff --git a/src/mainboard/google/auron/variants/lulu/spd/spd.c b/src/mainboard/google/auron/variants/lulu/spd/spd.c
index d80d3c0424..8d43464648 100644
--- a/src/mainboard/google/auron/variants/lulu/spd/spd.c
+++ b/src/mainboard/google/auron/variants/lulu/spd/spd.c
@@ -20,29 +20,15 @@
 /* Copy SPD data for on-board memory */
 void mainboard_fill_spd_data(struct pei_data *pei_data)
 {
-	int spd_bits[4] = {
+	const int gpio_vector[] = {
 		SPD_GPIO_BIT0,
 		SPD_GPIO_BIT1,
 		SPD_GPIO_BIT2,
-		SPD_GPIO_BIT3
+		SPD_GPIO_BIT3,
+		-1,
 	};
-	int spd_gpio[4];
-	int spd_index;
 
-	spd_gpio[0] = get_gpio(SPD_GPIO_BIT0);
-	spd_gpio[1] = get_gpio(SPD_GPIO_BIT1);
-	spd_gpio[2] = get_gpio(SPD_GPIO_BIT2);
-	spd_gpio[3] = get_gpio(SPD_GPIO_BIT3);
-
-	spd_index = (spd_gpio[3] << 3) | (spd_gpio[2] << 2) |
-		(spd_gpio[1] << 1) | spd_gpio[0];
-
-	printk(BIOS_DEBUG, "SPD: index %d (GPIO%d=%d GPIO%d=%d GPIO%d=%d GPIO%d=%d)\n",
-		spd_index,
-		spd_bits[3], spd_gpio[3],
-		spd_bits[2], spd_gpio[2],
-		spd_bits[1], spd_gpio[1],
-		spd_bits[0], spd_gpio[0]);
+	const unsigned int spd_index = get_gpios(gpio_vector);
 
 	/* CH0 */
 	fill_spd_for_index(pei_data->spd_data[0][0], spd_index);
diff --git a/src/mainboard/google/auron/variants/samus/spd/spd.c b/src/mainboard/google/auron/variants/samus/spd/spd.c
index 25ff928b6d..06270b8ebb 100644
--- a/src/mainboard/google/auron/variants/samus/spd/spd.c
+++ b/src/mainboard/google/auron/variants/samus/spd/spd.c
@@ -20,27 +20,15 @@
 /* Copy SPD data for on-board memory */
 void mainboard_fill_spd_data(struct pei_data *pei_data)
 {
-	int spd_bits[4] = {
+	const int gpio_vector[] = {
 		SPD_GPIO_BIT0,
 		SPD_GPIO_BIT1,
 		SPD_GPIO_BIT2,
-		SPD_GPIO_BIT3
+		SPD_GPIO_BIT3,
+		-1,
 	};
-	int spd_gpio[4];
-	int spd_index;
 
-	spd_gpio[0] = get_gpio(spd_bits[0]);
-	spd_gpio[1] = get_gpio(spd_bits[1]);
-	spd_gpio[2] = get_gpio(spd_bits[2]);
-	spd_gpio[3] = get_gpio(spd_bits[3]);
-
-	spd_index = (spd_gpio[3] << 3) | (spd_gpio[2] << 2) |
-		(spd_gpio[1] << 1) | spd_gpio[0];
-
-	printk(BIOS_DEBUG, "SPD: index %d (GPIO%d=%d GPIO%d=%d "
-	       "GPIO%d=%d GPIO%d=%d)\n", spd_index,
-	       spd_bits[3], spd_gpio[3], spd_bits[2], spd_gpio[2],
-	       spd_bits[1], spd_gpio[1], spd_bits[0], spd_gpio[0]);
+	const unsigned int spd_index = get_gpios(gpio_vector);
 
 	fill_spd_for_index(pei_data->spd_data[0][0], spd_index);