All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marcel Holtmann <marcel@holtmann.org>
To: ofono@ofono.org
Subject: Re: Access to SIM card when Modem is not "Powered"
Date: Tue, 30 Mar 2010 11:02:46 -0700	[thread overview]
Message-ID: <1269972166.11714.350.camel@localhost.localdomain> (raw)
In-Reply-To: <80fd4e751003300940m1a6d9005se0de6e0de03f6e49@mail.gmail.com>

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

Hi Pekka,

> >> > So I'm still having trouble understanding the issue.  When oFono calls
> >> > disable, the driver is expected to take all necessary steps to disable the
> >> > modem.  If that means waiting N seconds after the command has been sent, so be
> >> > it.  During shutdown of the daemon, oFonod waits for a grace period and waits
> >> > on any devices that are being shut down.  In effect it "hangs around" after
> >> > power off.
> >>
> >> >> Another solution is to use sscd-like daemon also with ofono (the oFono
> >> >> Powered property would then just follow the power state of the modem).
> >> >
> >> > Automatic powerup is actually possible from the driver.  See HFP driver for
> >> > details.
> >> >
> >> >> > We reply with the busy error, you're correct.  However, I don't really
> >> >> > see anything better we can do here, do you have any suggestions?
> >> >>
> >> >> Keep the target state around somewhere, or call enable/disable
> >> >> regardless of the current state of the Powered property?
> >> >>
> >> >
> >> > Note that oFono does not record the powered preferences, ConnMan is
> >> > responsible for that.
> >> >
> >> > Sending a disable when we are already disabled would be wrong and would break
> >> > some plugins.
> >> >
> >> > And I'm still having trouble understanding why you want this.  Please give
> >> > concrete use-cases.
> >>
> >> Sure.
> >>
> >> I want Powered-1 that controls the atoms. Atoms should be loaded when
> >> modem is in responsive state and removed when, e.g., modem reboots.
> >> This we can do now, iow, if you connect a Nokia phone via USB, oFono
> >> can follow its state via the MTC indications it sends on top of the
> >> phonet link running over USB.
> >>
> >> I want Powered-2 that controls the modem power. When ofonod starts in
> >> N900, it should power up the internal modem. When ofonod terminates
> >> itself, it should shut down modem nicely before calling exit().
> >>
> >> Now, enable/disable/ofono_modem_set_powered() controls both aspects; I
> >> want to separate them. It is also possible to implement Powered-2 in
> >> the probe/remove methods; however, they are quite time-consuming
> >> operations and best done from the mainloop.
> >
> > I am with Denis here. I am missing the point in what you are trying to
> > achieve. The complexity you propose should not be exposed to the
> > applications at all. This can be all handled internally. Or I am missing
> > something essential, but right now, I don't see it.
> 
> I'm trying to
> 1) load atoms only after when isimodem is up and running and reset the
> state of the isimodem atoms in case the isimodem reboots (or user
> turns off a Nokia handset connected via USB)
> 2) have asyncronous probe and remove
> 3) separate rf state and availablity of the atoms, especially the SIM atoms.
> 
> With the current enable/disable/ofono_modem_set_powered, I can do 1)
> or 2), but not both. I can not do 3 at all.

non of these should be solved via the D-Bus at all. Really this is
internal modem specific details. You are approaching this wrongly.

1) If the modem reboots, then handle this inside the isimodem plugin of
oFono. No reason to involve userspace here. If the handset gets turned
off, then the modem object just goes away.

2) What do you mean by this. They are asynchronous.

3) I don't understand this. We have pre-sim and post-sim functionality.
If you RFKILL a radio it would be same as removing its object path. Or
do you wanna access the SIM card while RFKILLed. What is that good for?

> >> It seems to me that Marcel thinks "Powered" should control the RF
> >> state, too. So, a separate property for enabling he RF would be nice,
> >> too.
> >
> > That is what I call RFKILL and we have a proper subsystem for that.
> 
> Huh? How do you plan to rfkill a modem behind serial line (over
> bluetooth? over USB? over SSI?)? Should we add a kernel driver for all
> the modems? How do you plan to solve the interaction between ofonod at
> commands and the rfkill driver?
> 
> Phonet is just a glorified serial line with binary protocol. Kernel
> drivers do not care what is at the other end of that serial line.
> There are gpio lines to wake up the thing at the end of serial line;
> gpio lines have their own kernel driver. That is all we need from
> kernel.

This is no reason to not integrate Phonet with RFKILL. RFKILL can kill
the radio interface or the whole device. So if it chooses to take the
device off the Phonet "bus", then that is fine as well.

Regards

Marcel



  reply	other threads:[~2010-03-30 18:02 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 [this message]
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
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=1269972166.11714.350.camel@localhost.localdomain \
    --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 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.