From e3a1ba08ab4ced37a7a76292971f7b018249051b Mon Sep 17 00:00:00 2001
From: Tectu <joel@unormal.org>
Date: Mon, 16 Jul 2012 03:05:12 +0200
Subject: [PATCH] implemented worker thread for glcd

---
 glcd.c   | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++---
 glcd.h   |  1 +
 worker.h | 39 ++++++++++++++++++++++++++++++++++++++
 3 files changed, 94 insertions(+), 3 deletions(-)
 create mode 100644 worker.h

diff --git a/glcd.c b/glcd.c
index 930a7e40..d956571c 100644
--- a/glcd.c
+++ b/glcd.c
@@ -1,11 +1,49 @@
 #include "glcd.h" 
-#include "fonts.h"
 #include <stdlib.h>
 #include <math.h>
 
 uint16_t lcd_width, lcd_height;
+static Thread *workerThread = NULL;
+ 
+static WORKING_AREA(waGLCDWorkerThread, GLCD_WORKER_SIZE);
+static msg_t ThreadGLCDWorker(void *arg) {
+	(void)arg;
+	Thread *p;
+ 
+	chRegSetThreadName("GLCDWorker");
+ 
+	while(TRUE) {
+		/* Wait for msg with work to do. */
+		p = chMsgWait();
+		struct glcd_msg_base *msg = (struct glcd_msg_base*)chMsgGet(p);
+		msg->result = GLCD_PROGRESS;
+ 
+		/* do work here */
+		switch(msg->action) {
+			case GLCD_SET_CURSOR: {
+				const struct glcd_msg_set_cursor *emsg = (const struct glcd_msg_set_cursor*)msg;
+				lld_lcdSetCursor(emsg->x, emsg->y);
+				msg->result = GLCD_DONE;
+				break;
+			}
+ 
+			case GLCD_DRAW_PIXEL: {
+				const struct glcd_msg_draw_pixel *emsg = (const struct glcd_msg_draw_pixel*)msg;
+				lld_lcdDrawPixel(emsg->x, emsg->y, emsg->color);
+				msg->result = GLCD_DONE;
+				break;
+			}
+		}
+
+		/* Done, release msg again. */
+		chMsgRelease(p, 0);
+	}
+ 
+	return 0;
+}
 
 void lcdInit(GLCDDriver *glcdp) {
+	workerThread = chThdCreateStatic(waGLCDWorkerThread, sizeof(waGLCDWorkerThread), NORMALPRIO, ThreadGLCDWorker, NULL);
 
 	lld_lcdInit();
 	lcd_width = lcdGetWidth();
@@ -28,7 +66,13 @@ uint16_t lcdGetOrientation(void) {
 }
 
 static void lcdSetCursor(uint16_t x, uint16_t y) {
-	lld_lcdSetCursor(x, y);
+	struct glcd_msg_set_cursor msg;
+ 
+	msg.action = GLCD_SET_CURSOR;
+	msg.x = x;
+	msg.y = y;
+
+	chMsgSend(workerThread, (msg_t)&msg);
 }
 
 void lcdSetPowerMode(uint8_t powerMode) {
@@ -64,7 +108,14 @@ uint16_t lcdGetPixelColor(uint16_t x, uint16_t y) {
 }
 
 void lcdDrawPixel(uint16_t x, uint16_t y, uint16_t color) {
-	lld_lcdDrawPixel(x, y, color);
+	struct glcd_msg_draw_pixel msg;
+ 
+	msg.action = GLCD_DRAW_PIXEL;
+	msg.x = x;
+	msg.y = y;
+	msg.color = color;
+
+	chMsgSend(workerThread, (msg_t)&msg);
 }
 
 static void lcdWriteStreamStart(void) {
diff --git a/glcd.h b/glcd.h
index ac699dba..86e2dde5 100644
--- a/glcd.h
+++ b/glcd.h
@@ -4,6 +4,7 @@
 #include "ch.h"
 #include "hal.h"
 #include "fonts.h"
+#include "worker.h"
 
 #if !defined(LCD_USE_FSMC) && !defined(LCD_USE_GPIO) && !defined(LCD_USE_SPI)
 #include "glcdconf.h"
diff --git a/worker.h b/worker.h
new file mode 100644
index 00000000..6b52fb24
--- /dev/null
+++ b/worker.h
@@ -0,0 +1,39 @@
+#ifndef WORKER_H
+#define WORKER_H
+
+#define GLCD_WORKER_SIZE 2048
+
+enum glcd_action {	GLCD_SET_CURSOR,
+					GLCD_DRAW_PIXEL,
+				};
+
+enum glcd_result {	GLCD_DONE,
+					GLCD_FAILED,
+					GLCD_PROGRESS
+				};
+
+#define _glcd_msg_base			\
+	enum glcd_action action;	\
+	enum glcd_result result;
+
+struct glcd_msg_base {
+	_glcd_msg_base
+};
+
+struct glcd_msg_set_cursor {
+	_glcd_msg_base
+
+	uint16_t x;
+	uint16_t y;
+};
+
+struct glcd_msg_draw_pixel {
+	_glcd_msg_base
+
+	uint16_t x;
+	uint16_t y;
+	uint16_t color;
+};
+
+#endif
+