8 comments:
File src/drivers/spi/tpm/Kconfig:
Patch Set #2, Line 16: MAINBOARD_HAS_SPI_TPM_CR50
while at it, can we change MAINBOARD_HAS_SPI_TPM_CR50 to MAINBOARD_HAS_SPI_TPM? CR50 is not alone an […]
Refactoring that part should not be done in this commit. While at it the MAINBOARD_HAS_I2C_TPM_CR50 needs to be updated, too.
Patch Set #2, Line 21: Board has SPI TPM support
nit: should say something different from the one below
Done
File src/drivers/spi/tpm/tpm.c:
Patch Set #1, Line 85: static int tpm_sync(void)
This whole out-of-band interrupt syncing thing is Cr50-specific, you'll want to bypass it for other […]
Done
Patch Set #1, Line 190: (header_resp.body[3] & 1)
that's what the spec says. The last byte of the address has the ready bit set.
Done
Patch Set #1, Line 460: if (ENV_VERSTAGE || ENV_BOOTBLOCK || !CONFIG(VBOOT))
Hmm... this is making an existing hacky situation even hackier. […]
Added a FIXME
File src/drivers/spi/tpm/tpm.c:
Patch Set #2, Line 198: SPI transfer error\n
nit: Mention something about this being in the TPM driver?
Done
Patch Set #2, Line 203: !CONFIG(TPM_CR50) && (
CONFIG() test is unnecessary here.
Done
File src/security/tpm/Kconfig:
Patch Set #1, Line 29: || MAINBOARD_HAS_SPI_TPM_CR50 || MAINBOARD_HAS_CRB_TPM \
I think the intention is still that Kconfig files adhere to the usual 80 (or now 96) character limit […]
Done
To view, visit change 39693. To unsubscribe, or for help writing mail filters, visit settings.