[coreboot-gerrit] New patch to review for coreboot: b2ca25b cpu/samsung/exynos5250/cpu.c: Remove unused variable `ret`

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Wed Jan 1 14:49:38 CET 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 b2ca25b3780e9b312aa6cc1564d5aec7a8ca1969
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 61f937b..c663d38 100644
--- a/src/cpu/samsung/exynos5250/cpu.c
+++ b/src/cpu/samsung/exynos5250/cpu.c
@@ -64,7 +64,6 @@ static void set_cpu_id(void)
  */
 static void exynos_displayport_init(device_t dev)
 {
-	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.
@@ -117,7 +116,7 @@ static void exynos_displayport_init(device_t dev)
 	mmio_resource(dev, 1, lcdbase/KiB, (fb_size + KiB - 1)/KiB);
 	printk(BIOS_DEBUG,
 	       "Initializing Exynos VGA, base %p\n", (void *)lcdbase);
-	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