[coreboot-gerrit] New patch to review for coreboot: e056b95 exynos5420: Fix build warning

Allen Martin (amartin@nvidia.com) gerrit at coreboot.org
Thu Sep 26 20:13:41 CEST 2013

Allen Martin (amartin at nvidia.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3949


commit e056b95c105d768d306c8de412924d20c630f7d9
Author: Allen Martin <amartin at nvidia.com>
Date:   Thu Sep 26 11:13:01 2013 -0700

    exynos5420: Fix build warning
    Fix "set but not used" variable warning with gcc 4.7.3
    Change-Id: Ia27291ecb4f993c4ba6f29b134167dc23a449bf5
    Signed-off-by: Allen Martin <amartin at nvidia.com>
 src/cpu/samsung/exynos5420/cpu.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/cpu/samsung/exynos5420/cpu.c b/src/cpu/samsung/exynos5420/cpu.c
index d204556..744f7ae 100644
--- a/src/cpu/samsung/exynos5420/cpu.c
+++ b/src/cpu/samsung/exynos5420/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_exynos5420_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);
 	       "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