All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: Access to SIM card when Modem is not "Powered"
Date: Thu, 01 Apr 2010 10:07:51 -0500	[thread overview]
Message-ID: <201004011007.52611.denkenz@gmail.com> (raw)
In-Reply-To: <k2m80fd4e751004010214r8f0ca85dx3675b4a02eacf505@mail.gmail.com>

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

Hi Pekka,

> I want to prevent core from exiting while a modem is in transitional
> state. Before ofonod terminates, it calls disable and waits until
> Powered gets false. I'd rather change Powered to false immediately
> (when modem shutdown starts) but still keep waiting until modem is
> properly powered down.

So the core exits once modems have been powered down and plugins unloaded.  We 
established that already.  I already considered the powered to false 
immediately semantic and discarded it in favor of the current one.  Please 
present an actual argument / use case why you think powered going immediately 
to false is better than the current implementation.

> It seems to me that Marcel and Aki want to control TX power with the
> Powered property. (BTW, there already is a post-sim mechanism to
> control the RX/TX, namely Register/Deregister.)

Register / Deregister refer to something else entirely actually, and the 
modems that do allow Deregister still keep RX/TX circuit on for emergency 
calls.

I think you're expecting oFono to provide more information that it actually 
wants or needs to.  In the end it is up to the modem driver to decide what 
powered off means.  Most simply keep the TX/RX circuits off and the SIM powered 
up, others (like calypso) physically power off the modem.  But even oFono 
doesn't know nor care.

> Currently I'm using config to determine if the isimodem should or
> should not control the modem and which phonet device it should or
> should not use. It is used to determine if ofonod is run with maemo5
> cellular daemons or without them.

You're not helping your case here ;)  Why should oFono care about this use 
case?

In the end I'm fine if you want to implement ofono_modem_unregister that takes 
the modem object off DBus and resets the driver but keeps the attributes 
around.  I know Marcel wanted this anyway for something else.

> You don't seem to handle modem resets at all. Once atdriver gets so
> far, other hardware will benefit, too.

As you mentioned earlier, modem resets are handled by setting the modem 
powered=off then true.  

> What we miss from the core is a way to do asynchronous cleanup
> (separate from Powered). A shutdown() method in modem driver and
> corresponding callback, like ofono_modem_shutdown_ready(). This way we
> can do what ever N900 requires us to do and there is no need to touch
> other drivers.
> 

Send a patch with your proposal.  So far I like the idea of disable having a 
hint on whether a shutdown is in progress more than an explicit shutdown 
method.  

Regards,
-Denis

  parent reply	other threads:[~2010-04-01 15:07 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-19 17:24 Access to SIM card when Modem is not "Powered" Pekka Pessi
2010-03-19 18:18 ` Denis Kenzior
2010-03-29 18:29   ` Pekka Pessi
2010-03-29 18:40     ` Bastian, Waldo
2010-03-30 11:39       ` Pekka Pessi
2010-03-30 14:22         ` Marcel Holtmann
2010-03-29 18:53     ` Denis Kenzior
2010-03-30 11:36       ` Pekka Pessi
2010-03-30  4:50         ` Denis Kenzior
2010-03-30 15:34           ` Pekka Pessi
2010-03-30 15:45             ` Marcel Holtmann
2010-03-30 16:40               ` Pekka Pessi
2010-03-30 18:02                 ` Marcel Holtmann
2010-03-30 22:55                   ` Pekka Pessi
2010-03-30 23:24                     ` Denis Kenzior
2010-04-01  9:14                       ` Pekka Pessi
2010-04-01 12:44                         ` Aki Niemi
2010-04-01 15:09                           ` Denis Kenzior
2010-04-01 15:07                         ` Denis Kenzior [this message]
2010-04-12 14:08                           ` Pekka Pessi
2010-04-15 21:54                             ` Denis Kenzior
2010-03-30 17:26               ` Aki Niemi
2010-03-30 16:13             ` Denis Kenzior
2010-03-30 17:37               ` Aki Niemi
2010-03-30 18:05                 ` Denis Kenzior
2010-03-30 18:27                   ` Aki Niemi
2010-03-30 11:57         ` Aki Niemi
2010-03-30 14:19           ` Marcel Holtmann

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