picoLCDGraphic: update display regularly using a timer (*much* faster than redrawing...

git-svn-id: https://ssl.bulix.org/svn/lcd4linux/trunk@1094 3ae390bd-cb1e-0410-b409-cd5a39f66f1f

diff --git a/drv_G15.c b/drv_G15.c
index 476bfc5a5e02f7cc6d67d5a1576d9a1d9c69dbd2..8a95b99df3de82c019a9a9b174130089de0150f4 100644 (file)
--- a/drv_G15.c
+++ b/drv_G15.c
@@ -330,8 +330,8 @@ static int drv_G15_open()
            debug("%s: open %s/%s/%s", Name, bus->dirname, dev->bus->dirname, dev->filename);
            if ((g15_lcd = usb_open(dev))) {
                if (dev->descriptor.idVendor == G15_VENDOR) {
-                    debug("%s: Found USB vendor ID 0x%x (Logitech), checking productID 0x%x...",
-                          Name, G15_VENDOR, dev->descriptor.idProduct);
+                   debug("%s: Found USB vendor ID 0x%x (Logitech), checking productID 0x%x...",
+                         Name, G15_VENDOR, dev->descriptor.idProduct);
                    switch (dev->descriptor.idProduct) {
                    case G15_DEVICE:
                    case G15_DEVICE2:
@@ -351,8 +351,8 @@ static int drv_G15_open()
                            break;
                        }
                    default:
-                        debug("%s: Don't found USB product IDs 0x%x|0x%x/0x%x for G-15/M1730 or 0x%x for Z10",
-                              Name, G15_DEVICE, G15_DEVICE2, M1730_DEVICE, Z10_DEVICE);
+                       debug("%s: Don't found USB product IDs 0x%x|0x%x/0x%x for G-15/M1730 or 0x%x for Z10",
+                             Name, G15_DEVICE, G15_DEVICE2, M1730_DEVICE, Z10_DEVICE);
                        usb_close(g15_lcd);
                    }
 

diff --git a/drv_picoLCDGraphic.c b/drv_picoLCDGraphic.c
index 5e7e159aae50ab844e56eea7b91504045380bed7..afbc61f10d4d896b215eed33aba63f9b2472bf7c 100644 (file)
--- a/drv_picoLCDGraphic.c
+++ b/drv_picoLCDGraphic.c
@@ -53,6 +53,7 @@
 #include "qprintf.h"
 #include "udelay.h"
 #include "plugin.h"
+#include "timer.h"
 #include "widget.h"
 #include "widget_text.h"
 #include "widget_icon.h"
@@ -85,6 +86,11 @@
 #define DEBUG(x)
 #endif
 
+/* "dirty" marks the display to be redrawn from frame buffer */
+static int dirty = 1;
+
+/* timer for display redraw (set to zero for "direct updates") */
+static int update = 0;
 
 static char Name[] = "picoLCDGraphic";
 static unsigned char *pLG_framebuffer;
@@ -200,8 +206,14 @@ static void drv_pLG_update_img()
     unsigned char cs, line;
     unsigned char pixel;
 
-    info("In %s\n", __FUNCTION__);
+    /* do not redraw display if frame buffer has not changed, unless
+       "direct updates" have been requested (update is zero) */
+    if ((!dirty) && (update > 0)) {
+       debug("Skipping %s\n", __FUNCTION__);
+       return;
+    }
 
+    info("In %s\n", __FUNCTION__);
 
     for (cs = 0; cs < 4; cs++) {
        unsigned char chipsel = (cs << 2);      //chipselect
@@ -259,6 +271,8 @@ static void drv_pLG_update_img()
        }
     }
 
+    /* mark display as up-to-date */
+    dirty = 0;
     //drv_pLG_clear();
 }
 
@@ -286,7 +300,13 @@ static void drv_pLG_blit(const int row, const int col, const int height, const i
        fprintf(stderr, "\n");
        }
      */
-    drv_pLG_update_img();
+
+    /* display needs to be redrawn from frame buffer */
+    dirty = 1;
+
+    /* if "direct updates" have been requested, redraw now */
+    if (update <= 0)
+       drv_pLG_update_img();
 }
 
 
@@ -442,6 +462,9 @@ static int drv_pLG_start(const char *section, const int quiet)
     int value;
     char *s;
 
+    /* set display redraw interval (set to zero for "direct updates") */
+    cfg_number(section, "update", 200, 0, -1, &update);
+
     s = cfg_get(section, "Size", NULL);
     if (s == NULL || *s == '\0') {
        error("%s: no '%s.Size' entry from %s", Name, section, cfg_source());
@@ -503,6 +526,11 @@ static int drv_pLG_start(const char *section, const int quiet)
        }
     }
 
+    /* setup a timer that regularly redraws the display from the frame
+       buffer (unless "direct updates" have been requested */
+    if (update > 0)
+       timer_add(drv_pLG_update_img, NULL, update, 0);
+
     return 0;
 }