diff options
| author | Angel Pons <th3fanbus@gmail.com> | 2021-06-28 15:40:58 +0200 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2021-07-01 07:36:49 +0000 |
| commit | 631cd29efbcd030d8c3fe3ce3c9d98562b306007 (patch) | |
| tree | 91f307036350ad149fad52414372602c06ce2fae /src | |
| parent | 35b99c64e62fd84866dd269a3641b66646f9557f (diff) | |
mb/pcengines/apuX: Refactor to avoid dead assignment
The initial value of `len` is never used. Declare and initialise all
local variables in a single statement to avoid this problem.
Change-Id: Ieb96758f4cd19d9d3f8fdc543e8ca17db06a2123
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/55905
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Michał Żygowski <michal.zygowski@3mdeb.com>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/mainboard/pcengines/apu1/mainboard.c | 9 | ||||
| -rw-r--r-- | src/mainboard/pcengines/apu2/mainboard.c | 9 |
2 files changed, 6 insertions, 12 deletions
diff --git a/src/mainboard/pcengines/apu1/mainboard.c b/src/mainboard/pcengines/apu1/mainboard.c index 05c496dc57..187f8cff3c 100644 --- a/src/mainboard/pcengines/apu1/mainboard.c +++ b/src/mainboard/pcengines/apu1/mainboard.c @@ -168,14 +168,11 @@ static void config_addon_uart(void) #if CONFIG(GENERATE_SMBIOS_TABLES) static int mainboard_smbios_type16(DMI_INFO *agesa_dmi, int *handle, unsigned long *current) { - struct smbios_type16 *t; - u32 max_capacity; - int len; + const u32 max_capacity = get_spd_offset() ? 4 : 2; /* 4GB or 2GB variant */ - t = (struct smbios_type16 *)*current; - len = sizeof(*t); + struct smbios_type16 *t = (struct smbios_type16 *)*current; + int len = sizeof(*t); memset(t, 0, len); - max_capacity = get_spd_offset() ? 4 : 2; /* 4GB or 2GB variant */ t->type = SMBIOS_PHYS_MEMORY_ARRAY; t->handle = *handle; diff --git a/src/mainboard/pcengines/apu2/mainboard.c b/src/mainboard/pcengines/apu2/mainboard.c index bcf2536237..2c3b861d38 100644 --- a/src/mainboard/pcengines/apu2/mainboard.c +++ b/src/mainboard/pcengines/apu2/mainboard.c @@ -150,14 +150,11 @@ static void config_gpio_mux(void) static int mainboard_smbios_type16(DMI_INFO *agesa_dmi, int *handle, unsigned long *current) { - struct smbios_type16 *t; - u32 max_capacity; - int len = 0; + const u32 max_capacity = get_spd_offset() ? 4 : 2; /* 4GB or 2GB variant */ - t = (struct smbios_type16 *)*current; - len = sizeof(*t); + struct smbios_type16 *t = (struct smbios_type16 *)*current; + int len = sizeof(*t); memset(t, 0, len); - max_capacity = get_spd_offset() ? 4 : 2; /* 4GB or 2GB variant */ t->type = SMBIOS_PHYS_MEMORY_ARRAY; t->handle = *handle; |