From 283b438f5ceaa88eb4860a5e1438b03c875bdf03 Mon Sep 17 00:00:00 2001
From: Elyes HAOUAS <ehaouas@noos.fr>
Date: Sun, 9 Jun 2019 13:31:36 +0200
Subject: [PATCH] soc/intel/common/block/pcr: Remove unneded 'else'

'else' is not needed after a 'break' or 'return'.

Change-Id: Ib3371ef6edb85a47ed734dd2ff9ce94008aa4e65
Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/33336
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-by: Vlado Cibic
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
 src/soc/intel/common/block/pcr/pcr.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/soc/intel/common/block/pcr/pcr.c b/src/soc/intel/common/block/pcr/pcr.c
index 4a35a03ff1..f3971c238f 100644
--- a/src/soc/intel/common/block/pcr/pcr.c
+++ b/src/soc/intel/common/block/pcr/pcr.c
@@ -394,10 +394,9 @@ int pcr_execute_sideband_msg(struct pcr_sbi_msg *msg, uint32_t *data,
 			break;
 		}
 		return 0;
-	} else {
-		printk(BIOS_ERR, "SBI Failure: Transaction Status = %x\n",
-				*response);
-		return -1;
 	}
+	printk(BIOS_ERR, "SBI Failure: Transaction Status = %x\n",
+			*response);
+	return -1;
 }
 #endif