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, 05 Aug 2009 14:35:59 -0500	[thread overview]
Message-ID: <200908051436.00016.denkenz@gmail.com> (raw)
In-Reply-To: <fb249edb0908051218l3315aa74v4aaf82ee372bdb3b@mail.gmail.com>

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

Hi Andrew,

> > For setting mailbox numbers, a working EF-MBDN is required since the only
> > alternative would be to store them on the filesystem.
>
> mw_set_indicator does the right thing, I missed this fact.  But when a
> mailbox number is set or received from network all clients get a
> PropertyChanged and the new number can be retrieved from the interface
> but only when the store succeeded, previously it would be available
> regardless of MBDN on the card and the writing to card was secondary.
> The number is kind of important if you want to hear the messages.

So you're right that even if we get an enhanced voice mail notification message 
the Mailbox number won't be signaled unless we have the SIM provisioned with 
EF-MBI & EF-MBDN.

The problem is, if we don't have the SIM files available, then the next time 
oFono is restarted this information is lost anyway until the next MWI 
notification arrives.  So you're back to not being able to dial the mailbox.

This is also an issue with the indications themselves.  If we don't have EF-
MWIS provisioned, the information is lost when oFono is restarted.  So we're 
now inconsistent between MBDN/MWIS, so maybe we shouldn't update them either 
in this case.

One solution is to always store this info on the filesystem and backup to SIM.

The other solution is to preserve current behavior, but to also expose the 
enhanced notifications in a separate interface (as discussed previously) and 
include any Mailbox numbers there.

Thoughts?

Regards,
-Denis

  reply	other threads:[~2009-08-05 19:35 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 [this message]
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
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=200908051436.00016.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.