From f3df9cb0ad418088ff6a7eef36fcf1307133944a Mon Sep 17 00:00:00 2001 From: David Hendricks Date: Thu, 2 May 2013 18:23:24 -0700 Subject: [PATCH] call fill_lb_framebuffer() earlier fill_lb_framebuffer() now sets the framebuffer pointer according to the EDID information, so it must be called before setting the tag and size. (credit to rminnich for this, I'm just uploading it) BRANCH=none BUG=none TEST=booted on Snow using depthcharge in dev mode Signed-off-by: David Hendricks Change-Id: I5ac783fa3a776eee504d39889284041d1dc2c92a Reviewed-on: https://gerrit.chromium.org/gerrit/50012 Reviewed-by: Gabe Black Commit-Queue: David Hendricks Tested-by: David Hendricks --- src/lib/coreboot_table.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/lib/coreboot_table.c b/src/lib/coreboot_table.c index d25b59d43d..711854ab5b 100644 --- a/src/lib/coreboot_table.c +++ b/src/lib/coreboot_table.c @@ -177,9 +177,9 @@ static void lb_framebuffer(struct lb_header *header) return; struct lb_framebuffer *framebuffer; framebuffer = (struct lb_framebuffer *)lb_new_record(header); + fill_lb_framebuffer(framebuffer); framebuffer->tag = LB_TAG_FRAMEBUFFER; framebuffer->size = sizeof(*framebuffer); - fill_lb_framebuffer(framebuffer); #endif }