commit ce768f3baff20815f227fffbf1bb406f3a7dd1a7
author Nico Huber <nico.h@gmx.de> Sat May 20 20:47:18 2017 +0200
committer Martin Roth <martinroth@google.com> Sun Jun 04 18:47:39 2017 +0200
tree ccbebc6464ee046cd956e74372efb7e587d58d82
parent 6d8266b91d7c8338350f7f8b81c21d10c97aceb6

lib/coreboot_tables: Remove weak fill_lb_framebuffer()

Remove the weak function stub fill_lb_framebuffer() and guard with the
new `CONFIG_LINEAR_FRAMEBUFFER` instead.

Change-Id: Ia9e477c2d501b249a632968b5636ac803323895a
Signed-off-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.coreboot.org/19807
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-by: Philippe Mathieu-Daudé <philippe.mathieu.daude@gmail.com>

src/lib/coreboot_table.c

diff --git a/src/lib/coreboot_table.c b/src/lib/coreboot_table.c
index f3cdb0b..aa6c7a5 100644
--- a/src/lib/coreboot_table.c
+++ b/src/lib/coreboot_table.c
@@ -135,17 +135,12 @@
 	console->type = consoletype;
 }
 
-int __attribute__((weak)) fill_lb_framebuffer(struct lb_framebuffer *fb)
-{
-	return -1;
-}
-
 static void lb_framebuffer(struct lb_header *header)
 {
 	struct lb_framebuffer *framebuffer;
 	struct lb_framebuffer fb;
 
-	if (fill_lb_framebuffer(&fb))
+	if (!IS_ENABLED(CONFIG_LINEAR_FRAMEBUFFER) || fill_lb_framebuffer(&fb))
 		return;
 
 	framebuffer = (struct lb_framebuffer *)lb_new_record(header);