From 78021190058462bf2ed7da33d3327bd9ebc5ab82 Mon Sep 17 00:00:00 2001 From: Paul Menzel Date: Sat, 7 May 2016 08:18:18 +0200 Subject: [PATCH] siemens/mc_tcu3: Remove unneeded variable assignment Assigning the value `1` to `status` in the default branch of the switch statement is not needed, as the stored value is overwritten before it can be used. Change-Id: I532b0e217ff4ed315cd30b08d339c755c6df7539 Found-by: Coverity, CID 1355008: Code maintainability issues (UNUSED_VALUE)) Signed-off-by: Paul Menzel Reviewed-on: https://review.coreboot.org/14699 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer --- src/mainboard/siemens/mc_tcu3/lcd_panel.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/mainboard/siemens/mc_tcu3/lcd_panel.c b/src/mainboard/siemens/mc_tcu3/lcd_panel.c index 214773aa26..50fedf4409 100644 --- a/src/mainboard/siemens/mc_tcu3/lcd_panel.c +++ b/src/mainboard/siemens/mc_tcu3/lcd_panel.c @@ -66,7 +66,6 @@ int setup_lcd_panel(void) break; default: printk(BIOS_ERR, "LCD: No supported panel found.\n"); - status = 1; break; } /* Now that we have the panel type, setup the DP2LVDS converter */