[coreboot-gerrit] Change in coreboot[master]: mb/google/volteer: Make use of fw_config_is_provisioned()

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has submitted this change. ( https://review.coreboot.org/c/coreboot/+/47972 )

Change subject: mb/google/volteer: Make use of fw_config_is_provisioned()
......................................................................

mb/google/volteer: Make use of fw_config_is_provisioned()

In cases when a volteer device is unprovisioned, the safest thing to do
for GPIOs that will normally be used for audio codec buses is to leave
them disabled (configured as PAD_CFG_NC). This patch adds support for
that.

BUG=none
TEST=add debug print to new if branch; remove fw_config from CBI and
see print on console

Change-Id: I8efd101174f6e3d7233d2bf803b680673cada81a
Signed-off-by: Tim Wawrzynczak 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/47972
Reviewed-by: Furquan Shaikh 
Reviewed-by: Nick Vaccaro 
Reviewed-by: Duncan Laurie 
Tested-by: build bot (Jenkins) 
---
M src/mainboard/google/volteer/fw_config.c
1 file changed, 7 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Duncan Laurie: Looks good to me, approved
  Furquan Shaikh: Looks good to me, approved
  Nick Vaccaro: Looks good to me, approved

diff --git a/src/mainboard/google/volteer/fw_config.c b/src/mainboard/google/volteer/fw_config.c
index c444b4c..1025119 100644
--- a/src/mainboard/google/volteer/fw_config.c
+++ b/src/mainboard/google/volteer/fw_config.c
@@ -72,6 +72,13 @@

static void fw_config_handle(void *unused)
 {
+	if (!fw_config_is_provisioned()) {
+		gpio_configure_pads(i2s_disable_pads, ARRAY_SIZE(i2s_disable_pads));
+		gpio_configure_pads(dmic_disable_pads, ARRAY_SIZE(dmic_disable_pads));
+		gpio_configure_pads(sndw_disable_pads, ARRAY_SIZE(sndw_disable_pads));
+		return;
+	}
+
    if (fw_config_probe(FW_CONFIG(AUDIO, NONE))) {
    	printk(BIOS_INFO, "Configure GPIOs for no audio.\n");
    	gpio_configure_pads(i2s_disable_pads, ARRAY_SIZE(i2s_disable_pads));