All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: [PATCH 4/4] Add a MessageWaiting interface to track message waiting indications.
Date: Wed, 12 Aug 2009 11:13:51 -0500	[thread overview]
Message-ID: <200908121113.51953.denkenz@gmail.com> (raw)
In-Reply-To: <a32762a20908120648q7d07800evb542b5ec253ae622@mail.gmail.com>

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

Hi Aki,

> 2009/8/10 Denis Kenzior <denkenz@gmail.com>:
> >> It would make sense for the providers to resend the MWI information
> >> everytime subscriber connects the network and then it would be less
> >> problematic in case the information couldn't be written to SIM, but I
> >> don't know if they ever do this.
> >
> > Probably they don't, there are also weird roaming cases to consider,
> > where you receive MWI, turn phone off, step on a plane and turn phone on
> > with some other network entirely.
>
> There is another case to consider when the SIM is moved to a
> completely different phone. In fact, I think this is the original
> reason to store the MWI flag to the SIM.

This is why we always attempt to write to the SIM for EFmwis.  If the SIM 
doesn't support this file,  then there's nothing we can do to preserve the MWIS 
information in the case you describe above.

There are other cases as well:
	- EFmbdn & perhaps EFmbi
	- EFmsisdn
	- EFcbmid
	- EFcbmir
	- EFcfis

>
> > Can we make the MWI interface smarter somehow?  Perhaps by not enabling
> > it unless at least the EFmwis file is available, or by always storing the
> > EFmwis contents on the filesystem?
>
> I think this makes sense. There is another, related issue with some
> Nokia modems, though. They might not support low-level, EF access
> directly, but have some higher level API for accessing things like the
> MWI flag, HPLMN list and SPN. We might need to think about adding such
> high-level API support to the driver API as (optional) ops --
> especially if other modems behave similarly -- otherwise the ISI modem
> driver may need some kluge to handing EF writes.

There is vast amount of information stored on the SIM that we need to expose.  
Today oFono processes:
	- EFspn
	- EFpnn
	- EFopl
	- EFmbi
	- EFmbdn
	- EFmwis
	- EFspdi
	- EFmsisdn

We also eventually need to support:
	- EFecc
	- EFsdn
	- EFad
	- EFcbmid
	- EFcbmir
	- EFehplmn
	- EFehplmnpi
	- EFici
	- EFict
	- EFoci
	- EFoct
	- EFcfis

 So there are two issues here:
	- There is absolutely no standard data format for such vendor commands.  So 
coming up with a higher level API that supports all modems might be tricky.  
The advantage of using low-level SIM access is that the command syntax & data 
format is standard and should be supported by any modem out there.  Of course, 
as you point out, many don't.
	- This presents the issue of having many more code paths to debug inside the 
core and potentially many more plugin functions to implement.

I'd like to keep the core as clean and small as possible.  What is your 
feeling for the level of bloat such higher level APIs will introduce in 
comparison to a giant if-else statement inside the driver itself?

Regards,
-Denis

  reply	other threads:[~2009-08-12 16:13 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-30  8:05 [PATCH 4/4] Add a MessageWaiting interface to track message waiting indications Andrzej Zaborowski
2009-07-31 16:19 ` Denis Kenzior
2009-08-01 23:09   ` Andrzej Zaborowski
2009-08-03 16:32     ` Denis Kenzior
2009-08-03 19:09       ` Andrzej Zaborowski
2009-08-03 19:29         ` Denis Kenzior
2009-08-03 21:56           ` andrzej zaborowski
2009-08-04 20:23             ` Denis Kenzior
2009-08-05 14:18               ` andrzej zaborowski
2009-08-05 17:47                 ` Denis Kenzior
2009-08-05 19:18                   ` andrzej zaborowski
2009-08-05 19:35                     ` Denis Kenzior
2009-08-07  3:25                       ` andrzej zaborowski
2009-08-10 16:35                         ` Denis Kenzior
2009-08-12 13:48                           ` Aki Niemi
2009-08-12 16:13                             ` Denis Kenzior [this message]
2009-08-14 12:07                               ` Aki Niemi
2009-08-19 13:44                           ` Andrzej Zaborowski

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=200908121113.51953.denkenz@gmail.com \
    --to=denkenz@gmail.com \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.