[coreboot-gerrit] Patch set updated for coreboot: 9aa65be cpu/samsung/exynos5250/cpu.c: Remove unused variable `ret`

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Tue May 6 14:17:14 CEST 2014


Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4594

-gerrit

commit 9aa65be0732b407b0b769fda8ca48a4681e6ce5e
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Wed Jan 1 14:37:33 2014 +0100

    cpu/samsung/exynos5250/cpu.c: Remove unused variable `ret`
    
    	    CC         cpu/samsung/exynos5250/cpu.ramstage.o
    	src/cpu/samsung/exynos5250/cpu.c: In function 'exynos_displayport_init':
    	src/cpu/samsung/exynos5250/cpu.c:88:6: error: variable 'ret' set but not used [-Werror=unused-but-set-variable]
    	cc1: all warnings being treated as errors
    
    Change-Id: I72a5a802aef0a75b0277eaac6ca6c24793812d98
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 src/cpu/samsung/exynos5250/cpu.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/cpu/samsung/exynos5250/cpu.c b/src/cpu/samsung/exynos5250/cpu.c
index 2354be1..1b4e130 100644
--- a/src/cpu/samsung/exynos5250/cpu.c
+++ b/src/cpu/samsung/exynos5250/cpu.c
@@ -85,7 +85,6 @@ static void set_cpu_id(void)
 static void exynos_displayport_init(device_t dev, u32 lcdbase,
 		unsigned long fb_size)
 {
-	int ret;
 	struct cpu_samsung_exynos5250_config *conf = dev->chip_info;
 	/* put these on the stack. If, at some point, we want to move
 	 * this code to a pre-ram stage, it will be much easier.
@@ -131,7 +130,7 @@ static void exynos_displayport_init(device_t dev, u32 lcdbase,
 
 	printk(BIOS_DEBUG, "Initializing Exynos LCD.\n");
 
-	ret = lcd_ctrl_init(fb_size, &panel, (void *)lcdbase);
+	lcd_ctrl_init(fb_size, &panel, (void *)lcdbase);
 }
 
 static void cpu_enable(device_t dev)



More information about the coreboot-gerrit mailing list