X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=app%2Ftriface%2Fprotocol.c;h=08910d0426e26b85b0caaaf6c7efa11f34bb9068;hb=cb475c2071d6be7048f8c5c06485b3cd8530708c;hp=4fe2385633f8b11850702599fcd3ec3caf75b377;hpb=25bba83340c0583e25871a35ccfb860151abe5f6;p=bertos.git

diff --git a/app/triface/protocol.c b/app/triface/protocol.c
old mode 100755
new mode 100644
index 4fe23856..08910d04
--- a/app/triface/protocol.c
+++ b/app/triface/protocol.c
@@ -1,6 +1,31 @@
 /**
  * \file
  * <!--
+ * This file is part of BeRTOS.
+ *
+ * Bertos is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+ *
+ * As a special exception, you may use this file as part of a free software
+ * library without restriction.  Specifically, if other files instantiate
+ * templates or use macros or inline functions from this file, or you compile
+ * this file and link it with other files to produce an executable, this
+ * file does not by itself cause the resulting executable to be covered by
+ * the GNU General Public License.  This exception does not however
+ * invalidate any other reasons why the executable file might be covered by
+ * the GNU General Public License.
+ *
  * Copyright 2006 Develer S.r.l. (http://www.develer.com/)
  * -->
  *
@@ -14,22 +39,6 @@
  * \author Bernardo Innocenti <bernie@develer.com>
  */
 
-/*#*
- *#* $Log$
- *#* Revision 1.4  2006/06/14 00:26:48  marco
- *#* Use new macros for defining commands.
- *#*
- *#* Revision 1.3  2006/06/13 19:07:31  marco
- *#* Fixed a bug in protocol_reply. Simplified rpc.
- *#*
- *#* Revision 1.2  2006/06/12 21:37:02  marco
- *#* implemented some commands (ver and sleep)
- *#*
- *#* Revision 1.1  2006/06/01 12:29:21  marco
- *#* Add first simple protocol command (version request).
- *#*
- *#*/
-
 #include "protocol.h"
 
 #include <drv/ser.h>
@@ -40,9 +49,10 @@
 #include <cfg/debug.h>
 #include <verstag.h>
 
+#include <stdlib.h>
 #include <string.h>
 
-#include <cmd_hunk.h>
+//#include <cmd_hunk.h>
 
 #include "cmd_ctor.h"  // MAKE_CMD, REGISTER_CMD
 
@@ -65,6 +75,7 @@ static struct RLContext rl_ctx;
 /**
  * Send a NAK asking the host to send the current message again.
  *
+ * \param ser  serial port handle to output to.
  * \param err  human-readable description of the error for debug purposes.
  */
 INLINE void NAK(Serial *ser, const char *err)
@@ -80,32 +91,30 @@ INLINE void NAK(Serial *ser, const char *err)
  * Print args on s, with format specified in t->result_fmt.
  * Return number of valid arguments or -1 in case of error.
  */
-static int protocol_reply(Serial *s, const struct CmdTemplate *t,
+static bool protocol_reply(Serial *s, const struct CmdTemplate *t,
 			  const parms *args)
 {
 	unsigned short offset = strlen(t->arg_fmt) + 1;
 	unsigned short nres = strlen(t->result_fmt);
 
-	if (nres > 0)
+	ser_printf(s, "0");
+	for (unsigned short i = 0; i < nres; ++i)
 	{
-		for (unsigned short i = 0; i < nres; ++i)
+		if (t->result_fmt[i] == 'd')
 		{
-			if (t->result_fmt[i] == 'd')
-			{
-				ser_printf(s, "%ld", args[offset+i].l);
-			}
-			else if (t->result_fmt[i] == 's')
-			{
-				ser_printf(s, "%s ", args[offset+i].s);
-			}
-			else
-			{
-				return -1;
-			}
+			ser_printf(s, " %ld", args[offset+i].l);
+		}
+		else if (t->result_fmt[i] == 's')
+		{
+			ser_printf(s, " %s", args[offset+i].s);
+		}
+		else
+		{
+			abort();
 		}
 	}
-	ser_print(s, "\r\n");
-	return nres;
+	ser_printf(s, "\r\n");
+	return true;
 }
 
 static void protocol_parse(Serial *ser, const char *buf)
@@ -116,30 +125,27 @@ static void protocol_parse(Serial *ser, const char *buf)
 	templ = parser_get_cmd_template(buf);
 	if (!templ)
 	{
-		NAK(ser, "Invalid command.");
+		ser_print(ser, "-1 Invalid command.");
 		return;
 	}
 
 	parms args[PARSER_MAX_ARGS];
 
-	/* Args Check.  */
+	/* Args Check.  TODO: Handle different case. see doc/PROTOCOL .  */
 	if (!parser_get_cmd_arguments(buf, templ, args))
 	{
-		NAK(ser, "Invalid arguments.");
+		ser_print(ser, "-2 Invalid arguments.");
 		return;
 	}
 
 	/* Execute. */
-	if (!parser_execute_cmd(templ, args))
+	if(!parser_execute_cmd(templ, args))
 	{
-		NAK(ser, "Command failed.");
+		NAK(ser, "Error in executing command.");
 	}
-	else
+	if (!protocol_reply(ser, templ, args))
 	{
-		if (protocol_reply(ser, templ, args) < 0)
-		{
-			NAK(ser, "Invalid return format.");
-		}
+		NAK(ser, "Invalid return format.");
 	}
 	return;
 }
@@ -219,17 +225,24 @@ void protocol_run(Serial *ser)
  *
  */
 
-/* Version.  */
-MAKE_CMD(ver, "", "s",
+/* Version. Example of declaring function and passing it to MAKE_CMD.  */
+static int ver_fn(const char **str)
+{
+	*str = VERS_TAG;
+	return 0;
+}
+MAKE_CMD(ver, "", "s", ver_fn(&args[1].s))
+
+/* Sleep. Example of declaring function body directly in macro call.  */
+MAKE_CMD(sleep, "d", "",
 ({
-	args[1].s = VERS_TAG;
+	timer_delay((mtime_t)args[1].l);
 	0;
 }))
 
-/* Sleep.  */
-MAKE_CMD(sleep, "d", "",
+/* Ping.  */
+MAKE_CMD(ping, "", "",
 ({
-	timer_delay((mtime_t)args[1].l);
 	0;
 }))
 
@@ -238,6 +251,7 @@ static void protocol_registerCmds(void)
 {
 	REGISTER_CMD(ver);
 	REGISTER_CMD(sleep);
+	REGISTER_CMD(ping);
 }
 
 /* Initialization: readline context, parser and register commands.  */