[coreboot-gerrit] Change in coreboot[master]: tpm2/tlcl_send_startup: correct TPM2_Startup error logic

[Philipp Deppenwiese (Code Review)]

Philipp Deppenwiese has submitted this change and it was merged. ( https://review.coreboot.org/28749 )

Change subject: tpm2/tlcl_send_startup: correct TPM2_Startup error logic
......................................................................

tpm2/tlcl_send_startup: correct TPM2_Startup error logic

BUG=b:114018226
TEST=compile coreboot

Change-Id: Iffad582d5c19252dde1c9dc97fc7fd145944a3b3
Signed-off-by: Joel Kitching 
Reviewed-on: https://review.coreboot.org/28749
Tested-by: build bot (Jenkins) 
Reviewed-by: Aaron Durbin 
Reviewed-by: Furquan Shaikh 
---
M src/security/tpm/tss/tcg-2.0/tss.c
1 file changed, 6 insertions(+), 7 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Aaron Durbin: Looks good to me, approved
  Furquan Shaikh: Looks good to me, approved

diff --git a/src/security/tpm/tss/tcg-2.0/tss.c b/src/security/tpm/tss/tcg-2.0/tss.c
index 7db746f..e6ec57c 100644
--- a/src/security/tpm/tss/tcg-2.0/tss.c
+++ b/src/security/tpm/tss/tcg-2.0/tss.c
@@ -61,14 +61,13 @@
    startup.startup_type = type;
    response = tpm_process_command(TPM2_Startup, &startup);

-	if (response && response->hdr.tpm_code &&
-	    (response->hdr.tpm_code != TPM_RC_INITIALIZE)) {
-		printk(BIOS_INFO, "%s: Startup return code is %x\n",
-		       __func__, response->hdr.tpm_code);
-		return TPM_E_IOERROR;
+	if (response && (response->hdr.tpm_code == 0 ||
+			 response->hdr.tpm_code == TPM_RC_INITIALIZE)) {
+		return TPM_SUCCESS;
    }
-	return TPM_SUCCESS;
-
+	printk(BIOS_INFO, "%s: Startup return code is %x\n",
+	       __func__, response->hdr.tpm_code);
+	return TPM_E_IOERROR;
 }

uint32_t tlcl_resume(void)