summaryrefslogtreecommitdiff
path: root/include
diff options
context:
space:
mode:
authorlaforge <laforge@99fdad57-331a-0410-800a-d7fa5415bdb3>2007-08-17 08:30:42 +0000
committerlaforge <laforge@99fdad57-331a-0410-800a-d7fa5415bdb3>2007-08-17 08:30:42 +0000
commit90bf3b4473da30d8b89c8ecfabcce42c0e4fe484 (patch)
tree8b1201b28ee581e6c550ddd8a4019e0d2b7fa65c /include
parent4e578aa3ef5ec658a2ff7e8417fdbfb9d61c15f1 (diff)
From: Andrzej Zaborowski <balrog@zabor.org>
Date: Fri, 27 Jul 2007 14:29:44 +0200 Subject: [PATCH] Multiline unsolicited responses parsing. +CMT, +CBM and +CDS responses two lines long in PDU mode and the parser has problem with them. As it was mentioned earlier there's no way for the parser to know if a new line is part of the most recent command response or unsolicited response, or a completely new unsolicited response, without looking at the contents of the first line. My idea is that the parser for the particular response decides if the response is too short, and returns -EAGAIN in which case the AT parser will append the following line to the response and resubmit it to the response parser. The disadvantage is that the generic parser gets one more special case to remember about, but it works quite well, hopefully there are no more cases where we will need to modify the parser. (We may need to use a separate multiline buffer (like the current mlbuf variable) to account for situations when a multiline unsolicited response comes in the middle of receiving a different response, which is also possible. Also, mlbuf should probably be moved into the parser state struct instead of being a global). This patch also moves initialisation of SMS options to after we are registered to the network, so that we don't get errors from the modem. Perhaps we want to have a dynamic list of callbacks to call on succesfull registration. Among other changes also prevents setting dev_state.on if the radio powering-on fails. --- include/gsmd/event.h | 8 +++++ include/gsmd/sms.h | 1 + src/gsmd/atcmd.c | 26 +++++++++++++++++ src/gsmd/sms_cb.c | 73 ++++++++++++++++++++++++------------------------ src/gsmd/unsolicited.c | 8 +++++ src/gsmd/usock.c | 21 ++++++++++++-- src/util/event.c | 10 +++---- 7 files changed, 103 insertions(+), 44 deletions(-) git-svn-id: http://svn.openmoko.org/trunk/src/target/gsm@2722 99fdad57-331a-0410-800a-d7fa5415bdb3
Diffstat (limited to 'include')
-rw-r--r--include/gsmd/event.h8
-rw-r--r--include/gsmd/sms.h1
2 files changed, 8 insertions, 1 deletions
diff --git a/include/gsmd/event.h b/include/gsmd/event.h
index 395c488..829b0fa 100644
--- a/include/gsmd/event.h
+++ b/include/gsmd/event.h
@@ -53,8 +53,14 @@ enum gsmd_call_type {
__NUM_GSMD_CALL
};
+/* Chapter 7.2 */
enum gsmd_netreg_state {
- GSMD_NETREG_NONE = 0,
+ GSMD_NETREG_UNREG = 0,
+ GSMD_NETREG_REG_HOME,
+ GSMD_NETREG_UNREG_BUSY,
+ GSMD_NETREG_DENIED,
+ GSMD_NETREG_UNKNOWN,
+ GSMD_NETREG_REG_ROAMING,
__NUM_GSMD_NETREG
};
diff --git a/include/gsmd/sms.h b/include/gsmd/sms.h
index f4def25..b48ce5d 100644
--- a/include/gsmd/sms.h
+++ b/include/gsmd/sms.h
@@ -6,6 +6,7 @@
#include <gsmd/gsmd.h>
int sms_cb_init(struct gsmd *gsmd);
+int sms_cb_network_init(struct gsmd *gsmd);
#define MAX_PDU_SIZE 180
int sms_pdu_make_smssubmit(char *dest, const struct gsmd_sms_submit *src);
personal git repositories of Harald Welte. Your mileage may vary