diff --git a/xpp/astribank_allow.c b/xpp/astribank_allow.c
index 39b7521..8f1ea92 100644
--- a/xpp/astribank_allow.c
+++ b/xpp/astribank_allow.c
@@ -142,7 +142,7 @@ int main(int argc, char *argv[])
 		return 1;
 	}
 	if (do_write) {
-		int used_marker;
+		unsigned int used_marker;
 		/* update capabilities based on input file */
 		file = stdin;
 		if (filename) {
diff --git a/xpp/astribank_license.c b/xpp/astribank_license.c
index 8eeb7c2..93c0d26 100644
--- a/xpp/astribank_license.c
+++ b/xpp/astribank_license.c
@@ -22,6 +22,7 @@
 
 #include <errno.h>
 #include <stdio.h>
+#include <ctype.h>
 #include <string.h>
 #include <debug.h>
 #include "astribank_license.h"
@@ -230,7 +231,7 @@ int read_from_file(
 	char buf[256];
 	char *line, *key, *value;
 	int lineno = 0;
-	unsigned int license_marker_begin;
+	unsigned int license_marker_begin = 0;
 	unsigned int license_marker_end;
 	struct table {
 		struct eeprom_table eeprom_table;
diff --git a/xpp/echo_loader.c b/xpp/echo_loader.c
index cbe6048..f5f28d3 100644
--- a/xpp/echo_loader.c
+++ b/xpp/echo_loader.c
@@ -123,7 +123,7 @@ static void usb_buffer_showstatistics(struct astribank_device *astribank, struct
 	long	usec;
 
 	usec = usb_buffer_usec(ub);
-	AB_INFO(astribank, "Octasic statistics: packet_size=[%d, %ld, %d] packets=%d, bytes=%ld msec=%ld usec/packet=%d\n",
+	AB_INFO(astribank, "Octasic statistics: packet_size=[%d, %ld, %d] packets=%d, bytes=%ld msec=%ld usec/packet=%ld\n",
 		ub->min_send,
 		ub->total_bytes / ub->num_sends,
 		ub->max_send,
@@ -822,9 +822,7 @@ UINT32 init_octasic(char *filename, struct astribank_device *astribank, int is_a
 
 int load_echo(struct astribank_device *astribank, char *filename, int is_alaw)
 {
-	int		iLen;
 	int		ret;
-	unsigned char	*pbyFileData = NULL;
 	UINT32		octasic_status;
 
 	AB_INFO(astribank, "Loading ECHOCAN Firmware: %s (%s)\n",
diff --git a/xpp/xtalk/xtalk.c b/xpp/xtalk/xtalk.c
index d3da5e0..7bfd359 100644
--- a/xpp/xtalk/xtalk.c
+++ b/xpp/xtalk/xtalk.c
@@ -243,16 +243,17 @@ static int send_command(struct xtalk_device *xtalk_dev, struct xtalk_command *cm
 {
 	int		ret;
 	int		len;
-	char		*buf;
 	void		*priv = xtalk_dev->transport_priv;
 
 	len = cmd->header.len;
 	cmd->header.seq = xtalk_dev->tx_sequenceno;
 
-	buf = (char *)cmd;
 	//printf("%s: len=%d\n", __FUNCTION__, len);
 #if 0
 	extern	FILE	*fp;
+	char		*buf;
+
+	buf = (char *)cmd;
 	if(fp) {
 		int	i;