RE: memory check error

"Oded Arbel" <[email protected]>
Newsgroups gmane.comp.mobile.kannel.devel
Message-ID <[email protected]>
First thing - the log lines should show the module "name" and not the
device file (totally my fault, sorry), and here's a patch for that.
 
Now, as to the error message, the module is correct in complaining -
AT+CPMS? shouldn't return OK, but a description of the memory locations
available. what kind of modem do you actually have ? and if it's a
wavecom, what model and firmware version ?
 
--
Oded Arbel
m-Wise Inc.
[email protected]
(972)-67-340014
(972)-9-9581711 (ext: 116)
 
::..
Joe Friday: "Ah, sure, but just like every other foaming, rabid psycho
in this city with a foolproof plan, you've forgotten you're facing the
single finest fighting force ever assembled."
Whirley: "The Israelis?"
 -- from 'Dragnet' (1987)

 



-----Original Message-----
From: amir guzman [mailto:[email protected]]
Sent: Wednesday, May 08, 2002 10:51 AM
To: [email protected]
Subject: memory check error


Hi,
 
2002-05-08 08:46:09 [6] DEBUG: AT2[A]: --> AT^M
2002-05-08 08:46:09 [6] DEBUG: AT2[A]: <-- OK
2002-05-08 08:46:09 [6] DEBUG: AT2[A]: --> AT+CPMS?^M
2002-05-08 08:46:09 [6] DEBUG: AT2[A]: <-- OK
2002-05-08 08:46:09 [6] DEBUG: AT2[/dev/ttySI2]: no correct header for
CPMS response.
2002-05-08 08:46:09 [6] DEBUG: AT2[/dev/ttySI2]: memory check error
2002-05-08 08:46:13 [6] DEBUG: AT2[A]: <-- OK
 
group = smsc
smsc = at2
smsc-id = smartA
denied-smsc-id = globeA
modemtype = wavecom
device = /dev/ttySI2
retry = true
keepalive = 20
sim-buffering = true
validityperiod = 15
speed = 9600

group = modems
id = wavecom
name = Wavecom
detect-string = "WAVECOM"
init-string = "AT+CNMI=1,2,0,0,0"
no-pin = true
speed=9600

 

  _____  

MSN Photos is the easiest way to share and print your photos: Click
<http://g.msn.com/1HM100901/157> Here
smsc_at2.patch (application/octet-stream, 2.7 KB)
--- gateway/gw/smsc_at2.c	2002-04-15 11:39:00.000000000 +0300
+++ gateway/gw/smsc_at2.c	2002-05-08 12:42:26.000000000 +0300
@@ -595,7 +595,7 @@
 
     /* get memory status */
     if (at2_check_sms_memory(privdata) == -1) {
-        debug("bb.smsc.at2", 0, "AT2[%s]: memory check error", octstr_get_cstr(privdata->device));
+        debug("bb.smsc.at2", 0, "AT2[%s]: memory check error", octstr_get_cstr(privdata->name));
         return ;
     }
 
@@ -629,7 +629,7 @@
         int message_count = 0; /* cound number of messages collected */
 
         debug("bb.smsc.at2", 0, "AT2[%s]: %d messages waiting in memory", 
-              octstr_get_cstr(privdata->device), privdata->sms_memory_usage);
+              octstr_get_cstr(privdata->name), privdata->sms_memory_usage);
 
         /*
          * loop till end of memory or collected enouch messages
@@ -642,14 +642,14 @@
             at2_write_line(privdata, cmd);
             if (at2_wait_modem_command(privdata, 0, 0, &message_count) != 0) {
                 debug("bb.smsc.at2", 0, "AT2[%s]: failed to get message %d.", 
-                      octstr_get_cstr(privdata->device), i);
+                      octstr_get_cstr(privdata->name), i);
                 continue; /* failed to read the message - skip to next message */
             }
 
             /* no need to delete if no message collected */
             if (old_message_count == message_count) { 
                 debug("bb.smsc.at2", 0, "AT2[%s]: not deleted.", 
-                      octstr_get_cstr(privdata->device));
+                      octstr_get_cstr(privdata->name));
                 continue;
             }
 
@@ -665,7 +665,7 @@
                  * we'll see it next time around. 
                  */                
                 debug("bb.smsc.at2", 0, "AT2[%s]: failed to delete message %d.", 
-                      octstr_get_cstr(privdata->device), i);
+                      octstr_get_cstr(privdata->name), i);
                 continue; 
 
             }
@@ -713,7 +713,7 @@
         if (index < 4) { 
             /* didn't get all memory data - I don't why, so I'll bail */
             debug("bb.smsc.at2", 0, "AT2[%s]: couldn't parse all memory locations : %d:'%s'.",
-                  octstr_get_cstr(privdata->device), index, 
+                  octstr_get_cstr(privdata->name), index,
                   &(octstr_get_cstr(privdata->lines)[pos]));
             O_DESTROY(search_cpms);
             return -1;
@@ -734,7 +734,7 @@
 
     } else {
         debug("bb.smsc.at2", 0, "AT2[%s]: no correct header for CPMS response.", 
-              octstr_get_cstr(privdata->device));
+              octstr_get_cstr(privdata->name));
 
         /* didn't get a +CPMS response - this is clearly an error */
         ret = -1;
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.