diff options
| author | Felix Held <felix.held@amd.corp-partner.google.com> | 2021-09-22 16:36:12 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2021-09-23 14:42:03 +0000 |
| commit | 7011fa1135009897a8fee5d96ade0f9fa9c960cc (patch) | |
| tree | 853e3f99abda14caa70ba0aa9b81ae8d2da1ed54 /src/mainboard/google/guybrush | |
| parent | 05df6ec8449355f5205807b84e7a6ba87252cf27 (diff) | |
soc/amd: rename program_gpios to gpio_configure_pads
Use the same function name as in soc/intel for this functionality. This
also brings the function name more in line with the extended version of
this function gpio_configure_pads_with_override which additionally
supports passing a GPIO override configuration.
This might cause some pain for out-of-tree boards, but at some point
this should be made more consistent, so I don't see a too strong reason
not to do this.
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I88852e040f79861ce7d190bf2203f9e0ce156690
Reviewed-on: https://review.coreboot.org/c/coreboot/+/57837
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Diffstat (limited to 'src/mainboard/google/guybrush')
| -rw-r--r-- | src/mainboard/google/guybrush/smihandler.c | 2 | ||||
| -rw-r--r-- | src/mainboard/google/guybrush/variants/baseboard/gpio.c | 4 | ||||
| -rw-r--r-- | src/mainboard/google/guybrush/verstage.c | 2 |
3 files changed, 4 insertions, 4 deletions
diff --git a/src/mainboard/google/guybrush/smihandler.c b/src/mainboard/google/guybrush/smihandler.c index 04280db316..f67ff31e49 100644 --- a/src/mainboard/google/guybrush/smihandler.c +++ b/src/mainboard/google/guybrush/smihandler.c @@ -22,7 +22,7 @@ void mainboard_smi_sleep(u8 slp_typ) chromeec_smi_sleep(slp_typ, MAINBOARD_EC_S3_WAKE_EVENTS, MAINBOARD_EC_S5_WAKE_EVENTS); gpios = variant_sleep_gpio_table(&num_gpios); - program_gpios(gpios, num_gpios); + gpio_configure_pads(gpios, num_gpios); } int mainboard_smi_apmc(u8 apmc) diff --git a/src/mainboard/google/guybrush/variants/baseboard/gpio.c b/src/mainboard/google/guybrush/variants/baseboard/gpio.c index 50595cd769..98780404fb 100644 --- a/src/mainboard/google/guybrush/variants/baseboard/gpio.c +++ b/src/mainboard/google/guybrush/variants/baseboard/gpio.c @@ -362,7 +362,7 @@ __weak void variant_fpmcu_reset(void) /* EN_PWR_FP */ PAD_GPO(GPIO_32, LOW), }; - program_gpios(fpmcu_bootblock_table, ARRAY_SIZE(fpmcu_bootblock_table)); + gpio_configure_pads(fpmcu_bootblock_table, ARRAY_SIZE(fpmcu_bootblock_table)); } __weak void variant_finalize_gpios(void) @@ -380,6 +380,6 @@ __weak void variant_finalize_gpios(void) /* Deassert the FPMCU reset to enable the FPMCU */ gpio_set(GPIO_11, 1); /* FPMCU_RST_L */ } else { - program_gpios(disable_fpmcu_table, ARRAY_SIZE(disable_fpmcu_table)); + gpio_configure_pads(disable_fpmcu_table, ARRAY_SIZE(disable_fpmcu_table)); } } diff --git a/src/mainboard/google/guybrush/verstage.c b/src/mainboard/google/guybrush/verstage.c index 904ae2f64c..e15cc33195 100644 --- a/src/mainboard/google/guybrush/verstage.c +++ b/src/mainboard/google/guybrush/verstage.c @@ -13,7 +13,7 @@ static void setup_gpio(void) if (CONFIG(VBOOT_STARTS_BEFORE_BOOTBLOCK)) { gpios = variant_early_gpio_table(&num_gpios); - program_gpios(gpios, num_gpios); + gpio_configure_pads(gpios, num_gpios); } } |