[coreboot-gerrit] New patch to review for coreboot: 9eec74b lenovo/x60: Change to common EDID framework.

[Vladimir Serbinenko (phcoder@gmail.com)]

Vladimir Serbinenko (phcoder at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5238

-gerrit

commit 9eec74b4cb718eb92376e6aa3853f3a7e790a04a
Author: Vladimir Serbinenko <phcoder at gmail.com>
Date:   Sat Feb 15 14:05:29 2014 +0100

    lenovo/x60: Change to common EDID framework.
    
    Currently lenovo/x60 gfx init provides vbe_mode_info_valid in
    incompatible way. Use EDID framework as do other inits.
    
    Change-Id: I887abd5a09064f26f473a2bf9caa2eb33e269c07
    Signed-off-by: Vladimir Serbinenko <phcoder at gmail.com>
---
 src/mainboard/lenovo/x60/i915.c | 32 +++-----------------------------
 1 file changed, 3 insertions(+), 29 deletions(-)

diff --git a/src/mainboard/lenovo/x60/i915.c b/src/mainboard/lenovo/x60/i915.c
index ea01ef1..3d4644f 100644
--- a/src/mainboard/lenovo/x60/i915.c
+++ b/src/mainboard/lenovo/x60/i915.c
@@ -177,8 +177,6 @@ static unsigned long globalmicroseconds(void)
 extern struct iodef iodefs[];
 extern int niodefs;
 
-static int i915_init_done = 0;
-
 /* fill the palette. This runs when the P opcode is hit. */
 static void palette(void)
 {
@@ -189,32 +187,6 @@ static void palette(void)
 		io_i915_WRITE32(color, _LGC_PALETTE_A + (i<<2));
 }
 
-int vbe_mode_info_valid(void);
-int vbe_mode_info_valid(void)
-{
-	return i915_init_done;
-}
-
-void fill_lb_framebuffer(struct lb_framebuffer *framebuffer);
-void fill_lb_framebuffer(struct lb_framebuffer *framebuffer)
-{
-	printk(BIOS_SPEW, "fill_lb_framebuffer: graphics is %p\n",
-	       (void *)graphics);
-	framebuffer->physical_address = graphics;
-	framebuffer->x_resolution = 1024;
-	framebuffer->y_resolution = 768;
-	framebuffer->bytes_per_line = 4096;
-	framebuffer->bits_per_pixel = 32;
-	framebuffer->red_mask_pos = 16;
-	framebuffer->red_mask_size = 8;
-	framebuffer->green_mask_pos = 8;
-	framebuffer->green_mask_size = 8;
-	framebuffer->blue_mask_pos = 0;
-	framebuffer->blue_mask_size = 8;
-	framebuffer->reserved_mask_pos = 24;
-	framebuffer->reserved_mask_size = 8;
-}
-
 static unsigned long times[4096];
 
 static int run(int index)
@@ -394,6 +366,8 @@ int i915lightup(unsigned int pphysbase, unsigned int piobase,
 	memset((void *)graphics, 0x00, FRAME_BUFFER_BYTES);
 
 	printk(BIOS_SPEW, "%ld microseconds\n", globalmicroseconds());
-	i915_init_done = 1;
+
+	set_vbe_mode_info_valid(&edid, graphics);
+
 	return 0;
 }