diff options
| author | Paul Menzel <paulepanter@users.sourceforge.net> | 2016-05-07 08:18:18 +0200 |
|---|---|---|
| committer | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2016-05-09 05:28:12 +0200 |
| commit | 78021190058462bf2ed7da33d3327bd9ebc5ab82 (patch) | |
| tree | 3fcd9f37cfbfef1c5d96c2cd7e819886aaf3136b /src | |
| parent | 1e162bf8bd74e8dcd0b164b627e1d536c7dbd5f5 (diff) | |
| download | coreboot-78021190058462bf2ed7da33d3327bd9ebc5ab82.tar.xz | |
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 <paulepanter@users.sourceforge.net>
Reviewed-on: https://review.coreboot.org/14699
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/mainboard/siemens/mc_tcu3/lcd_panel.c | 1 |
1 files changed, 0 insertions, 1 deletions
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 */ |