diff options
| author | Raul E Rangel <rrangel@chromium.org> | 2022-02-11 09:48:58 -0700 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2022-02-12 18:44:43 +0000 |
| commit | 507064b835df517723bdc2e06c47322c3d626bf4 (patch) | |
| tree | 319e85164a34e79e11be633ac4e713228bdb2c0a /src | |
| parent | d8cf72f7e68dcbfc69f2d2acaa82260cdbed46ab (diff) | |
soc/amd/cezanne/psp_verstage/uart: Fix off by 1 error
FCH_UART_ID_MAX == 2, and there are 2 UARTS, so we don't need the -1.
BUG=b:215599230
TEST=Build guybrush
Found-by: Coverity CID 1469611
Signed-off-by: Raul E Rangel <rrangel@chromium.org>
Change-Id: I5f0171ed2d3da7f86ba3cfd0457f60d2d5722625
Reviewed-on: https://review.coreboot.org/c/coreboot/+/61869
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/soc/amd/cezanne/psp_verstage/uart.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/soc/amd/cezanne/psp_verstage/uart.c b/src/soc/amd/cezanne/psp_verstage/uart.c index 0f0baba07e..5a45cb0545 100644 --- a/src/soc/amd/cezanne/psp_verstage/uart.c +++ b/src/soc/amd/cezanne/psp_verstage/uart.c @@ -4,7 +4,7 @@ #include <amdblocks/uart.h> #include <types.h> -static void *uart_bars[FCH_UART_ID_MAX - 1]; +static void *uart_bars[FCH_UART_ID_MAX]; uintptr_t get_uart_base(unsigned int idx) { |