Re: I minor error found in gw/smsc_at2.c
Andreas Fink <[email protected]>
| Newsgroups | gmane.comp.mobile.kannel.devel |
|---|---|
| Message-ID | <p05101203b898a28a7fa9@[10.0.0.2]> |
>"Nektarios K. Papadopoulos" wrote: >> >> I am using smsc_at with a NOKIA Premicell. >> >> Thoughth to give a try on smsc_at2. What I was getting was something >> like: >> >> 2002-02-20 01:20:24 [6] DEBUG: AT2[/dev/ttyS1]: <-- +CSMS: (0) >> 2002-02-20 01:20:24 [6] DEBUG: AT2[/dev/ttyS1]: <-- OK >> 2002-02-20 01:20:24 [6] INFO: AT2[/dev/ttyS1]: Phase 2+ is supported >> 2002-02-20 01:20:24 [6] DEBUG: AT2[/dev/ttyS1]: --> AT+CSMS=1^M >> 2002-02-20 01:20:24 [6] DEBUG: AT2[/dev/ttyS1]: <-- ERROR >> >> I looke my AT command reference and the code and think that I should not >> get >> >> Phase 2+ is supported >> >> I think that passing octstr_case_compare to list_search, makes it to >> search for at least one non matching list element, insteat for one >> matching. >> So I suggest changing it to octstr_item_match, since we care only for >> digits. >> >> the diff with todays cvs is attached, in case I am not wrong. > >any votes from the others, please?! > >Putting this at least to STATUS for now. yes this is a bug. +CSMS(0) should return Phase 2+ is not supported. the nokia premicel is probably the only device left on this planet which doesnt support phase 2+. -- Andreas Fink Fink-Consulting ------------------------------------------------------------------ Tel: +41-61-6932730 Fax: +41-61-6932729 Mobile: +41-79-2457333 Address: A. Fink, Schwarzwaldallee 16, 4058 Basel, Switzerland E-Mail: [email protected] Homepage: http://www.finkconsulting.com ------------------------------------------------------------------ Something urgent? Try http://www.smsrelay.com/ Nickname afink