Open Source Telephony
 help / color / mirror / Atom feed
From: Marcel Holtmann <marcel@holtmann.org>
To: ofono@ofono.org
Subject: Re: EF-SPN API (was: [gprs-provision PATCHv7 0/5] Plugin API for provisioning of GPRS context settings)
Date: Thu, 27 Jan 2011 08:37:17 +0100	[thread overview]
Message-ID: <1296113837.1520.142.camel@aeonflux> (raw)
In-Reply-To: <1296112918.30517.26.camel@jsaunama-desktop>

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

Hi Jukka,

> > you can have a look on what it would take to store a) read SPN in the
> > SIM atom and store it and b) let netreg atom use that value.
> 
> Simple solution for this would be something like:
> 
> During some SIM initialisation phase (at sim_ready()?), sim-atom
> requests ofono_sim_read(SIM_EFSPN_FILEID).
> 
> For netreg (and gprs/provisioning) it would provide:
> typedef void (*ofono_sim_spn_read_cb_t)(int ok, const char *spn,
> 					void *userdata);
> void ofono_sim_read_spn(struct ofono_sim *sim, ofono_sim_spn_read_cb_t cb,
> 			void *userdata);
> 
> ofono_sim_read_spn would queue requests until SPN is received (and
> immediately call callback if SPN is ready).

don't make this asynchronous. Just read the SPN and store it. Then it
can be easily accessed similar to MCC and MNC.

The GPRS atom and netreg for that matter will not be available before
the SIM atom has been fully initialized anyway.

If there is any race condition with this one, the we need to be really
really careful since it has a lot of consequences.

Regards

Marcel



  reply	other threads:[~2011-01-27  7:37 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-26 12:34 [gprs-provision PATCHv7 0/5] Plugin API for provisioning of GPRS context settings Jukka Saunamaki
2011-01-26 12:34 ` [PATCH 1/5] gprs-provision: add driver API header Jukka Saunamaki
2011-01-26 12:34 ` [PATCH 2/5] ofono.h: add gprs-provision Jukka Saunamaki
2011-01-26 12:34 ` [PATCH 3/5] gprs-provision: add driver API sources Jukka Saunamaki
2011-01-26 12:34 ` [PATCH 4/5] gprs: add gprs context provisioning Jukka Saunamaki
2011-01-26 12:34 ` [PATCH 5/5] gprs-provision: add example context provisioning driver Jukka Saunamaki
2011-01-26 20:56 ` [gprs-provision PATCHv7 0/5] Plugin API for provisioning of GPRS context settings Denis Kenzior
2011-01-27  6:41   ` Jukka Saunamaki
2011-01-27  6:59     ` Marcel Holtmann
2011-01-27  7:00       ` Marcel Holtmann
2011-01-27  7:21       ` EF-SPN API (was: [gprs-provision PATCHv7 0/5] Plugin API for provisioning of GPRS context settings) Jukka Saunamaki
2011-01-27  7:37         ` Marcel Holtmann [this message]
2011-01-27  7:53           ` Jukka Saunamaki

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=1296113837.1520.142.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