build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37029 )
Change subject: security/tpm: Drop CAR_GLOBAL_MIGRATION support ......................................................................
Patch Set 6:
(8 comments)
https://review.coreboot.org/c/coreboot/+/37029/6/src/drivers/crb/tis.c File src/drivers/crb/tis.c:
https://review.coreboot.org/c/coreboot/+/37029/6/src/drivers/crb/tis.c@23 PS6, Line 23: static unsigned tpm_is_open; Prefer 'unsigned int' to bare use of 'unsigned'
https://review.coreboot.org/c/coreboot/+/37029/6/src/drivers/crb/tis.c@71 PS6, Line 71: tpm_is_open = 0; code indent should use tabs where possible
https://review.coreboot.org/c/coreboot/+/37029/6/src/drivers/i2c/tpm/cr50.c File src/drivers/i2c/tpm/cr50.c:
https://review.coreboot.org/c/coreboot/+/37029/6/src/drivers/i2c/tpm/cr50.c@... PS6, Line 61: static int warning_displayed ; space prohibited before semicolon
https://review.coreboot.org/c/coreboot/+/37029/6/src/drivers/i2c/tpm/tpm.c File src/drivers/i2c/tpm/tpm.c:
https://review.coreboot.org/c/coreboot/+/37029/6/src/drivers/i2c/tpm/tpm.c@8... PS6, Line 83: static struct tpm_inf_dev g_tpm_dev ; space prohibited before semicolon
https://review.coreboot.org/c/coreboot/+/37029/6/src/drivers/pc80/tpm/tis.c File src/drivers/pc80/tpm/tis.c:
https://review.coreboot.org/c/coreboot/+/37029/6/src/drivers/pc80/tpm/tis.c@... PS6, Line 164: static u32 vendor_dev_id ; space prohibited before semicolon
https://review.coreboot.org/c/coreboot/+/37029/6/src/drivers/spi/tpm/tis.c File src/drivers/spi/tpm/tis.c:
https://review.coreboot.org/c/coreboot/+/37029/6/src/drivers/spi/tpm/tis.c@1... PS6, Line 12: static unsigned tpm_is_open; Prefer 'unsigned int' to bare use of 'unsigned'
https://review.coreboot.org/c/coreboot/+/37029/6/src/drivers/spi/tpm/tpm.c File src/drivers/spi/tpm/tpm.c:
https://review.coreboot.org/c/coreboot/+/37029/6/src/drivers/spi/tpm/tpm.c@7... PS6, Line 72: warning_displayed = 1; code indent should use tabs where possible
https://review.coreboot.org/c/coreboot/+/37029/6/src/drivers/spi/tpm/tpm.c@2... PS6, Line 216: static unsigned prev_reg; Prefer 'unsigned int' to bare use of 'unsigned'