[coreboot-gerrit] Patch set updated for coreboot: 0b517fa call fill_lb_framebuffer() earlier

[Stefan Reinauer (stefan.reinauer@coreboot.org)]

Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4155

-gerrit

commit 0b517fa79a98490e7fc06fb12337441345a22c0e
Author: David Hendricks <dhendrix at 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 at chromium.org>
    
    Change-Id: I5ac783fa3a776eee504d39889284041d1dc2c92a
    Reviewed-on: https://gerrit.chromium.org/gerrit/50012
    Reviewed-by: Gabe Black <gabeblack at chromium.org>
    Commit-Queue: David Hendricks <dhendrix at chromium.org>
    Tested-by: David Hendricks <dhendrix at 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
 }