From bdb5c8feaed4cf914b474deeb30686e327ee9228 Mon Sep 17 00:00:00 2001 From: Aaron Durbin Date: Tue, 16 May 2017 21:39:50 -0500 Subject: coreboot_tables: specify clear interface for lb_framebuffer() For some reason the "interface" for adding framebuffer information is sitting in src/include/vbe.h while also guarding the call to fill_lb_framebuffer() with vbe_mode_info_valid() along with some macro if CONFIG_* for good measure. Move the fill_lb_framebuffer() declaration to coreboot_tables.h and provide a comment about how it should be used. Also, now that there's no need for the notion of a global vbe_mode_info_valid() remove it from the conditional call path of fill_lb_framebuffer(). Change-Id: Ib3ade6314624091ae70424664527a02b279d0c9b Signed-off-by: Aaron Durbin Signed-off-by: Nico Huber Reviewed-on: https://review.coreboot.org/19729 Tested-by: build bot (Jenkins) Reviewed-by: Nico Huber --- src/include/boot/coreboot_tables.h | 5 +++-- src/include/vbe.h | 1 - 2 files changed, 3 insertions(+), 3 deletions(-) (limited to 'src/include') diff --git a/src/include/boot/coreboot_tables.h b/src/include/boot/coreboot_tables.h index 5647629d76..5bebd4a623 100644 --- a/src/include/boot/coreboot_tables.h +++ b/src/include/boot/coreboot_tables.h @@ -22,8 +22,9 @@ void lb_add_console(uint16_t consoletype, void *data); /* Define this in mainboard.c to add board-specific table entries. */ void lb_board(struct lb_header *header); -/* Define this in soc or fsp driver to add specific table entries. */ -void lb_framebuffer(struct lb_header *header); +/* Define this function to fill in the frame buffer returning 0 on success and + < 0 on error. */ +int fill_lb_framebuffer(struct lb_framebuffer *framebuffer); /* Allow arch to add records. */ void lb_arch_add_records(struct lb_header *header); diff --git a/src/include/vbe.h b/src/include/vbe.h index 54dc560699..70c412490b 100644 --- a/src/include/vbe.h +++ b/src/include/vbe.h @@ -112,7 +112,6 @@ typedef struct { #define VESA_SET_MODE 0x4f02 int vbe_mode_info_valid(void); -void fill_lb_framebuffer(struct lb_framebuffer *framebuffer); void vbe_set_graphics(void); void vbe_textmode_console(void); -- cgit v1.2.3