Open Source Telephony
 help / color / mirror / Atom feed
From: Marcel Holtmann <marcel@holtmann.org>
To: ofono@ofono.org
Subject: RE: [PATCH 4/4] atmodem: Add support for the PIN counter command AT+CPNNUM
Date: Thu, 04 Aug 2011 17:44:02 +0200	[thread overview]
Message-ID: <1312472643.3373.35.camel@aeonflux> (raw)
In-Reply-To: <568DE01DA537804BA73470071BAC288939006D29@irsmsx503.ger.corp.intel.com>

[-- Attachment #1: Type: text/plain, Size: 779 bytes --]

Hi Bertrand,

> > >  drivers/atmodem/sim.c |   95
> > +++++++++++++++++++++++++++++++++++++++++++++++++
> > >  1 files changed, 95 insertions(+), 0 deletions(-)
> > 
> > please include an OFONO_AT_DEBUG trace of this AT command.
> 
> I have attached a full log from a SpeedUp dongle, but here is the AT command that we are looking at:
> 
> ofonod[2534]: Aux: > AT+CPNNUM\r
> ofonod[2534]: Aux: < \r\nPIN1=3; PUK1=10; PIN2=3; PUK2=10\r\r\nOK\r\n

this is so ugly. I sometimes really question the manufactures on why
nobody can create proper AT commands. Also one fun fact, if you issue
this one on the wrong TTY, you get the answer on the other one. That
will screw up our parse big time ;)

Nevertheless, I added a patch for this now.

Regards

Marcel



  reply	other threads:[~2011-08-04 15:44 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-22 16:15 [PATCH 0/4] ZTE/SpeedUp specificities Philippe Nunes
2011-07-22 16:15 ` [PATCH 1/4] atmodem: Add vendor entry for SpeedUp modems Philippe Nunes
2011-07-27 14:13   ` Marcel Holtmann
2011-07-22 16:15 ` [PATCH 2/4] speedup: Create SIM atom with SpeedUp vendor to handle proprietary commands Philippe Nunes
2011-07-27 14:15   ` Marcel Holtmann
2011-07-22 16:15 ` [PATCH 3/4] atmodem: Add a polling based on CPIN query to detect SIM state change Philippe Nunes
2011-07-27 14:19   ` Marcel Holtmann
2011-07-22 16:15 ` [PATCH 4/4] atmodem: Add support for the PIN counter command AT+CPNNUM Philippe Nunes
2011-07-22 14:05   ` Denis Kenzior
2011-07-27 14:17   ` Marcel Holtmann
2011-08-04  7:34     ` Aygon, Bertrand
2011-08-04 15:44       ` Marcel Holtmann [this message]
2011-08-04 15:56         ` Aygon, Bertrand

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1312472643.3373.35.camel@aeonflux \
    --to=marcel@holtmann.org \
    --cc=ofono@ofono.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox