Michał Żygowski has submitted this change. ( https://review.coreboot.org/c/coreboot/+/38785 )
Change subject: mb/pcengines/apu2: Remove unnecessary initialization ......................................................................
mb/pcengines/apu2: Remove unnecessary initialization
The variable is never read before being assigned a value at the end of the function.
Change-Id: I3b42dcd564480005b2c520316933940d87b6e418 Signed-off-by: Paul Menzel pmenzel@molgen.mpg.de Reviewed-on: https://review.coreboot.org/c/coreboot/+/38785 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Michał Żygowski michal.zygowski@3mdeb.com --- M src/mainboard/pcengines/apu2/mainboard.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Michał Żygowski: Looks good to me, approved
diff --git a/src/mainboard/pcengines/apu2/mainboard.c b/src/mainboard/pcengines/apu2/mainboard.c index 9bf58a7..bd2ca39 100644 --- a/src/mainboard/pcengines/apu2/mainboard.c +++ b/src/mainboard/pcengines/apu2/mainboard.c @@ -192,7 +192,7 @@ unsigned long *current) { struct smbios_type17 *t; - int len = 0; + int len;
t = (struct smbios_type17 *)*current; memset(t, 0, sizeof(struct smbios_type17));