summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFelix Held <felix-coreboot@felixheld.de>2020-08-10 20:27:58 +0200
committerFelix Held <felix-coreboot@felixheld.de>2020-08-11 19:08:32 +0000
commit92dd678d6a89fac4afa199202ea1db97e620ba2b (patch)
tree0ed17b74dae31bfe81b45898ded17dabfde5510a
parente0b0697fedf756a811d0326426ab36a59dd83161 (diff)
soc/amd/common/espi_util: make reg parameter unsigned
Th register number passed to the low level read/write functions should never be negative. Change-Id: I5d7e117b3badab900d030be8e69ded026d659f8a Signed-off-by: Felix Held <felix-coreboot@felixheld.de> Reviewed-on: https://review.coreboot.org/c/coreboot/+/44348 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.c12
1 files changed, 6 insertions, 6 deletions
diff --git a/src/soc/amd/common/block/lpc/espi_util.c b/src/soc/amd/common/block/lpc/espi_util.c
index 29028a4b72..ae5edb6592 100644
--- a/src/soc/amd/common/block/lpc/espi_util.c
+++ b/src/soc/amd/common/block/lpc/espi_util.c
@@ -24,32 +24,32 @@ static uintptr_t espi_get_bar(void)
return espi_bar;
}
-static uint32_t espi_read32(int reg)
+static uint32_t espi_read32(unsigned int reg)
{
return read32((void *)(espi_get_bar() + reg));
}
-static void espi_write32(int reg, uint32_t val)
+static void espi_write32(unsigned int reg, uint32_t val)
{
write32((void *)(espi_get_bar() + reg), val);
}
-static uint16_t espi_read16(int reg)
+static uint16_t espi_read16(unsigned int reg)
{
return read16((void *)(espi_get_bar() + reg));
}
-static void espi_write16(int reg, uint16_t val)
+static void espi_write16(unsigned int reg, uint16_t val)
{
write16((void *)(espi_get_bar() + reg), val);
}
-static uint8_t espi_read8(int reg)
+static uint8_t espi_read8(unsigned int reg)
{
return read8((void *)(espi_get_bar() + reg));
}
-static void espi_write8(int reg, uint8_t val)
+static void espi_write8(unsigned int reg, uint8_t val)
{
write8((void *)(espi_get_bar() + reg), val);
}