summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFelix Held <felix-coreboot@felixheld.de>2020-08-10 20:33:25 +0200
committerFelix Held <felix-coreboot@felixheld.de>2020-08-11 19:09:03 +0000
commit4bf419fbf750eb4c3f3f7f05486ea4bf503274ed (patch)
treea704c1d5d52bde73eb05f273ff9229cba3a349e5
parentf08fbf882a1517676c4b91f4c755b97a9125c6b2 (diff)
downloadcoreboot-4bf419fbf750eb4c3f3f7f05486ea4bf503274ed.tar.xz
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 <felix-coreboot@felixheld.de> Reviewed-on: https://review.coreboot.org/c/coreboot/+/44350 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Furquan Shaikh <furquan@google.com>
-rw-r--r--src/soc/amd/common/block/lpc/espi_util.c18
1 files changed, 5 insertions, 13 deletions
diff --git a/src/soc/amd/common/block/lpc/espi_util.c b/src/soc/amd/common/block/lpc/espi_util.c
index ba633a9a6c..3f26a29eb8 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_get_unused_io_window(void)
*/
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)