[coreboot-gerrit] New patch to review for coreboot: df05ad5 call fill_lb_framebuffer() earlier

[Stefan Reinauer]

Stefan Reinauer ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4155

-gerrit

commit df05ad55d799dc7dab9eb9bbdf91e5dac83b06a4
Author: David Hendricks 
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

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 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
 }