r3807 - in trunk/src/target/gsm: include/gsmd include/libgsmd src/gsmd src/libgsmd src/util

erin_yueh at sita.openmoko.org erin_yueh at sita.openmoko.org
Thu Jan 10 06:15:14 CET 2008


Author: erin_yueh
Date: 2008-01-10 06:14:53 +0100 (Thu, 10 Jan 2008)
New Revision: 3807

Modified:
   trunk/src/target/gsm/include/gsmd/gsmd.h
   trunk/src/target/gsm/include/gsmd/machineplugin.h
   trunk/src/target/gsm/include/gsmd/sms.h
   trunk/src/target/gsm/include/gsmd/unsolicited.h
   trunk/src/target/gsm/include/gsmd/usock.h
   trunk/src/target/gsm/include/gsmd/vendorplugin.h
   trunk/src/target/gsm/include/libgsmd/misc.h
   trunk/src/target/gsm/src/gsmd/atcmd.c
   trunk/src/target/gsm/src/gsmd/ext_response.c
   trunk/src/target/gsm/src/gsmd/gsmd.c
   trunk/src/target/gsm/src/gsmd/machine.c
   trunk/src/target/gsm/src/gsmd/machine_generic.c
   trunk/src/target/gsm/src/gsmd/sms_cb.c
   trunk/src/target/gsm/src/gsmd/timer.c
   trunk/src/target/gsm/src/gsmd/unsolicited.c
   trunk/src/target/gsm/src/gsmd/usock.c
   trunk/src/target/gsm/src/gsmd/vendor_bcm.c
   trunk/src/target/gsm/src/gsmd/vendor_qc.c
   trunk/src/target/gsm/src/gsmd/vendor_ti.c
   trunk/src/target/gsm/src/gsmd/vendor_tihtc.c
   trunk/src/target/gsm/src/libgsmd/libgsmd.c
   trunk/src/target/gsm/src/libgsmd/libgsmd_network.c
   trunk/src/target/gsm/src/libgsmd/libgsmd_sms.c
   trunk/src/target/gsm/src/util/atcmd.c
   trunk/src/target/gsm/src/util/event.c
   trunk/src/target/gsm/src/util/pin.c
   trunk/src/target/gsm/src/util/shell.c
Log:
gsmd: eliminate gcc warnings (Erin Yueh)


Modified: trunk/src/target/gsm/include/gsmd/gsmd.h
===================================================================
--- trunk/src/target/gsm/include/gsmd/gsmd.h	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/include/gsmd/gsmd.h	2008-01-10 05:14:53 UTC (rev 3807)
@@ -4,6 +4,7 @@
 #ifdef __GSMD__
 
 #include <sys/types.h>
+#include <sys/time.h>
 
 #include <common/linux_list.h>
 
@@ -117,6 +118,8 @@
 #define DEBUGP(x, args ...)	gsmd_log(GSMD_DEBUG, x, ## args)
 
 extern int gsmd_simplecmd(struct gsmd *gsmd, char *cmdtxt);
+extern int gsmd_initsettings(struct gsmd *gsmd);
+extern int gsmd_alive_start(struct gsmd *gsmd);
 
 /***********************************************************************
  * timer handling
@@ -130,8 +133,7 @@
 };
 
 int gsmd_timer_init(void);
-void gmsd_timer_check_n_run(void);
-
+void gsmd_timer_check_n_run(void);
 struct gsmd_timer *gsmd_timer_alloc(void);
 int gsmd_timer_register(struct gsmd_timer *timer);
 void gsmd_timer_unregister(struct gsmd_timer *timer);

Modified: trunk/src/target/gsm/include/gsmd/machineplugin.h
===================================================================
--- trunk/src/target/gsm/include/gsmd/machineplugin.h	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/include/gsmd/machineplugin.h	2008-01-10 05:14:53 UTC (rev 3807)
@@ -10,13 +10,15 @@
 
 struct gsmd_machine_plugin {
 	struct llist_head list;
-	unsigned char *name;
+	char *name;
 	int (*power)(struct gsmd *g, int power);
 	int (*ex_submit)(struct gsmd *g);
 	int (*detect)(struct gsmd *g);
 	int (*init)(struct gsmd *g, int fd);
 };
 
+extern int gsmd_machine_plugin_init(struct gsmd *g,
+		char *machine_name, char *vendor_name);
 extern int gsmd_machine_plugin_register(struct gsmd_machine_plugin *pl);
 extern void gsmd_machine_plugin_unregister(struct gsmd_machine_plugin *pl);
 extern int gsmd_machine_plugin_find(struct gsmd *g);

Modified: trunk/src/target/gsm/include/gsmd/sms.h
===================================================================
--- trunk/src/target/gsm/include/gsmd/sms.h	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/include/gsmd/sms.h	2008-01-10 05:14:53 UTC (rev 3807)
@@ -18,7 +18,8 @@
 
 int usock_rcv_sms(struct gsmd_user *gu, struct gsmd_msg_hdr *gph, int len);
 int usock_rcv_cb(struct gsmd_user *gu, struct gsmd_msg_hdr *gph, int len);
-
+int sms_pdu_decode_dcs(struct gsmd_sms_datacodingscheme *dcs,
+		const u_int8_t *data);
 #endif /* __GSMD__ */
 
 #endif

Modified: trunk/src/target/gsm/include/gsmd/unsolicited.h
===================================================================
--- trunk/src/target/gsm/include/gsmd/unsolicited.h	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/include/gsmd/unsolicited.h	2008-01-10 05:14:53 UTC (rev 3807)
@@ -7,14 +7,18 @@
 
 struct gsmd_unsolicit {
 	const char *prefix;
-	int (*parse)(char *unsol, int len, const char *param, struct gsmd *gsmd);
+	int (*parse)(const char *unsol,
+			int len, const char *param, struct gsmd *gsmd);
 };
 
-extern int unsolicited_parse(struct gsmd *g, char *buf, int len, const char *param);
+extern void unsolicited_init(struct gsmd *g);
+extern int unsolicited_parse(struct gsmd *g,
+		const char *buf, int len, const char *param);
 extern int generate_event_from_cme(struct gsmd *g, unsigned int cme_error);
 extern void unsolicited_generic_init(struct gsmd *g);
 extern int unsolicited_register_array(const struct gsmd_unsolicit *arr,
 		int len);
+extern int generate_event_from_cms(struct gsmd *g, unsigned int cms_error);
 extern const int pintype_from_cme[];
 
 #endif /* __GSMD__ */

Modified: trunk/src/target/gsm/include/gsmd/usock.h
===================================================================
--- trunk/src/target/gsm/include/gsmd/usock.h	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/include/gsmd/usock.h	2008-01-10 05:14:53 UTC (rev 3807)
@@ -741,7 +741,9 @@
 extern int usock_evt_send(struct gsmd *gsmd, struct gsmd_ucmd *ucmd, u_int32_t evt);
 extern int gsmd_ucmd_submit(struct gsmd_user *gu, u_int8_t msg_type,
 		u_int8_t msg_subtype, u_int16_t id, int len, const void *data);
-
+extern int gsmd_opname_init(struct gsmd *g);
+extern int gsmd_opname_add(struct gsmd *g, const char *numeric_bcd_string,
+		const char *alnum_long);
 #endif /* __GSMD__ */
 
 #endif

Modified: trunk/src/target/gsm/include/gsmd/vendorplugin.h
===================================================================
--- trunk/src/target/gsm/include/gsmd/vendorplugin.h	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/include/gsmd/vendorplugin.h	2008-01-10 05:14:53 UTC (rev 3807)
@@ -11,14 +11,15 @@
 
 struct gsmd_vendor_plugin {
 	struct llist_head list;
-	unsigned char *name;
-	unsigned char *ext_chars;
+	char *name;
+	char *ext_chars;
 	unsigned int num_unsolicit;
 	const struct gsmd_unsolicit *unsolicit;
 	int (*detect)(struct gsmd *g);
 	int (*initsettings)(struct gsmd *g);
 };
 
+extern int gsmd_vendor_plugin_load(char *name);
 extern int gsmd_vendor_plugin_register(struct gsmd_vendor_plugin *pl);
 extern void gsmd_vendor_plugin_unregister(struct gsmd_vendor_plugin *pl);
 extern int gsmd_vendor_plugin_find(struct gsmd *g);

Modified: trunk/src/target/gsm/include/libgsmd/misc.h
===================================================================
--- trunk/src/target/gsm/include/libgsmd/misc.h	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/include/libgsmd/misc.h	2008-01-10 05:14:53 UTC (rev 3807)
@@ -71,5 +71,8 @@
 
 /* Retrieve IMSI information */
 extern int lgsm_get_imsi(struct lgsm_handle *lh);
-
+/* Set voice mail number */
+extern int lgsm_voicemail_set(struct lgsm_handle *lh, const char *number);
+/* Get currently configured voice mail number */
+extern int lgsm_voicemail_get(struct lgsm_handle *lh);
 #endif

Modified: trunk/src/target/gsm/src/gsmd/atcmd.c
===================================================================
--- trunk/src/target/gsm/src/gsmd/atcmd.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/gsmd/atcmd.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -49,12 +49,14 @@
 	NUM_FINAL_RESULTS,
 };
 
+#if 0
 static const char *final_results[] = {
 	"OK",
 	"ERROR",
 	"+CME ERROR:",
 	"+CMS ERROR:",
 };
+#endif
 
 /* we basically implement a parse that can deal with
  * - receiving and queueing commands from higher level of libgmsd
@@ -188,6 +190,7 @@
 	return 0;
 }
 
+#if 0
 /* mid-level parser */
 
 static int parse_final_result(const char *res)
@@ -200,6 +203,7 @@
 	
 	return -1;
 }
+#endif
 
 void atcmd_wake_pending_queue (struct gsmd *g) 
 {
@@ -224,10 +228,10 @@
                 /* send final result code if there is no information
                 * response in mlbuf */
                 if (g->mlbuf_len) {
-                        cmd->resp = g->mlbuf;
+                        cmd->resp = (char *) g->mlbuf;
                         g->mlbuf[g->mlbuf_len] = 0;
                 } else {
-                        cmd->resp = buf;
+                        cmd->resp = (char *) buf;
                 }
                 rc = cmd->cb(cmd, cmd->ctx, cmd->resp);
                 DEBUGP("Clearing mlbuf\n");
@@ -367,7 +371,7 @@
 					gsmd_log(GSMD_NOTICE, "command without cb!!!\n");
 				} else {
 					DEBUGP("Calling cmd->cb()\n");
-					cmd->resp = g->mlbuf;
+					cmd->resp = (char *) g->mlbuf;
 					rc = cmd->cb(cmd, cmd->ctx, cmd->resp);
 					DEBUGP("Clearing mlbuf\n");
 					memset(g->mlbuf, 0, MLPARSE_BUF_SIZE);
@@ -429,8 +433,8 @@
 	g->mlbuf_len += len;
 
 	if (g->mlunsolicited) {
-		rc = unsolicited_parse(g, g->mlbuf, g->mlbuf_len,
-				strchr(g->mlbuf, ':') + 1);
+		rc = unsolicited_parse(g, (const char*) g->mlbuf, (int) g->mlbuf_len,
+				strchr((const char*)g->mlbuf, ':') + 1);
 		if (rc == -EAGAIN) {
 			/* The parser wants one more line of
 			 * input.  Wait for the next line, concatenate
@@ -464,6 +468,7 @@
 	struct gsmd *g = data;
 
         atcmd_wake_pending_queue(g);
+	return 0;
 }
 
 /* callback to be called if [virtual] UART has some data for us */
@@ -642,7 +647,6 @@
 /* submit an atcmd in the global queue of pending atcmds */
 int atcmd_submit(struct gsmd *g, struct gsmd_atcmd *cmd)
 {
-        int empty;
 
 	if (g->machinepl->ex_submit) {
 		DEBUGP("extra-submiting command\n");

Modified: trunk/src/target/gsm/src/gsmd/ext_response.c
===================================================================
--- trunk/src/target/gsm/src/gsmd/ext_response.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/gsmd/ext_response.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -67,7 +67,6 @@
 {
 	const char *cur = input;
 	struct gsm_extrsp *er;
-	int cur_tok = 0;
 	enum parser_state state = IDLE;
 	char buf[512];
 	char *cur_buf = buf;
@@ -166,6 +165,8 @@
 				/* ERROR */
 			}
 			break;
+		default:
+			break;
 		}
 		cur++;
 	}

Modified: trunk/src/target/gsm/src/gsmd/gsmd.c
===================================================================
--- trunk/src/target/gsm/src/gsmd/gsmd.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/gsmd/gsmd.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -44,6 +44,8 @@
 #include <gsmd/usock.h>
 #include <gsmd/vendorplugin.h>
 #include <gsmd/talloc.h>
+#include <gsmd/sms.h>
+#include <gsmd/unsolicited.h>
 
 #define GSMD_ALIVECMD		"AT"
 #define GSMD_ALIVE_INTERVAL	5*60
@@ -157,7 +159,7 @@
 
 static int gsmd_initsettings2(struct gsmd *gsmd)
 {
-	int rc;
+	int rc = 0;
 	
 	/* echo on, verbose */
 	rc |= gsmd_simplecmd(gsmd, "ATE0V1");
@@ -215,19 +217,10 @@
 	return gsmd_initsettings2(gsmd);
 }
 
-static void firstcmd_tmr_cb(struct gsmd_timer *tmr, void *data)
-{
-	if (firstcmd_response == 0) {
-		gsmd_log(GSMD_FATAL, "No response from GSM Modem");
-		exit(4);
-	}
-	gsmd_timer_free(tmr);
-}
 
 int gsmd_initsettings(struct gsmd *gsmd)
 {
 	struct gsmd_atcmd *cmd;
-	struct timeval tv;
 
 	cmd = atcmd_fill("ATZ", strlen("ATZ")+1, &firstcmd_atcb, gsmd, 0, NULL);
 	if (!cmd)
@@ -366,7 +359,6 @@
 	int bps = 115200;
 	int hwflow = 0;
 	char *device = NULL;
-	char *logfile = "syslog";
 	char *vendor_name = NULL;
 	char *machine_name = NULL;
 	int wait = -1;
@@ -475,7 +467,7 @@
 		exit(1);
 	}
 
-        write(fd,'\r',1);
+        write(fd,"\r",1);
 	atcmd_drain(fd);
 
 	if (usock_init(&g) < 0) {

Modified: trunk/src/target/gsm/src/gsmd/machine.c
===================================================================
--- trunk/src/target/gsm/src/gsmd/machine.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/gsmd/machine.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -117,7 +117,7 @@
 	buf[sizeof(buf)-1] = '\0';
 
 	line = strtok(buf, "\n");
-	while (line = strtok(NULL, "\n")) {
+	while ((line = strtok(NULL, "\n"))) {
 		if (strncmp(line, "Hardware\t: ", 11) == 0) {
 			hw = line+11;
 			break;
@@ -138,7 +138,7 @@
 
 				if (vendor_name)
 					DEBUGP("wanring: auto-detected vendor '%s', "
-						"but user override to '%s'\m",
+						"but user override to '%s'\n",
 						machines[i].vendor, vendor_name);
 				else
 					vendor_name = machines[i].vendor;

Modified: trunk/src/target/gsm/src/gsmd/machine_generic.c
===================================================================
--- trunk/src/target/gsm/src/gsmd/machine_generic.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/gsmd/machine_generic.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -41,8 +41,6 @@
 
 static int generic_init(struct gsmd *g, int fd)
 {
-	int rc;
-
 	/*
 	 * We assume that the GSM chipset can take
 	 * input immediately, so we don't have to

Modified: trunk/src/target/gsm/src/gsmd/sms_cb.c
===================================================================
--- trunk/src/target/gsm/src/gsmd/sms_cb.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/gsmd/sms_cb.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -38,6 +38,7 @@
 #include <gsmd/usock.h>
 #include <gsmd/unsolicited.h>
 #include <gsmd/sms.h>
+#include <gsmd/talloc.h>
 
 static const char *ts0705_memtype_name[] = {
 	[GSM0705_MEMTYPE_NONE]		= "NONE",
@@ -402,7 +403,8 @@
 }
 
 /* Unsolicited messages related to SMS / CB */
-static int cmti_parse(char *buf, int len, const char *param, struct gsmd *gsmd)
+static int cmti_parse(const char *buf, int len, const char *param,
+		struct gsmd *gsmd)
 {
 	char memstr[3];
 	struct gsmd_evt_auxdata *aux;
@@ -424,7 +426,8 @@
 	return usock_evt_send(gsmd, ucmd, GSMD_EVT_IN_SMS);
 }
 
-static int cmt_parse(char *buf, int len, const char *param, struct gsmd *gsmd)
+static int cmt_parse(const char *buf, int len, const char *param,
+		struct gsmd *gsmd)
 {
 	/* TODO: TEXT mode */
 	u_int8_t pdu[SMS_MAX_PDU_SIZE];
@@ -474,7 +477,8 @@
 	return usock_evt_send(gsmd, ucmd, GSMD_EVT_IN_SMS);
 }
 
-static int cbmi_parse(char *buf, int len, const char *param, struct gsmd *gsmd)
+static int cbmi_parse(const char *buf, int len, const char *param,
+		struct gsmd *gsmd)
 {
 	char memstr[3];
 	struct gsmd_evt_auxdata *aux;
@@ -496,7 +500,8 @@
 	return usock_evt_send(gsmd, ucmd, GSMD_EVT_IN_CBM);
 }
 
-static int cbm_parse(char *buf, int len, const char *param, struct gsmd *gsmd)
+static int cbm_parse(const char *buf, int len, const char *param,
+		struct gsmd *gsmd)
 {
 	/* TODO: TEXT mode */
 	u_int8_t pdu[CBM_MAX_PDU_SIZE];
@@ -541,7 +546,8 @@
 	return usock_evt_send(gsmd, ucmd, GSMD_EVT_IN_CBM);
 }
 
-static int cdsi_parse(char *buf, int len, const char *param, struct gsmd *gsmd)
+static int cdsi_parse(const char *buf, int len, const char *param,
+		struct gsmd *gsmd)
 {
 	char memstr[3];
 	struct gsmd_evt_auxdata *aux;
@@ -563,7 +569,8 @@
 	return usock_evt_send(gsmd, ucmd, GSMD_EVT_IN_DS);
 }
 
-static int cds_parse(char *buf, int len, const char *param, struct gsmd *gsmd)
+static int cds_parse(const char *buf, int len, const char *param,
+		struct gsmd *gsmd)
 {
 	/* TODO: TEXT mode */
 	u_int8_t pdu[SMS_MAX_PDU_SIZE];

Modified: trunk/src/target/gsm/src/gsmd/timer.c
===================================================================
--- trunk/src/target/gsm/src/gsmd/timer.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/gsmd/timer.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -37,12 +37,13 @@
 static LLIST_HEAD(gsmd_timers);
 static void *__tmr_ctx;
 
+#if 0
 static void tv_normalize(struct timeval *out)
 {
 	out->tv_sec += (out->tv_usec / 1000000);
 	out->tv_usec = (out->tv_usec % 1000000);
 }
-
+#endif
 /* subtract two struct timevals */
 static int tv_sub(struct timeval *res, const struct timeval *from,
 		  const struct timeval *sub)
@@ -57,7 +58,7 @@
 
 	return 0;
 }
-
+#if 0
 static int tv_add(struct timeval *res, const struct timeval *a1,
 		  const struct timeval *a2)
 {
@@ -68,7 +69,7 @@
 
 	tv_normalize(res);
 }
-
+#endif
 static int tv_later(const struct timeval *expires, const struct timeval *now)
 {
 	if (expires->tv_sec < now->tv_sec)

Modified: trunk/src/target/gsm/src/gsmd/unsolicited.c
===================================================================
--- trunk/src/target/gsm/src/gsmd/unsolicited.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/gsmd/unsolicited.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -35,6 +35,7 @@
 #include <gsmd/ts0707.h>
 #include <gsmd/unsolicited.h>
 #include <gsmd/talloc.h>
+#include <gsmd/sms.h>
 
 struct gsmd_ucmd *usock_build_event(u_int8_t type, u_int8_t subtype, u_int16_t len)
 {
@@ -147,7 +148,7 @@
 				&state_ringing_timeout, gsmd);
 }
 
-static int ring_parse(char *buf, int len, const char *param,
+static int ring_parse(const char *buf, int len, const char *param,
 		      struct gsmd *gsmd)
 {
         struct gsmd_ucmd *ucmd;
@@ -167,7 +168,7 @@
 	return usock_evt_send(gsmd, ucmd, GSMD_EVT_IN_CALL);
 }
 
-static int cring_parse(char *buf, int len, const char *param, struct gsmd *gsmd)
+static int cring_parse(const char *buf, int len, const char *param, struct gsmd *gsmd)
 {
 	struct gsmd_ucmd *ucmd = usock_build_event(GSMD_MSG_EVENT, GSMD_EVT_IN_CALL,
 					     sizeof(struct gsmd_evt_auxdata));
@@ -201,7 +202,7 @@
 }
 
 /* Chapter 7.2, network registration */
-static int creg_parse(char *buf, int len, const char *param,
+static int creg_parse(const char *buf, int len, const char *param,
 		      struct gsmd *gsmd)
 {
 	const char *comma = strchr(param, ',');
@@ -250,7 +251,7 @@
 }
 
 /* Chapter 7.11, call waiting */
-static int ccwa_parse(char *buf, int len, const char *param,
+static int ccwa_parse(const char *buf, int len, const char *param,
 		      struct gsmd *gsmd)
 {
 	struct gsmd_evt_auxdata *aux;
@@ -309,7 +310,7 @@
 }
 
 /* Chapter 7.14, unstructured supplementary service data */
-static int cusd_parse(char *buf, int len, const char *param,
+static int cusd_parse(const char *buf, int len, const char *param,
 		      struct gsmd *gsmd)
 {
 	/* FIXME: parse */
@@ -317,7 +318,7 @@
 }
 
 /* Chapter 7.15, advise of charge */
-static int cccm_parse(char *buf, int len, const char *param,
+static int cccm_parse(const char *buf, int len, const char *param,
 		      struct gsmd *gsmd)
 {
 	/* FIXME: parse */
@@ -325,7 +326,7 @@
 }
 
 /* Chapter 10.1.13, GPRS event reporting */
-static int cgev_parse(char *buf, int len, const char *param,
+static int cgev_parse(const char *buf, int len, const char *param,
 		      struct gsmd *gsmd)
 {
 	/* FIXME: parse */
@@ -333,7 +334,7 @@
 }
 
 /* Chapter 10.1.14, GPRS network registration status */
-static int cgreg_parse(char *buf, int len, const char *param,
+static int cgreg_parse(const char *buf, int len, const char *param,
 		       struct gsmd *gsmd)
 {
 	/* FIXME: parse */
@@ -341,7 +342,7 @@
 }
 
 /* Chapter 7.6, calling line identification presentation */
-static int clip_parse(char *buf, int len, const char *param,
+static int clip_parse(const char *buf, int len, const char *param,
 		      struct gsmd *gsmd)
 {
 	struct gsmd_ucmd *ucmd = usock_build_event(GSMD_MSG_EVENT, GSMD_EVT_IN_CLIP,
@@ -369,7 +370,7 @@
 }
 
 /* Chapter 7.9, calling line identification presentation */
-static int colp_parse(char *buf, int len, const char *param,
+static int colp_parse(const char *buf, int len, const char *param,
 		      struct gsmd *gsmd)
 {
 	struct gsmd_ucmd *ucmd = usock_build_event(GSMD_MSG_EVENT, GSMD_EVT_OUT_COLP,
@@ -395,7 +396,7 @@
 	return usock_evt_send(gsmd, ucmd, GSMD_EVT_OUT_COLP);
 }
 
-static int ctzv_parse(char *buf, int len, const char *param,
+static int ctzv_parse(const char *buf, int len, const char *param,
 		      struct gsmd *gsmd)
 {
 	struct gsmd_ucmd *ucmd = usock_build_event(GSMD_MSG_EVENT, GSMD_EVT_TIMEZONE,
@@ -419,7 +420,7 @@
 	return usock_evt_send(gsmd, ucmd, GSMD_EVT_TIMEZONE);
 }
 
-static int copn_parse(char *buf, int len, const char *param,
+static int copn_parse(const char *buf, int len, const char *param,
 		      struct gsmd *gsmd)
 {
 	struct gsm_extrsp *er = extrsp_parse(gsmd_tallocs, param);
@@ -468,11 +469,10 @@
 static struct gsmd_unsolicit unsolicit[256] = {{ 0, 0 }};
 
 /* called by midlevel parser if a response seems unsolicited */
-int unsolicited_parse(struct gsmd *g, char *buf, int len, const char *param)
+int unsolicited_parse(struct gsmd *g, const char *buf, int len, const char *param)
 {
 	struct gsmd_unsolicit *i;
 	int rc;
-	struct gsmd_vendor_plugin *vpl = g->vendorpl;
 
 	/* call unsolicited code parser */
 	for (i = unsolicit; i->prefix; i ++) {

Modified: trunk/src/target/gsm/src/gsmd/usock.c
===================================================================
--- trunk/src/target/gsm/src/gsmd/usock.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/gsmd/usock.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -43,7 +43,7 @@
 #include <gsmd/ts0707.h>
 #include <gsmd/sms.h>
 
-static void *__ucmd_ctx, *__gu_ctx, *__pb_ctx;
+static void *__ucmd_ctx, *__gu_ctx;
 
 struct gsmd_ucmd *ucmd_alloc(int extra_size)
 {
@@ -112,6 +112,7 @@
 		return -EINVAL;
 
 	gu->subscriptions = *evtmask;
+	return 0;
 }
 
 static int voicecall_get_stat_cb(struct gsmd_atcmd *cmd, void *ctx, char *resp) 
@@ -222,7 +223,6 @@
 	struct gsmd_user *gu = ctx;
 	struct gsm_extrsp *er;
 	struct gsmd_call_fwd_stat gcfs;
-	int ret = 0;
 	
 	DEBUGP("resp: %s\n", resp);
 	
@@ -955,7 +955,7 @@
 	case GSMD_NETWORK_REGISTER:
 		if ((*oper)[0])
 			cmdlen = sprintf(buffer, "AT+COPS=1,2,\"%.*s\"",
-					sizeof(gsmd_oper_numeric), oper);
+					sizeof(gsmd_oper_numeric), (char *)oper);
 		else
 			cmdlen = sprintf(buffer, "AT+COPS=0");
 		cmd = atcmd_fill(buffer, cmdlen + 1, &null_cmd_cb, gu, 0, NULL);
@@ -996,7 +996,7 @@
 		break;
 	case GSMD_NETWORK_PREF_ADD:
 		cmdlen = sprintf(buffer, "AT+CPOL=,2,\"%.*s\"",
-				sizeof(gsmd_oper_numeric), oper);
+				sizeof(gsmd_oper_numeric), (char *)oper);
 		cmd = atcmd_fill(buffer, cmdlen + 1, &null_cmd_cb, gu, 0, NULL);
 		break;
 	case GSMD_NETWORK_PREF_SPACE:
@@ -1253,9 +1253,8 @@
 	struct gsmd_phonebook_readrg *gpr;
 	struct gsmd_phonebook *gp;
 	struct gsmd_phonebook_find *gpf;
-	struct gsmd_phonebooks *cur, *cur2;
-	int *index, *num;
-	int atcmd_len, i, ret;
+	int *index;
+	int atcmd_len;
 	char *storage;
 	char buf[1024];
 

Modified: trunk/src/target/gsm/src/gsmd/vendor_bcm.c
===================================================================
--- trunk/src/target/gsm/src/gsmd/vendor_bcm.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/gsmd/vendor_bcm.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -35,7 +35,7 @@
 #include <gsmd/vendorplugin.h>
 #include <gsmd/unsolicited.h>
 
-static int mrdy_parse(char *buf, int len, const char *param,
+static int mrdy_parse(const char *buf, int len, const char *param,
 		     struct gsmd *g)
 {
 	int status = atoi(param);
@@ -70,11 +70,9 @@
 	return 0;
 }
 
-static int mtsmenu_parse(char *buf, int len, const char *param,
+static int mtsmenu_parse(const char *buf, int len, const char *param,
 		     struct gsmd *g)
 {
-	char *tok, *param_tmp;
-	struct mtsmenu *mtsmenu;
 
 	DEBUGP("mtsmenu_parse param=`%s'\n", param);
 	return 0;
@@ -95,8 +93,7 @@
 
 static int bcm_initsettings(struct gsmd *g)
 {
-	int rc;
-	struct gsmd_atcmd *cmd;
+	int rc = 0;
 
 	/* bcm sometimes sends LFCR instead of CRLF (eg *MTSMENU message) */
 	g->llp.flags |= LGSM_ATCMD_F_LFCR;

Modified: trunk/src/target/gsm/src/gsmd/vendor_qc.c
===================================================================
--- trunk/src/target/gsm/src/gsmd/vendor_qc.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/gsmd/vendor_qc.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -36,10 +36,9 @@
 #include <gsmd/vendorplugin.h>
 #include <gsmd/unsolicited.h>
 
-static int htccsq_parse(char *buf, int len, const char *param,
+static int htccsq_parse(const char *buf, int len, const char *param,
 		     struct gsmd *gsmd)
 {
-	char *tok;
 	struct gsmd_evt_auxdata *aux;
 	struct gsmd_ucmd *ucmd = usock_build_event(GSMD_MSG_EVENT, GSMD_EVT_SIGNAL,
 					     sizeof(*aux));
@@ -63,13 +62,9 @@
 	usock_evt_send(gsmd, ucmd, GSMD_EVT_SIGNAL);
 
 	return 0;
-
-out_free_io:
-	free(ucmd);
-	return -EIO;
 }
 
-static int wcdma_parse(char *buf, int len, const char *param,
+static int wcdma_parse(const char *buf, int len, const char *param,
 		     struct gsmd *gsmd)
 {
 	return 0;
@@ -96,8 +91,7 @@
 
 static int qc_initsettings(struct gsmd *g)
 {
-	int rc;
-	struct gsmd_atcmd *cmd;
+	int rc = 0;
 
 	/* enable @HTCCSQ: signal quality reports */
 	rc |= gsmd_simplecmd(g, "AT at HTCCSQ=1");

Modified: trunk/src/target/gsm/src/gsmd/vendor_ti.c
===================================================================
--- trunk/src/target/gsm/src/gsmd/vendor_ti.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/gsmd/vendor_ti.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -38,7 +38,7 @@
 #include <gsmd/unsolicited.h>
 
 
-static int csq_parse(char *buf, int len, const char *param,
+static int csq_parse(const char *buf, int len, const char *param,
 		     struct gsmd *gsmd)
 {
 	struct gsmd_evt_auxdata *aux;
@@ -63,11 +63,13 @@
 	return -EIO;
 }
 
-static int cpri_parse(char *buf, int len, const char *param, struct gsmd *gsmd)
+static int cpri_parse(const char *buf, int len, const char *param, struct gsmd *gsmd)
 {
 	char *tok1, *tok2;
-
-	tok1 = strtok(buf, ",");
+	char tx_buf[20];
+	
+	strcpy(tx_buf, buf);
+	tok1 = strtok(tx_buf, ",");
 	if (!tok1)
 		return -EIO;
 	
@@ -103,7 +105,7 @@
 	return 0;
 }
 
-static int ctzv_parse(char *buf, int len, const char *param, struct gsmd *gsmd)
+static int ctzv_parse(const char *buf, int len, const char *param, struct gsmd *gsmd)
 {
 	/* FIXME: decide what to do with it.  send as event to clients? or keep
 	 * locally? Offer option to sync system RTC? */
@@ -111,14 +113,16 @@
 }
 
 /* Call Progress Information */
-static int cpi_parse(char *buf, int len, const char *param, struct gsmd *gsmd)
+static int cpi_parse(const char *buf, int len, const char *param, struct gsmd *gsmd)
 {
 	char *tok;
 	struct gsmd_evt_auxdata *aux;
 	struct gsmd_ucmd *ucmd = usock_build_event(GSMD_MSG_EVENT,
 						   GSMD_EVT_OUT_STATUS,
 						   sizeof(*aux));
-	
+	char tx_buf[64];
+
+	strcpy(tx_buf, buf);
 	DEBUGP("entering cpi_parse param=`%s'\n", param);
 	if (!ucmd)
 		return -EINVAL;
@@ -128,7 +132,7 @@
 	/* Format: cId, msgType, ibt, tch, dir,[mode],[number],[type],[alpha],[cause],line */
 
 	/* call ID */
-	tok = strtok(buf, ",");
+	tok = strtok(tx_buf, ",");
 	if (!tok)
 		goto out_free_io;
 

Modified: trunk/src/target/gsm/src/gsmd/vendor_tihtc.c
===================================================================
--- trunk/src/target/gsm/src/gsmd/vendor_tihtc.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/gsmd/vendor_tihtc.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -52,10 +52,9 @@
 	return atcmd_submit(gsmd, cmd);
 }
 
-static int htccsq_parse(char *buf, int len, const char *param,
+static int htccsq_parse(const char *buf, int len, const char *param,
 		     struct gsmd *gsmd)
 {
-	char *tok;
 	struct gsmd_evt_auxdata *aux;
 	struct gsmd_ucmd *ucmd = usock_build_event(GSMD_MSG_EVENT, GSMD_EVT_SIGNAL,
 					     sizeof(*aux));
@@ -79,17 +78,15 @@
 	usock_evt_send(gsmd, ucmd, GSMD_EVT_SIGNAL);
 
 	return 0;
-
-out_free_io:
-	free(ucmd);
-	return -EIO;
 }
 
-static int cpri_parse(char *buf, int len, const char *param, struct gsmd *gsmd)
+static int cpri_parse(const char *buf, int len, const char *param, struct gsmd *gsmd)
 {
 	char *tok1, *tok2;
-
-	tok1 = strtok(buf, ",");
+	char tx_buf[20];
+	
+	strcpy(tx_buf, buf);
+	tok1 = strtok(tx_buf, ",");
 	if (!tok1)
 		return -EIO;
 
@@ -126,14 +123,16 @@
 }
 
 /* Call Progress Information */
-static int cpi_parse(char *buf, int len, const char *param, struct gsmd *gsmd)
+static int cpi_parse(const char *buf, int len, const char *param, struct gsmd *gsmd)
 {
 	char *tok;
 	struct gsmd_evt_auxdata *aux;
 	struct gsmd_ucmd *ucmd = usock_build_event(GSMD_MSG_EVENT,
 						   GSMD_EVT_OUT_STATUS,
 						   sizeof(*aux));
+	char tx_buf[64];
 
+	strcpy(tx_buf, buf);
 	DEBUGP("entering cpi_parse param=`%s'\n", param);
 	if (!ucmd)
 		return -EINVAL;
@@ -143,7 +142,7 @@
 	/* Format: cId, msgType, ibt, tch, dir,[mode],[number],[type],[alpha],[cause],line */
 
 	/* call ID */
-	tok = strtok(buf, ",");
+	tok = strtok(tx_buf, ",");
 	if (!tok)
 		goto out_free_io;
 
@@ -248,7 +247,7 @@
 
 static int tihtc_initsettings(struct gsmd *g)
 {
-	int rc;
+	int rc = 0;
 	struct gsmd_atcmd *cmd;
 
 	/* use %CGREG */

Modified: trunk/src/target/gsm/src/libgsmd/libgsmd.c
===================================================================
--- trunk/src/target/gsm/src/libgsmd/libgsmd.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/libgsmd/libgsmd.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -32,9 +32,11 @@
 
 #include <gsmd/usock.h>
 #include <libgsmd/libgsmd.h>
+#include <libgsmd/event.h>
 
 #include "lgsm_internals.h"
 
+#if 0
 static int lgsm_get_packet(struct lgsm_handle *lh)
 {
 	static char buf[GSMD_MSGSIZE_MAX];
@@ -56,6 +58,7 @@
 
 	return 0;
 }
+#endif
 
 static int lgsm_open_backend(struct lgsm_handle *lh, const char *device)
 {

Modified: trunk/src/target/gsm/src/libgsmd/libgsmd_network.c
===================================================================
--- trunk/src/target/gsm/src/libgsmd/libgsmd_network.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/libgsmd/libgsmd_network.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -38,6 +38,7 @@
 		enum lgsm_netreg_state *state)
 {
 	*state = lh->netreg_state;
+	return 0;
 }
 
 int lgsm_oper_get(struct lgsm_handle *lh)
@@ -139,7 +140,6 @@
 {
 	struct gsmd_msg_hdr *gmh;
 	struct gsmd_voicemail *vmail;
-	int rc;
 
 	gmh = lgsm_gmh_fill(GSMD_MSG_NETWORK,
 			GSMD_NETWORK_VMAIL_SET, sizeof(*vmail));

Modified: trunk/src/target/gsm/src/libgsmd/libgsmd_sms.c
===================================================================
--- trunk/src/target/gsm/src/libgsmd/libgsmd_sms.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/libgsmd/libgsmd_sms.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -135,7 +135,7 @@
 	/* FIXME: only support PDU mode */
 	struct gsmd_msg_hdr *gmh;
 	struct gsmd_sms_submit *gss;
-	int i, rc;
+	int rc;
 
 	gmh = lgsm_gmh_fill(GSMD_MSG_SMS,
 			GSMD_SMS_SEND, sizeof(*gss));
@@ -258,7 +258,6 @@
 {
 	int i,j = 0;
 	unsigned char ch1, ch2;
-	char tmp[2];
 	int shift = 0;
 
 	dest->alpha = ALPHABET_DEFAULT;

Modified: trunk/src/target/gsm/src/util/atcmd.c
===================================================================
--- trunk/src/target/gsm/src/util/atcmd.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/util/atcmd.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -35,6 +35,7 @@
 {
 	char *payload = (char *)gmh + sizeof(*gmh);
 	printf("RSTR=`%s'\n", payload);
+	return 0;
 }
 
 int atcmd_main(struct lgsm_handle *lgsmh)
@@ -42,7 +43,7 @@
 	int rc;
 	char buf[STDIN_BUF_SIZE+1];
 	char rbuf[STDIN_BUF_SIZE+1];
-	int rlen = sizeof(rbuf);
+	unsigned int rlen = sizeof(rbuf);
 	fd_set readset;
 
 	lgsm_register_handler(lgsmh, GSMD_MSG_PASSTHROUGH, &pt_msghandler);
@@ -99,4 +100,5 @@
 			fflush(stdout);
 		}
 	}
+	return 0;
 }

Modified: trunk/src/target/gsm/src/util/event.c
===================================================================
--- trunk/src/target/gsm/src/util/event.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/util/event.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -115,7 +115,7 @@
 				msg->page, msg->pages);
 
 		if (msg->coding_scheme == ALPHABET_DEFAULT) {
-			cbm_unpacking_7bit_character(msg->data, payload);
+			cbm_unpacking_7bit_character((char *)msg->data, payload);
 			printf("\"%s\"\n", payload);
 		} else if (msg->coding_scheme == ALPHABET_8BIT)
 			printf("8-bit encoded data\n");
@@ -176,6 +176,8 @@
 	case GSMD_NETREG_REG_ROAMING:
 		printf("registered (roaming) ");
 		break;
+	default:
+		break;
 	}
 
 	if (aux->u.netreg.lac)

Modified: trunk/src/target/gsm/src/util/pin.c
===================================================================
--- trunk/src/target/gsm/src/util/pin.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/util/pin.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -52,7 +52,7 @@
 	} else {
 		do {
 			printf("Please enter %s: ", lgsm_pin_name(type));
-			rc = fscanf(stdin, "%8s", &pinbuf);
+			rc = fscanf(stdin, "%8s", pinbuf);
 		} while (rc < 1);
 
 		switch (type) {
@@ -60,7 +60,7 @@
 		case GSMD_PIN_SIM_PUK2:
 			do {
 				printf("Please enter new PIN: ");
-				rc = fscanf(stdin, "%8s", &pinbuf2);
+				rc = fscanf(stdin, "%8s", pinbuf2);
 				newpin = pinbuf2;
 			} while (rc < 1);
 			break;

Modified: trunk/src/target/gsm/src/util/shell.c
===================================================================
--- trunk/src/target/gsm/src/util/shell.c	2008-01-09 23:05:16 UTC (rev 3806)
+++ trunk/src/target/gsm/src/util/shell.c	2008-01-10 05:14:53 UTC (rev 3807)
@@ -31,6 +31,7 @@
 #include <libgsmd/misc.h>
 #include <libgsmd/phonebook.h>
 #include <libgsmd/sms.h>
+#include <libgsmd/pin.h>
 #include <gsmd/usock.h>
 #include <gsmd/ts0705.h>
 
@@ -49,6 +50,7 @@
 {
 	char *payload = (char *)gmh + sizeof(*gmh);
 	printf("RSTR=`%s'\n", payload);
+	return 0;
 }
 
 /* this is the handler for receiving phonebook responses */
@@ -135,7 +137,8 @@
 		break;
 	default:
 		return -EINVAL;
-	}	
+	}
+	return 0;
 }
 
 /* this is the handler for receiving sms responses */
@@ -273,6 +276,7 @@
 		pending_responses --;
 		return -EINVAL;
 	}
+	return 0;
 }
 
 /* this is the handler for responses to network/operator commands */
@@ -359,6 +363,7 @@
 	default:
 		return -EINVAL;
 	}
+	return 0;
 }
 
 static int phone_msghandler(struct lgsm_handle *lh, struct gsmd_msg_hdr *gmh)
@@ -482,7 +487,7 @@
 	{ 0, 0 }
 };
 
-static int shell_help(void)
+static void shell_help(void)
 {
 	printf( "\tA\tAnswer incoming call\n"
 		"\tD\tDial outgoing number\n"
@@ -542,8 +547,6 @@
 {
 	int rc;
 	char buf[STDIN_BUF_SIZE+1];
-	char rbuf[STDIN_BUF_SIZE+1];
-	int rlen = sizeof(rbuf);
 	fd_set readset;
 	char *ptr, *fcomma, *lcomma;
 	int gsm_fd = lgsm_fd(lgsmh);
@@ -923,4 +926,5 @@
 		}
 		fflush(stdout);
 	}
+	return 0;
 }





More information about the commitlog mailing list