[lcd4linux @ 2005-08-21 08:18:56 by reinelt]

CrystalFontz ACK processing

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

diff --git a/drv_Crystalfontz.c b/drv_Crystalfontz.c
index 6ac7fe0793c1f862f115c36ce0be52c11cc06fde..105f5f2c9542b0ed5e28aa163638eaa8f1bb19a4 100644 (file)
--- a/drv_Crystalfontz.c
+++ b/drv_Crystalfontz.c
@@ -1,4 +1,4 @@
-/* $Id: drv_Crystalfontz.c,v 1.34 2005/05/08 04:32:44 reinelt Exp $
+/* $Id: drv_Crystalfontz.c,v 1.35 2005/08/21 08:18:56 reinelt Exp $
  *
  * new style driver for Crystalfontz display modules
  *
@@ -23,6 +23,9 @@
  *
  *
  * $Log: drv_Crystalfontz.c,v $
+ * Revision 1.35  2005/08/21 08:18:56  reinelt
+ * CrystalFontz ACK processing
+ *
  * Revision 1.34  2005/05/08 04:32:44  reinelt
  * CodingStyle added and applied
  *
@@ -173,10 +176,12 @@
 #include <stdio.h>
 #include <string.h>
 #include <unistd.h>
+#include <sys/time.h>
 
 #include "debug.h"
 #include "cfg.h"
 #include "qprintf.h"
+#include "thread.h"
 #include "timer.h"
 #include "plugin.h"
 #include "widget.h"
@@ -202,6 +207,7 @@ static unsigned int RingWPos = 0;
 /* packet from the display */
 struct {
     unsigned char type;
+    unsigned char code;
     unsigned char size;
     unsigned char data[16 + 1];        /* trailing '\0' */
 } Packet;
@@ -287,38 +293,62 @@ static void drv_CF_process_packet(void)
 {
 
     switch (Packet.type) {
+       
+    case 0x02:
 
-    case 0x80:                 /* Key Activity */
-       debug("Key Activity: %d", Packet.data[0]);
-       break;
-
-    case 0x81:                 /* Fan Speed Report */
-       if (Packet.data[1] == 0xff) {
-           Fan_RPM[Packet.data[0]] = -1.0;
-       } else if (Packet.data[1] < 4) {
-           Fan_RPM[Packet.data[0]] = 0.0;
-       } else {
-           Fan_RPM[Packet.data[0]] = (double) 27692308L *(Packet.data[1] - 3) / (Packet.data[2] + 256 * Packet.data[3]);
-       }
-       break;
+       /* async response from display to host */
+       switch (Packet.code) {
 
-    case 0x82:                 /* Temperature Sensor Report */
-       switch (Packet.data[3]) {
-       case 0:
-           error("%s: 1-Wire device #%d: CRC error", Name, Packet.data[0]);
+       case 0x00:
+           /* Key Activity */
+           debug("Key Activity: %d", Packet.data[0]);
+           break;
+           
+       case 0x01:
+           /* Fan Speed Report */
+           if (Packet.data[1] == 0xff) {
+               Fan_RPM[Packet.data[0]] = -1.0;
+           } else if (Packet.data[1] < 4) {
+               Fan_RPM[Packet.data[0]] = 0.0;
+           } else {
+               Fan_RPM[Packet.data[0]] = (double) 27692308L *(Packet.data[1] - 3) / (Packet.data[2] + 256 * Packet.data[3]);
+           }
            break;
-       case 1:
-       case 2:
-           Temperature[Packet.data[0]] = (Packet.data[1] + 256 * Packet.data[2]) / 16.0;
+           
+       case 0x02:                      
+           /* Temperature Sensor Report */
+           switch (Packet.data[3]) {
+           case 0:
+               error("%s: 1-Wire device #%d: CRC error", Name, Packet.data[0]);
+               break;
+           case 1:
+           case 2:
+               Temperature[Packet.data[0]] = (Packet.data[1] + 256 * Packet.data[2]) / 16.0;
+               break;
+           default:
+               error("%s: 1-Wire device #%d: unknown CRC status %d", Name, Packet.data[0], Packet.data[3]);
+               break;
+           }
            break;
+           
        default:
-           error("%s: 1-Wire device #%d: unknown CRC status %d", Name, Packet.data[0], Packet.data[3]);
+           /* this should not happen */
+           error ("%s: unexpected response type=0x%02x code=0x%02x size=%d", Packet.type, Packet.code, Packet.size);
            break;
        }
+
+       break;
+       
+    case 0x03:
+       /* error response from display to host */
+       error ("%s: error response type=0x%02x code=0x%02x size=%d", Packet.type, Packet.code, Packet.size);
        break;
 
     default:
-       /* just ignore packet */
+       /* these should not happen: */
+       /* type 0x00: command from host to display: should never come back */
+       /* type 0x01: command response from display to host: are processed within send() */
+       error ("%s: unexpected packet type=0x%02x code=0x%02x size=%d", Packet.type, Packet.code, Packet.size);
        break;
     }
 
@@ -375,7 +405,8 @@ static int drv_CF_poll(void)
        if (MSB(crc) != buffer[size + 3])
            goto GARBAGE;
        /* process packet */
-       Packet.type = buffer[0];
+       Packet.type = buffer[0] >> 6;
+       Packet.code = buffer[0] & 0x3f;
        Packet.size = size;
        memcpy(Packet.data, buffer + 2, size);
        Packet.data[size] = '\0';       /* trailing zero */
@@ -406,14 +437,15 @@ static void drv_CF_timer(void __attribute__ ((unused)) * notused)
 }
 
 
-static void drv_CF_send(const int cmd, int len, const unsigned char *data)
+static void drv_CF_send(const unsigned char cmd, const unsigned char len, const unsigned char *data)
 {
     unsigned char buffer[22];
     unsigned short crc;
+    struct timeval now, end;
 
     if (len > Payload) {
        error("%s: internal error: packet length %d exceeds payload size %d", Name, len, Payload);
-       len = sizeof(buffer) - 1;
+       return;
     }
 
     buffer[0] = cmd;
@@ -423,12 +455,33 @@ static void drv_CF_send(const int cmd, int len, const unsigned char *data)
     buffer[len + 2] = LSB(crc);
     buffer[len + 3] = MSB(crc);
 
-#if 0
-    debug("Tx Packet %d len=%d", buffer[0], buffer[1]);
-#endif
-
     drv_generic_serial_write((char *) buffer, len + 4);
-
+    
+    /* wait for acknowledge packet */
+    gettimeofday(&now, NULL);
+    while (1) {
+       /* delay 1 msec */
+       usleep (1 * 1000);
+       if (drv_CF_poll()) {
+           if (Packet.type == 0x01 && Packet.code == cmd) {
+               /* this is the ack we're waiting for */
+               if (0) {
+                   gettimeofday(&end, NULL);
+                   debug ("%s: ACK after %d usec", Name, 1000000 * (end.tv_sec - now.tv_sec) + end.tv_usec - now.tv_usec);
+               }
+               break;
+           } else {
+               /* some other (maybe async) packet, just process it */
+               drv_CF_process_packet();
+           }
+       }
+       gettimeofday(&end, NULL);
+       /* don't wait more than 250 msec */
+       if ((1000000 * (end.tv_sec - now.tv_sec) + end.tv_usec - now.tv_usec) > 250*1000) {
+           error("%s: timeout waiting for response to cmd 0x%02x", Name, cmd);
+           break;
+       }
+    }
 }
 
 
@@ -634,7 +687,7 @@ static int drv_CF_fan_pwm(int fan, int power)
 
 static int drv_CF_autodetect(void)
 {
-    int i, m;
+    int m;
 
     /* only autodetect newer displays */
     if (Protocol < 2)
@@ -642,43 +695,30 @@ static int drv_CF_autodetect(void)
 
     /* read display type */
     drv_CF_send(1, 0, NULL);
-
-    i = 0;
-    while (1) {
-       /* wait 10 msec */
-       usleep(10 * 1000);
-       /* packet available? */
-       if (drv_CF_poll()) {
-           /* display type */
-           if (Packet.type == 0x41) {
-               char t[7], c;
-               float h, v;
-               info("%s: display identifies itself as '%s'", Name, Packet.data);
-               if (sscanf((char *) Packet.data, "%6s:h%f,%c%f", t, &h, &c, &v) != 4) {
-                   error("%s: error parsing display identification string", Name);
-                   return -1;
-               }
-               info("%s: display type '%s', hardware version %3.1f, firmware version %c%3.1f", Name, t, h, c, v);
-               if (strncmp(t, "CFA", 3) == 0) {
-                   for (m = 0; Models[m].type != -1; m++) {
-                       /* omit the 'CFA' */
-                       if (strcasecmp(Models[m].name, t + 3) == 0)
-                           return m;
-                   }
-               }
-               error("%s: display type '%s' may be not supported!", Name, t);
-               return -1;
-           }
-           drv_CF_process_packet();
-       }
-       /* wait no longer than 300 msec */
-       if (++i > 30) {
-           error("%s: display detection timed out", Name);
+    
+    /* send() did already wait for response packet */
+    if (Packet.type == 0x01 && Packet.code == 0x01) {
+       char t[7], c;
+       float h, v;
+       info("%s: display identifies itself as '%s'", Name, Packet.data);
+       if (sscanf((char *) Packet.data, "%6s:h%f,%c%f", t, &h, &c, &v) != 4) {
+           error("%s: error parsing display identification string", Name);
            return -1;
        }
+       info("%s: display type '%s', hardware version %3.1f, firmware version %c%3.1f", Name, t, h, c, v);
+       if (strncmp(t, "CFA", 3) == 0) {
+           for (m = 0; Models[m].type != -1; m++) {
+               /* omit the 'CFA' */
+               if (strcasecmp(Models[m].name, t + 3) == 0)
+                   return m;
+           }
+       }
+       error("%s: display type '%s' may be not supported!", Name, t);
+       return -1;
     }
 
-    /* not reached */
+    error("%s: display detection failed!", Name);
+    
     return -1;
 }
 
@@ -708,7 +748,7 @@ static int drv_CF_scan_DOW(unsigned char index)
        /* packet available? */
        if (drv_CF_poll()) {
            /* DOW Device Info */
-           if (Packet.type == 0x52) {
+           if (Packet.type == 0x01 && Packet.code == 0x12) {
                switch (Packet.data[1]) {
                case 0x00:
                    /* no device found */

diff --git a/drv_Trefon.c b/drv_Trefon.c
index 502e55a3a3c3fc9e7c588e8ae27db9604c81c619..6f4682c4c41a79ead45f282b2e6136f09437a647 100644 (file)
--- a/drv_Trefon.c
+++ b/drv_Trefon.c
@@ -1,4 +1,4 @@
-/* $Id: drv_Trefon.c,v 1.4 2005/08/20 10:10:13 reinelt Exp $
+/* $Id: drv_Trefon.c,v 1.5 2005/08/21 08:18:56 reinelt Exp $
  *
  * driver for TREFON USB LCD displays - http://www.trefon.de
  *
@@ -23,9 +23,10 @@
  *
  *
  * $Log: drv_Trefon.c,v $
- * Revision 1.4  2005/08/20 10:10:13  reinelt
- *
+ * Revision 1.5  2005/08/21 08:18:56  reinelt
+ * CrystalFontz ACK processing
  *
+ * Revision 1.4  2005/08/20 10:10:13  reinelt
  * TREFON patch from Stephan Trautvetter:
  * drv_TF_init: CHAR0 set to 0 instead of 1
  * drv_TF_write: combine the GOTO and the data into one packet

diff --git a/lcd4linux.conf.sample b/lcd4linux.conf.sample
index c918590868e92cca39600b9e76a13661f9b7012c..618313542d63f180be81a052666697f467c4b959 100644 (file)
--- a/lcd4linux.conf.sample
+++ b/lcd4linux.conf.sample
@@ -55,7 +55,7 @@ Display CF631 {
     Port '/dev/tts/USB0'
     Speed 115200
     Contrast 95
-    Backlight 255
+    Backlight 50
     Icons 1
 }
 
@@ -680,6 +680,21 @@ Widget Test {
     width 40
 }
 
+Widget Test1 {
+    class 'Text'
+    expression 'ABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890'
+    width 20
+    align 'M'
+    speed 25
+}
+
+Widget Test2 {
+    class 'Text'
+    expression '1234567890abcdefghijklmnopqrstuvwxyz'
+    width 20
+    align 'M'
+    speed 10
+}
 
 Layout Default {
     Row1 {
@@ -768,6 +783,17 @@ Layout Test {
 #    Row16.Col1 'Test'
 }
 
+Layout Test2 {
+    Row01.Col1 'Test1'
+    #Row02.Col1 'Rain'
+    Row02.Col1 'Test2'
+    Row03.Col1 'Test1'
+    Row04.Col1 'Test2'
+    Row05.Col1 'Test1'
+    Row06.Col1 'Test2'
+    Row07.Col1 'Test1'
+    Row08.Col1 'Test2'
+}
 
 Layout testMySQL {
        Row1 {
@@ -789,14 +815,14 @@ Layout testMySQL {
 #Display 'WDC2704M'
 #Display 'SC1602D'
 #Display 'LCM-162'
-#Display 'CF631'
+Display 'CF631'
 #Display 'CF632'
 #Display 'CF633'
 #Display 'Curses'
 #Display 'M50530-24x8'
 #Display 'LCDTerm'
 #Display 'SimpleLCD'
-Display 'BA63'
+#Display 'BA63'
 #Display 'CT20x4'
 #Display 'T6963-240x64'
 #Display 'XWindow'
@@ -808,8 +834,9 @@ Display 'BA63'
 #Layout  'Default'
 #Layout 'L8x2'
 #Layout 'L16x2'
-Layout 'L20x2'
+#Layout 'L20x2'
 #Layout 'Test'
+Layout 'Test2'
 
 
 Variables {