[coreboot-gerrit] Change in coreboot[master]: drivers/spi/tpm: Enable long cr50 ready pulses for Tiger Lake systems

[Julius Werner (Code Review)]

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

Change subject: drivers/spi/tpm: Enable long cr50 ready pulses for Tiger Lake systems
......................................................................

Patch Set 22:

(1 comment)

https://review.coreboot.org/c/coreboot/+/43741/22/src/drivers/spi/tpm/tpm.c 
File src/drivers/spi/tpm/tpm.c:

https://review.coreboot.org/c/coreboot/+/43741/22/src/drivers/spi/tpm/tpm.c@... 
PS22, Line 631: 			if (version_str[++chunk_count * chunk_size - 1])
Careful, I think you flipped a while condition to a break condition here and forgot to negate? This should have an == '\0' or something at the end.