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

google/gale: Remove #ifdef of Kconfig bool symbol 

Kconfig symbols of type bool are ALWAYS defined, so this code was
always being included and run, which isn't what the author wanted.

Change to use IS_ENABLED(), and a regular if() instead of an #ifdef.

Change-Id: I72623fa27e47980c602135f4b73f371c7f50139b
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/16837
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
This commit is contained in:
Martin Roth 2016-09-30 14:51:24 -06:00
parent 8bc8be4d0e
commit 6d6c00a502
1 changed files with 26 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

53
src/mainboard/google/gale/verstage.c
View File

@ -24,34 +24,33 @@
static void ipq_setup_tpm(void) static void ipq_setup_tpm(void)
{ {
#ifdef CONFIG_I2C_TPM if (IS_ENABLED(CONFIG_I2C_TPM)) {
gpio_tlmm_config_set(TPM_RESET_GPIO, FUNC_SEL_GPIO, gpio_tlmm_config_set(TPM_RESET_GPIO, FUNC_SEL_GPIO,
GPIO_PULL_UP, GPIO_6MA, 1); GPIO_PULL_UP, GPIO_6MA, 1);
gpio_set(TPM_RESET_GPIO, 0); gpio_set(TPM_RESET_GPIO, 0);
udelay(100); udelay(100);
gpio_set(TPM_RESET_GPIO, 1); gpio_set(TPM_RESET_GPIO, 1);
/* /*
* ----- Per the SLB 9615XQ1.2 spec ----- * ----- Per the SLB 9615XQ1.2 spec -----
* *
* 4.7.1 Reset Timing * 4.7.1 Reset Timing
* *
* The TPM_ACCESS_x.tpmEstablishment bit has the correct value * The TPM_ACCESS_x.tpmEstablishment bit has the correct value
* and the TPM_ACCESS_x.tpmRegValidSts bit is typically set * and the TPM_ACCESS_x.tpmRegValidSts bit is typically set
* within 8ms after RESET# is deasserted. * within 8ms after RESET# is deasserted.
* *
* The TPM is ready to receive a command after less than 30 ms. * The TPM is ready to receive a command after less than 30 ms.
* *
* -------------------------------------- * --------------------------------------
* *
* I'm assuming this means "wait for 30ms" * I'm assuming this means "wait for 30ms"
* *
* If we don't wait here, subsequent QUP I2C accesses * If we don't wait here, subsequent QUP I2C accesses
* to the TPM either fail or timeout. * to the TPM either fail or timeout.
*/ */
mdelay(30); mdelay(30);
}
#endif /* CONFIG_I2C_TPM */
} }
void verstage_mainboard_init(void) void verstage_mainboard_init(void)