[display] Refactor to avoid implicit framebuffer allocation.

target/qemu-m4/m4display.c

Index: target/qemu-m4/m4display.c
diff --git a/target/qemu-m4/m4display.c b/target/qemu-m4/m4display.c
index c038dfed350ab8f884e5060668a835af9429f9a3..7173fe4a4eb1ac81b6faaea77a399c912c3e0af0 100644
--- a/target/qemu-m4/m4display.c
+++ b/target/qemu-m4/m4display.c
@@ -34,7 +34,6 @@
 
 #include <dev/display.h>
 #include <dev/gpio.h>
-#include <lib/gfx.h>
 #include <platform/gpio.h>
 
 #define LOCAL_TRACE 0
@@ -202,16 +201,28 @@ static void s4lcd_flush(uint starty, uint endy)
     chip_select(true);
 }
 
+status_t display_get_framebuffer(struct display_framebuffer *fb)

[gkalsi, 2016/03/09 21:13:37]

DEBUG_ASSERT(fb);

[cdotstout, 2016/03/09 22:48:00]

Done.

+{
+    LTRACEF("display_get_framebuffer %p\n", fb);
+
+    fb->image.pixels = (void *)framebuffer;
+    fb->image.format = IMAGE_FORMAT_RGB_2220;
+    fb->image.width = M4DISPLAY_WIDTH;
+    fb->image.height = M4DISPLAY_HEIGHT;
+    fb->image.rowbytes = M4DISPLAY_WIDTH;
+    fb->flush = s4lcd_flush;
+    fb->format = DISPLAY_FORMAT_UNKNOWN; //TODO
+
+    return NO_ERROR;
+}
+
 status_t display_get_info(struct display_info *info)
 {
     LTRACEF("display_info %p\n", info);
 
-    info->framebuffer = (void *)framebuffer;
-    info->format = GFX_FORMAT_RGB_2220;
+    info->format = DISPLAY_FORMAT_UNKNOWN; //TODO
     info->width = M4DISPLAY_WIDTH;
     info->height = M4DISPLAY_HEIGHT;
-    info->stride = M4DISPLAY_WIDTH;
-    info->flush = s4lcd_flush;
 
     return NO_ERROR;
 }