Re: Digicom Pocket GSM serial modem - PART III: Last famous words

[Daniele Forsi]

R.C. wrote:

> I have then compiled smsd but the effects of the patch
> are quite strange.

> Message sent: 0x00 / 0x0017
> 41 54 2b 43 50 4d 53 3d 22 53 4d 22 2c 22 53 4d | AT+CPMS="SM","SM
> 22 0d 00 00 b4 53 de                            | "    S
> write: [AT+CPMS="SM","SM"<cr>address@hidden@^ôS^]

there is traling garbage

> read : [AT+CPMS="SM","SM"<cr><cr><lf>+CPMS: 
> 3,10,3,10<cr><lf><cr><lf>OK<cr><lf>]
> Message received: 0x00 / 0x002c
> 02 41 54 2b 43 50 4d 53 3d 22 53 4d 22 2c 22 53 |  AT+CPMS="SM","S
> 4d 22 0d 0d 0a 2b 43 50 4d 53 3a 20 33 2c 31 30 | M"   +CPMS: 3,10
> 2c 33 2c 31 30 0d 0a 0d 0a 4f 4b 0d             | ,3,10    OK
> Received message type 00
> Message sent: 0x12 / 0x0009
> 41 54 2b 43 50 4d 53 3f 0d                      | AT+CPMS?
> write: [AT+CPMS?<cr>]
> read : address@hidden@^ôS^AT+CPMS?<cr><cr><lf>+CPMS: 
> "SM",3,10,"SM",3,10<cr><lf><cr><lf>OK<cr><lf>]

here that garbage is echoed back by the phone and the driver misses the 
AT+CPMS part and reports an error

> Message received: 0x12 / 0x0032
> 02 00 00 b4 53 de 41 54 2b 43 50 4d 53 3f 0d 0d |     S AT+CPMS?
> 0a 2b 43 50 4d 53 3a 20 22 53 4d 22 2c 33 2c 31 |  +CPMS: "SM",3,1
> 30 2c 22 53 4d 22 2c 33 2c 31 30 0d 0a 0d 0a 4f | 0,"SM",3,10    O
> 4b 0d                                           | K
> Received message type 12
> GN_OP_GetSMSStatus at line 400 in file lowlevel.c returns error 1

check the format string you changed, there must be only two %s directives:

sprintf(req, "AT+CPMS=\"%s\",\"%s\"\r", memorynames[mt], memorynames[mt]);
--
Daniele