[coreboot-gerrit] [M] Change in coreboot[master]: security/tpm: remove public tis_close()

[Julius Werner (Code Review)]

Attention is currently required from: Michał Żygowski, Christian Walter, Krystian Hebel, Sergii Dmytruk.

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68989 )

Change subject: security/tpm: remove public tis_close()
......................................................................

Patch Set 1: Code-Review+2

(1 comment)

File src/drivers/pc80/tpm/tis.c:

https://review.coreboot.org/c/coreboot/+/68989/comment/03b30eb5_b2f22a69 
PS1, Line 664: 	if (tis_close())
I think rather than doing the whole "drop access, re-request access" thing, a simpler way to write this would be to just check tis_has_access() and then skip the tis_request_access() if that's true. That seems to basically be how the I2C version handles this.