From: Marcel Holtmann <marcel@holtmann.org>
To: ofono@ofono.org
Subject: Re: [RFC sim-ready-v1 PATCH 0/3] ofono_sim_ready_notify
Date: Tue, 19 Oct 2010 11:43:35 +0200 [thread overview]
Message-ID: <1287481415.3316.138.camel@aeonflux> (raw)
In-Reply-To: <1287416435-4307-1-git-send-email-Pekka.Pessi@nokia.com>
[-- Attachment #1: Type: text/plain, Size: 1393 bytes --]
Hi Pekka,
> Here is my 2nd stab at the ofono_sim_ready_notify().
>
> Main difference from Denis's and Kristen's patch is that the modem
> driver now has a way to indicate that is ready or not: if SIM is not
> ready, the ofono_sim_driver indicates that with query_passwd_state
> callback returning OFONO_SIM_PASSWORD_INVALID. A separate enum and a
> string value for PinRequired property for that case would probably be
> nice, however, it is not very clear to me how the modems handle the
> transition from PIN-locked SIM to SIM-locked modem.
>
> After the driver determines that the SIM is ready, it calls
> ofono_sim_ready_notify().
>
> The patches has now been tested with the PIN and PUK codes with mbm and
> the new isimodem sim driver by Jukka Saunamaki. IFX seems to be
> identical in PIN and PUK cases, too.
>
> The isimodem driver demonstrates also the case where PIN is disabled but
> SIM is not ready yet when oFono tries to initialize the modem. The
> isimodem driver tries to read some CPHS information in order to
> determine if the modem is ready. I'm not sure how that kind of probing
> should be implemented in atmodem driver.
so how do we handle AT command based modems that have no extra
indication of SIM ready state? Are we stuck with quirks in the atmodem
SIM driver or can we push these into the modem plugins?
Regards
Marcel
next prev parent reply other threads:[~2010-10-19 9:43 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-18 15:40 [RFC sim-ready-v1 PATCH 0/3] ofono_sim_ready_notify Pekka.Pessi
2010-10-18 15:40 ` [RFC sim-ready-v1 PATCH 1/3] sim: add ofono_sim_ready_notify Pekka.Pessi
2010-10-18 15:40 ` [RFC sim-ready-v1 PATCH 2/3] isimodem/sim: added PIN and SIM state handling Pekka.Pessi
2010-10-18 15:40 ` [RFC sim-ready-v1 PATCH 3/3] atmodem/sim: use ofono_sim_ready_notify Pekka.Pessi
2010-10-22 18:55 ` Denis Kenzior
2010-10-19 9:43 ` Marcel Holtmann [this message]
2010-10-19 9:56 ` [RFC sim-ready-v1 PATCH 0/3] ofono_sim_ready_notify Pekka Pessi
2010-10-22 18:54 ` Denis Kenzior
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=1287481415.3316.138.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