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
next prev 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.