Stefan Reinauer (stefan.reinauer@coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4155
-gerrit
commit 35319b9c6a419e84ed4502e2696ddc1a38b6ba0f Author: David Hendricks dhendrix@chromium.org Date: Thu May 2 18:23:24 2013 -0700
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)
Signed-off-by: David Hendricks dhendrix@chromium.org
Change-Id: I5ac783fa3a776eee504d39889284041d1dc2c92a Reviewed-on: https://gerrit.chromium.org/gerrit/50012 Reviewed-by: Gabe Black gabeblack@chromium.org Commit-Queue: David Hendricks dhendrix@chromium.org Tested-by: David Hendricks dhendrix@chromium.org --- 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 e5729ba..a0a806d 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 }