X-Git-Url: https://codewiz.org/gitweb?a=blobdiff_plain;f=mware%2Fxmodem.c;h=559a0c5df331609be6287dd13209f008450ca802;hb=0cfcaed1ee7c5dc44cac2840107033798f32b40e;hp=ffb078c5497cc173d34019d309b8ca7d4f2489ed;hpb=9a9063256c370f7cfa9ef210180e61eb81cf264e;p=bertos.git

diff --git a/mware/xmodem.c b/mware/xmodem.c
index ffb078c5..559a0c5d 100755
--- a/mware/xmodem.c
+++ b/mware/xmodem.c
@@ -1,3 +1,4 @@
+#error This module has not been revised for the API changes in several DevLib modules
 /*!
  * \file
  * <!--
@@ -10,19 +11,51 @@
  * Suppots the CRC-16 and 1K-blocks variants of the standard.
  * \see ymodem.txt for the protocol description.
  *
+ * \todo Decouple this code from the LCD, buzzer and timer drivers
+ *       introducing user hooks or macros like CHECK_ABORT.
+ *
+ * \todo Break xmodem_send() and xmodem_recv() in smaller functions.
+ *
+ * \todo Add CONFIG_* vars to exclude either the receiver or the sender,
+ *       to reduce the footprint for applications that don't need both.
+ *
+ * \todo Maybe convert drv/ser.c to the KFile interface for symmetry and
+ *       flexibility.
+ *
  * \version $Id$
  * \author Bernardo Innocenti <bernie@develer.com>
  */
 
-/*
- * $Log$
- * Revision 1.2  2004/08/12 23:24:07  bernie
- * Rename UPDCRC() to UPDCRC16().
- *
- * Revision 1.1  2004/08/11 19:54:22  bernie
- * Import XModem protocol into DevLib.
- *
- */
+/*#*
+ *#* $Log$
+ *#* Revision 1.8  2004/08/25 14:12:09  rasky
+ *#* Aggiornato il comment block dei log RCS
+ *#*
+ *#* Revision 1.7  2004/08/15 06:30:06  bernie
+ *#* Make the buffer a local variable, as documented.
+ *#*
+ *#* Revision 1.6  2004/08/15 05:31:46  bernie
+ *#* Add an #error to spread some FUD about the quality of this module;
+ *#* Add a few TODOs from Rasky's review;
+ *#* Update to the new drv/ser.c API;
+ *#* Move FlushSerial() to drv/ser.c and generalize.
+ *#*
+ *#* Revision 1.5  2004/08/12 23:46:21  bernie
+ *#* Remove extra indentation level in switch statements.
+ *#*
+ *#* Revision 1.4  2004/08/12 23:35:50  bernie
+ *#* Replace a handmade loop with memset().
+ *#*
+ *#* Revision 1.3  2004/08/12 23:34:36  bernie
+ *#* Replace if/else with continue to reduce indentation level.
+ *#*
+ *#* Revision 1.2  2004/08/12 23:24:07  bernie
+ *#* Rename UPDCRC() to UPDCRC16().
+ *#*
+ *#* Revision 1.1  2004/08/11 19:54:22  bernie
+ *#* Import XModem protocol into DevLib.
+ *#*
+ *#*/
 
 #include "xmodem.h"
 
@@ -32,6 +65,8 @@
 #include <mware/crc.h>
 #include <mware/kfile.h>
 
+#include <string.h> /* for memset() */
+
 
 /*!
  * \name Protocol control codes
@@ -68,21 +103,16 @@
 #endif /* ARCH_BOOT */
 
 
-/*! Buffer to hold a block of data */
-static char block_buffer[XM_BUFSIZE];
-
-
 /*!
- * Decode serial driver errors and print
- * them on the display.
+ * Decode serial driver errors and print them on the display.
  */
-static void SerialError(int retries)
+static void print_serial_error(struct Serial *port, int retries)
 {
 	serstatus_t err, status;
 
 	/* Get serial error code and reset it */
-	status = ser_getstatus();
-	ser_setstatus(0);
+	status = ser_getstatus(port);
+	ser_setstatus(port, 0);
 
 	/* Mostra tutti gli errori in sequenza */
 	for (err = 0; status != 0; status >>= 1, err++)
@@ -99,21 +129,16 @@ static void SerialError(int retries)
 
 
 /*!
- * Reset previous serial errors and flush the receive buffer
- * (set a short timeout to speed up purge)
+ * \brief Receive a file using the XModem protocol.
+ *
+ * \param port Serial port to use for transfer
+ * \param fd Destination file
+ *
+ * \note This function allocates a large amount of stack (>1KB).
  */
-static void FlushSerial(void)
-{
-	ser_setstatus(0);
-	ser_settimeouts(200, SER_DEFTXTIMEOUT);
-	while (ser_getchar() != EOF) {}
-	ser_settimeouts(SER_DEFRXTIMEOUT, SER_DEFTXTIMEOUT);
-	ser_setstatus(0);
-}
-
-
-bool xmodem_recv(KFile *fd)
+bool xmodem_recv(struct Serial *port, KFile *fd)
 {
+	char block_buffer[XM_BUFSIZE]; /* Buffer to hold a block of data */
 	int c, i, blocksize;
 	int blocknr = 0, last_block_done = 0, retries = 0;
 	char *buf;
@@ -126,16 +151,16 @@ bool xmodem_recv(KFile *fd)
 	lcd_printf(0, 2, LCD_FILL, "Starting Transfer...");
 	lcd_clear();
 	purge = true;
-	ser_settimeouts(SER_DEFRXTIMEOUT, SER_DEFTXTIMEOUT);
-	ser_setstatus(0);
+	ser_settimeouts(port, SER_DEFRXTIMEOUT, SER_DEFTXTIMEOUT);
+	ser_setstatus(port, 0);
 
 	/* Send initial NAK to start transmission */
 	for(;;)
 	{
 		if (CHECK_ABORT)
 		{
-			ser_putchar(XM_CAN);
-			ser_putchar(XM_CAN);
+			ser_putchar(XM_CAN, port);
+			ser_putchar(XM_CAN, port);
 			lcd_printf(0, 2, LCD_FILL, "Transfer aborted");
 			return false;
 		}
@@ -151,13 +176,13 @@ bool xmodem_recv(KFile *fd)
 			if (ser_getstatus())
 				SerialError(retries);
 
-			FlushSerial();
+			ser_resync(port, 200);
 			retries++;
 
 			if (retries >= XM_MAXRETRIES)
 			{
-				ser_putchar(XM_CAN);
-				ser_putchar(XM_CAN);
+				ser_putchar(XM_CAN, port);
+				ser_putchar(XM_CAN, port);
 				lcd_printf(0, 2, LCD_FILL, "Transfer aborted");
 				return false;
 			}
@@ -168,159 +193,169 @@ bool xmodem_recv(KFile *fd)
 				if (retries < XM_MAXCRCRETRIES)
 				{
 					lcd_printf(0, 2, LCD_FILL, "Request Tx (CRC)");
-					ser_putchar(XM_C);
+					ser_putchar(XM_C, port);
 				}
 				else
 				{
 					/* Give up with CRC and fall back to checksum */
 					usecrc = false;
 					lcd_printf(0, 2, LCD_FILL, "Request Tx (BCC)");
-					ser_putchar(XM_NAK);
+					ser_putchar(XM_NAK, port);
 				}
 			}
 			else
-				ser_putchar(XM_NAK);
+				ser_putchar(XM_NAK, port);
 		}
 
-		switch (ser_getchar())
+		switch (ser_getchar(port))
 		{
-			case XM_STX:  /* Start of header (1024-byte block) */
-				blocksize = 1024;
-				goto getblock;
+		case XM_STX:  /* Start of header (1024-byte block) */
+			blocksize = 1024;
+			goto getblock;
 
-			case XM_SOH:  /* Start of header (128-byte block) */
-				blocksize = 128;
+		case XM_SOH:  /* Start of header (128-byte block) */
+			blocksize = 128;
 
-			getblock:
-				/* Get block number */
-				c = ser_getchar();
+		getblock:
+			/* Get block number */
+			c = ser_getchar(port);
 
-				/* Check complemented block number */
-				if ((~c & 0xff) != ser_getchar())
-				{
-					lcd_printf(0, 3, LCD_FILL, "Bad blk (%d)", c);
-					purge = true;
-					break;
-				}
+			/* Check complemented block number */
+			if ((~c & 0xff) != ser_getchar(port))
+			{
+				lcd_printf(0, 3, LCD_FILL, "Bad blk (%d)", c);
+				purge = true;
+				break;
+			}
 
-				/* Determine which block is being sent */
-				if (c == (blocknr & 0xff))
-					/* Last block repeated */
-					lcd_printf(0, 2, LCD_FILL, "Repeat blk %d", blocknr);
-				else if (c == ((blocknr + 1) & 0xff))
-					/* Next block */
-					lcd_printf(0, 2, LCD_FILL, "Recv blk %d", ++blocknr);
-				else
+			/* Determine which block is being sent */
+			if (c == (blocknr & 0xff))
+				/* Last block repeated */
+				lcd_printf(0, 2, LCD_FILL, "Repeat blk %d", blocknr);
+			else if (c == ((blocknr + 1) & 0xff))
+				/* Next block */
+				lcd_printf(0, 2, LCD_FILL, "Recv blk %d", ++blocknr);
+			else
+			{
+				/* Sync lost */
+				lcd_printf(0, 3, LCD_FILL, "Sync lost (%d/%d)", c, blocknr);
+				purge = true;
+				break;
+			}
+
+			buf = block_buffer;	/* Reset pointer to start of buffer */
+			checksum = 0;
+			crc = 0;
+			for (i = 0; i < blocksize; i++)
+			{
+				if ((c = ser_getchar(port)) == EOF)
 				{
-					/* Sync lost */
-					lcd_printf(0, 3, LCD_FILL, "Sync lost (%d/%d)", c, blocknr);
 					purge = true;
 					break;
 				}
 
-				buf = block_buffer;	/* Reset pointer to start of buffer */
-				checksum = 0;
-				crc = 0;
-				for (i = 0; i < blocksize; i++)
-				{
-					if ((c = ser_getchar()) == EOF)
-					{
-						purge = true;
-						break;
-					}
+				/* Store in buffer */
+				*buf++ = (char)c;
 
-					/* Store in buffer */
-					*buf++ = (char)c;
+				/* Calculate block checksum or CRC */
+				if (usecrc)
+					crc = UPDCRC16(c, crc);
+				else
+					checksum += (char)c;
+			}
 
-					/* Calculate block checksum or CRC */
-					if (usecrc)
-						crc = UPDCRC16(c, crc);
-					else
-						checksum += (char)c;
-				}
+			if (purge)
+				break;
 
-				if (purge)
-					break;
+			/* Get the checksum byte or the CRC-16 MSB */
+			if ((c = ser_getchar(port)) == EOF)
+			{
+				purge = true;
+				break;
+			}
+
+			if (usecrc)
+			{
+				crc = UPDCRC16(c, crc);
 
-				/* Get the checksum byte or the CRC-16 MSB */
-				if ((c = ser_getchar()) == EOF)
+				/* Get CRC-16 LSB */
+				if ((c = ser_getchar(port)) == EOF)
 				{
 					purge = true;
 					break;
 				}
 
-				if (usecrc)
-				{
-					crc = UPDCRC16(c, crc);
+				crc = UPDCRC16(c, crc);
 
-					/* Get CRC-16 LSB */
-					if ((c = ser_getchar()) == EOF)
-					{
-						purge = true;
-						break;
-					}
-
-					crc = UPDCRC16(c, crc);
-
-					if (crc)
-					{
-						lcd_printf(0, 3, LCD_FILL, "Bad CRC: %04x", crc);
-						purge = true;
-						break;
-					}
-				}
-				/* Compare the checksum */
-				else if (c != checksum)
+				if (crc)
 				{
-					lcd_printf(0, 3, LCD_FILL, "Bad sum: %04x/%04x", checksum, c);
+					lcd_printf(0, 3, LCD_FILL, "Bad CRC: %04x", crc);
 					purge = true;
 					break;
 				}
+			}
+			/* Compare the checksum */
+			else if (c != checksum)
+			{
+				lcd_printf(0, 3, LCD_FILL, "Bad sum: %04x/%04x", checksum, c);
+				purge = true;
+				break;
+			}
 
-				/*
-				 * Avoid flushing the same block twice.
-				 * This could happen when the sender does not receive our
-				 * acknowledge and resends the same block.
-				 */
-				if (last_block_done < blocknr)
+			/*
+			 * Avoid flushing the same block twice.
+			 * This could happen when the sender does not receive our
+			 * acknowledge and resends the same block.
+			 */
+			if (last_block_done < blocknr)
+			{
+				/* Call user function to flush the buffer */
+				if (fd->write(fd, block_buffer, blocksize))
 				{
-					/* Call user function to flush the buffer */
-					if (fd->write(fd, block_buffer, blocksize))
-					{
-						/* Acknowledge block and clear error counter */
-						ser_putchar(XM_ACK);
-						retries = 0;
-						last_block_done = blocknr;
-					}
-					else
-					{
-						/* User callback failed: abort transfer immediately */
-						retries = XM_MAXRETRIES;
-						purge = true;
-					}
+					/* Acknowledge block and clear error counter */
+					ser_putchar(XM_ACK, port);
+					retries = 0;
+					last_block_done = blocknr;
 				}
-				break;
+				else
+				{
+					/* User callback failed: abort transfer immediately */
+					retries = XM_MAXRETRIES;
+					purge = true;
+				}
+			}
+			break;
 
-			case XM_EOT:	/* End of transmission */
-				ser_putchar(XM_ACK);
-				lcd_printf(0, 2, LCD_FILL, "Transfer completed");
-				return true;
+		case XM_EOT:	/* End of transmission */
+			ser_putchar(XM_ACK, port);
+			lcd_printf(0, 2, LCD_FILL, "Transfer completed");
+			return true;
 
-			case EOF: /* Timeout or serial error */
-				purge = true;
-				break;
+		case EOF: /* Timeout or serial error */
+			purge = true;
+			break;
 
-			default:
-				lcd_printf(0, 3, LCD_FILL, "Skipping garbage");
-				purge = true;
-				break;
+		default:
+			lcd_printf(0, 3, LCD_FILL, "Skipping garbage");
+			purge = true;
+			break;
 		}
 	} /* End forever */
 }
 
 
-bool xmodem_send(KFile *fd)
+/*!
+ * \brief Transmit a file using the XModem protocol.
+ *
+ * \param port Serial port to use for transfer
+ * \param fd Source file
+ *
+ * \note This function allocates a large amount of stack for
+ *       the XModem transfer buffer (1KB).
+ */
+bool xmodem_send(struct Serial *port, KFile *fd)
 {
+	char block_buffer[XM_BUFSIZE]; /* Buffer to hold a block of data */
 	size_t size = -1;
 	int blocknr = 1, retries = 0, c, i;
 	bool proceed, usecrc = false;
@@ -328,9 +363,9 @@ bool xmodem_send(KFile *fd)
 	uint8_t sum;
 
 
-	ser_settimeouts(SER_DEFRXTIMEOUT, SER_DEFTXTIMEOUT);
-	ser_setstatus(0);
-	FlushSerial();
+	ser_settimeouts(port, SER_DEFRXTIMEOUT, SER_DEFTXTIMEOUT);
+	ser_setstatus(port, 0);
+	ser_purge(port);
 	lcd_printf(0, 2, LCD_FILL, "Wait remote host");
 
 	for(;;)
@@ -341,93 +376,92 @@ bool xmodem_send(KFile *fd)
 			if (CHECK_ABORT)
 				return false;
 
-			switch (c = ser_getchar())
+			switch (c = ser_getchar(port))
 			{
-				case XM_NAK:
-				case XM_C:
-					if (blocknr == 1)
+			case XM_NAK:
+			case XM_C:
+				if (blocknr == 1)
+				{
+					if (c == XM_C)
 					{
-						if (c == XM_C)
-						{
-							lcd_printf(0, 2, LCD_FILL, "Tx start (CRC)");
-							usecrc = true;
-						}
-						else
-							lcd_printf(0, 2, LCD_FILL, "Tx start (BCC)");
-
-						/* Call user function to read in one block */
-						size = fd->read(fd, block_buffer, XM_BUFSIZE);
+						lcd_printf(0, 2, LCD_FILL, "Tx start (CRC)");
+						usecrc = true;
 					}
 					else
-						lcd_printf(0, 2, LCD_FILL, "Resend blk %d", blocknr);
-					proceed = true;
-					break;
-
-				case XM_ACK:
-					/* End of transfer? */
-					if (!size)
-						return true;
+						lcd_printf(0, 2, LCD_FILL, "Tx start (BCC)");
 
 					/* Call user function to read in one block */
 					size = fd->read(fd, block_buffer, XM_BUFSIZE);
-					blocknr++;
-					retries = 0;
-					proceed = true;
-					lcd_printf(0, 2, LCD_FILL, "Send blk %d", blocknr);
-					break;
-
-				case EOF:
-					retries++;
-					SerialError(retries);
-					if (retries <= XM_MAXRETRIES)
-						break;
-					/* falling through! */
+				}
+				else
+					lcd_printf(0, 2, LCD_FILL, "Resend blk %d", blocknr);
+				proceed = true;
+				break;
 
-				case XM_CAN:
-					lcd_printf(0, 2, LCD_FILL, "Transfer aborted");
-					return false;
+			case XM_ACK:
+				/* End of transfer? */
+				if (!size)
+					return true;
+
+				/* Call user function to read in one block */
+				size = fd->read(fd, block_buffer, XM_BUFSIZE);
+				blocknr++;
+				retries = 0;
+				proceed = true;
+				lcd_printf(0, 2, LCD_FILL, "Send blk %d", blocknr);
+				break;
 
-				default:
-					lcd_printf(0, 3, LCD_FILL, "Skipping garbage");
+			case EOF:
+				retries++;
+				SerialError(retries);
+				if (retries <= XM_MAXRETRIES)
 					break;
+				/* falling through! */
+
+			case XM_CAN:
+				lcd_printf(0, 2, LCD_FILL, "Transfer aborted");
+				return false;
+
+			default:
+				lcd_printf(0, 3, LCD_FILL, "Skipping garbage");
+				break;
 			}
 		}
 		while (!proceed);
 
 		if (!size)
-			ser_putchar(XM_EOT);
-		else
 		{
-			/* Pad block with 0xFF if it's partially full */
-			if (size < XM_BUFSIZE)
-				for (i = size; i < XM_BUFSIZE; i++)
-					block_buffer[i] = (char)0xFF;
-
-			/* Send block header (STX, blocknr, ~blocknr) */
-			ser_putchar(XM_STX);
-			ser_putchar(blocknr & 0xFF);
-			ser_putchar(~blocknr & 0xFF);
-
-			/* Send block and compute its CRC/checksum */
-			sum = 0;
-			crc = 0;
-			for (i = 0; i < XM_BUFSIZE; i++)
-			{
-				ser_putchar(block_buffer[i]);
-				crc = UPDCRC16(block_buffer[i], crc);
-				sum += block_buffer[i];
-			}
+			ser_putchar(XM_EOT, port);
+			continue;
+		}
 
-			/* Send CRC/Checksum */
-			if (usecrc)
-			{
-				crc = UPDCRC16(0, crc);
-				crc = UPDCRC16(0, crc);
-				ser_putchar(crc >> 8);
-				ser_putchar(crc & 0xFF);
-			}
-			else
-				ser_putchar(sum);
+		/* Pad block with 0xFF if it's partially full */
+		memset(block_buffer + size, 0xFF, XM_BUFSIZE - size);
+
+		/* Send block header (STX, blocknr, ~blocknr) */
+		ser_putchar(XM_STX, port);
+		ser_putchar(blocknr & 0xFF, port);
+		ser_putchar(~blocknr & 0xFF, port);
+
+		/* Send block and compute its CRC/checksum */
+		sum = 0;
+		crc = 0;
+		for (i = 0; i < XM_BUFSIZE; i++)
+		{
+			ser_putchar(block_buffer[i], port);
+			crc = UPDCRC16(block_buffer[i], crc);
+			sum += block_buffer[i];
 		}
+
+		/* Send CRC/Checksum */
+		if (usecrc)
+		{
+			crc = UPDCRC16(0, crc);
+			crc = UPDCRC16(0, crc);
+			ser_putchar(crc >> 8, port);
+			ser_putchar(crc & 0xFF, port);
+		}
+		else
+			ser_putchar(sum, port);
 	}
 }