GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

mb/google/zork: Remove 50ms WIFI delay 

As a part of trying to get our boot time as low as possible, any delays
in the code should try to be refactored out.  This removes the 50ms
delay in the WIFI sequence by enabling power and putting the wifi module
into reset in bootblock, then bringing it out of reset in ramstage.
This is significantly longer than the 50ms requirement.  The reset GPIO
was already being set high in ramstage, so that code didn't need to be
added.

BUG=b:171513520
TEST=Boot on boards with different module types, WIFI works on both.
BRANCH=Zork

Signed-off-by: Martin Roth <martinroth@chromium.org>
Change-Id: I211d3da338ad368d1f011f03cf7d05121c057075
Reviewed-on: https://review.coreboot.org/c/coreboot/+/47719
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
This commit is contained in:
Martin Roth 2020-11-18 09:20:01 -07:00 committed by Felix Held
parent 5b47d77047
commit cdd7d18120
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/mainboard/google/zork/variants/baseboard/gpio_baseboard_trembyle.c
View File

@ -199,8 +199,8 @@ static void wifi_power_reset_configure_active_low_power(void)
* Configure WiFi GPIOs such that: * Configure WiFi GPIOs such that:
* - WIFI_AUX_RESET_L is configured first to assert PERST# to WiFi device. * - WIFI_AUX_RESET_L is configured first to assert PERST# to WiFi device.
* - Enable power to WiFi using EN_PWR_WIFI_L. * - Enable power to WiFi using EN_PWR_WIFI_L.
* - Wait for 50ms after power to WiFi is enabled. * - Wait for >50ms after power to WiFi is enabled. (Time between bootblock & ramstage)
* - Deassert WIFI_AUX_RESET_L. * - WIFI_AUX_RESET_L gets deasserted later in mainboard_configure_gpios in ramstage
*/ */
static const struct soc_amd_gpio v3_wifi_table[] = { static const struct soc_amd_gpio v3_wifi_table[] = {
/* WIFI_AUX_RESET_L */ /* WIFI_AUX_RESET_L */
@ -210,8 +210,6 @@ static void wifi_power_reset_configure_active_low_power(void)
}; };
program_gpios(v3_wifi_table, ARRAY_SIZE(v3_wifi_table)); program_gpios(v3_wifi_table, ARRAY_SIZE(v3_wifi_table));
mdelay(50);
gpio_set(GPIO_86, 1);
} }
static void wifi_power_reset_configure_active_high_power(void) static void wifi_power_reset_configure_active_high_power(void)