X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=arducontrol%2Finterface.c;h=a8ae2a28af9a0fce11cd391b5dd80da3634cf38b;hb=4150740d3a5c2384289865923cef08c5852cb520;hp=4064120992bd16e4a38e20446f3d98085ff785ba;hpb=3bbca0dcce53ff9e2cdaa3d39ae18afcbf627f9d;p=model-railway-devices.git

diff --git a/arducontrol/interface.c b/arducontrol/interface.c
index 4064120..a8ae2a2 100644
--- a/arducontrol/interface.c
+++ b/arducontrol/interface.c
@@ -1,3 +1,4 @@
+#include <avr/io.h>
 #include "interface.h"
 #include "monitor.h"
 #include "motorola.h"
@@ -5,14 +6,11 @@
 #include "serial.h"
 #include "s88.h"
 
-volatile uint8_t recv_buf[32];
-uint8_t recv_head = 0;
-uint8_t recv_tail = 0;
-volatile uint8_t recv_fill = 0;
-volatile uint8_t recv_overrun = 0;
+static uint8_t cmd_buffer[15];
+static uint8_t cmd_length = 0;
+static uint8_t cmd_read_pos = 0;
 
-void process_commands(void);
-uint8_t dispatch_command(const uint8_t *, uint8_t);
+static uint8_t dispatch_command(const uint8_t *, uint8_t);
 
 void interface_init(void)
 {
@@ -23,55 +21,47 @@ void interface_init(void)
 
 void interface_check(void)
 {
-	if(recv_overrun)
-	{
+	uint8_t count;
+	if(serial_read_overrun())
 		interface_send1(RECEIVE_OVERRUN);
-		recv_overrun = 0;
-	}
-	if(recv_fill>0)
-		process_commands();
-}
 
-void process_commands(void)
-{
-	while(recv_fill>0)
+	count = serial_read_available();
+	if(count>0)
 	{
-		uint8_t cmd[15];
-		uint8_t length = ~recv_buf[recv_tail];
-
-		if(length<0x10)
+		if(cmd_length==0)
 		{
-			if(recv_fill<=length)
-				break;
-
-			uint8_t i, j;
-			for(i=0, j=recv_tail+1; i<length; ++i, ++j)
+			uint8_t l = ~serial_read();
+			if(l==0)
+				serial_write(0xFF);
+			else if(l>=0x10)
+				interface_send1(FRAMING_ERROR);
+			else
 			{
-				if(j>=sizeof(recv_buf))
-					j = 0;
-				cmd[i] = recv_buf[j];
+				cmd_length = l;
+				--count;
+				cmd_read_pos = 0;
 			}
 		}
-		else
-		{
-			length = 0;
-			interface_send1(FRAMING_ERROR);
-		}
-
-		recv_tail += length+1;
-		if(recv_tail>=sizeof(recv_buf))
-			recv_tail -= sizeof(recv_buf);
-		recv_fill -= length+1;
 
-		if(length>0)
+		if(cmd_read_pos<cmd_length)
 		{
-			uint8_t result = dispatch_command(cmd, length);
-			interface_send1(result);
+			uint8_t i;
+			if(cmd_read_pos+count>cmd_length)
+				count = cmd_length-cmd_read_pos;
+			for(i=0; i<count; ++i)
+				cmd_buffer[cmd_read_pos++] = serial_read();
+
+			if(cmd_read_pos>=cmd_length)
+			{
+				uint8_t result = dispatch_command(cmd_buffer, cmd_length);
+				interface_send1(result);
+				cmd_length = 0;
+			}
 		}
 	}
 }
 
-uint8_t dispatch_command(const uint8_t *cmd, uint8_t length)
+static uint8_t dispatch_command(const uint8_t *cmd, uint8_t length)
 {
 	uint8_t type = cmd[0]>>4;
 	if(type==0)
@@ -104,19 +94,3 @@ void interface_send1(uint8_t cmd)
 	serial_write(0xFE);
 	serial_write(cmd);
 }
-
-static inline void receive(uint8_t c)
-{
-	if(recv_fill>=sizeof(recv_buf))
-	{
-		recv_overrun = 1;
-		return;
-	}
-
-	recv_buf[recv_head++] = c;
-	if(recv_head>=sizeof(recv_buf))
-		recv_head = 0;
-	++recv_fill;
-}
-
-SERIAL_SET_CALLBACK(receive)