[coreboot-gerrit] Change in coreboot[master]: soc/amd/common/espi_util: simplify espi_std_io_decode function

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/44350 )

Change subject: soc/amd/common/espi_util: simplify espi_std_io_decode function
......................................................................

soc/amd/common/espi_util: simplify espi_std_io_decode function

We can just return at all places where the ret variable was written
before its value gets returned at the end of the function.

Change-Id: Id87f41c0d9e3397879ac3d15b13179cca1a1263f
Signed-off-by: Felix Held 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/44350
Tested-by: build bot (Jenkins) 
Reviewed-by: Furquan Shaikh 
---
M src/soc/amd/common/block/lpc/espi_util.c
1 file changed, 5 insertions(+), 13 deletions(-)

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

diff --git a/src/soc/amd/common/block/lpc/espi_util.c b/src/soc/amd/common/block/lpc/espi_util.c
index ba633a9..3f26a29 100644
--- a/src/soc/amd/common/block/lpc/espi_util.c
+++ b/src/soc/amd/common/block/lpc/espi_util.c
@@ -104,29 +104,21 @@
  */
 static int espi_std_io_decode(uint16_t base, size_t size)
 {
-	int ret = -1;
-
    if (size != 1)
-		return ret;
+		return -1;

switch (base) {
    case 0x80:
-		ret = ESPI_DECODE_IO_0x80_EN;
-		break;
+		return ESPI_DECODE_IO_0x80_EN;
    case 0x60:
    case 0x64:
-		ret = ESPI_DECODE_IO_0X60_0X64_EN;
-		break;
+		return ESPI_DECODE_IO_0X60_0X64_EN;
    case 0x2e:
    case 0x2f:
-		ret = ESPI_DECODE_IO_0X2E_0X2F_EN;
-		break;
+		return ESPI_DECODE_IO_0X2E_0X2F_EN;
    default:
-		ret = -1;
-		break;
+		return -1;
    }
-
-	return ret;
 }

static size_t espi_get_io_window_size(int idx)