All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: [PATCH v2 2/6] gatchat: Add g_at_chat_send_full.
Date: Mon, 10 May 2010 18:48:39 -0500	[thread overview]
Message-ID: <201005101848.40097.denkenz@gmail.com> (raw)
In-Reply-To: <AANLkTimad9xDirmjF1WHzrb6Qu6b39plcgRki-eq7rLF@mail.gmail.com>

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

Hi Andrew,

> Hi Denis,
> 
> On 10 May 2010 21:13, Denis Kenzior <denkenz@gmail.com> wrote:
> >> g_at_chat_send_full takes two additional parameters compared to
> >> g_at_chat_send.  The destroy notification is removed from
> >> g_at_chat_send and users updated.
> >> ---
> >
> > You stated earlier that the modem will return an error and not time out.
> >  If this is the case then I'm really failing to see the point of needing
> > the submit callback.
> 
> I stated that as en example of an incompliant modem.  The specs are
> not very clear here but say that we should assume the modem is dead if
> no response to the AT+CSIM is sent within a timeout of at least 5
> seconds.

The modem or the SIM?  I think the specs are talking about the SIM here and 
the modems do need to poll and detect SIM dead conditions.  If we were to 
implement a GSM soft-modem, then we'd definitely need to poll...

> 
> My thinking so far was that ofono should support an (imaginary) modem
> that's described by the specs, and on top of that add support for
> vendor specific things, but maybe that is useless in this case.  What
> do you think?

I think we should drop this capability until we actually see this 'imaginary' 
modem you describe in the wild.  Most vendors intercept everything related to 
SIM / STK and can even do hardware SIM detection.  Makes no sense for us to 
worry about it right now.

Regards,
-Denis

      reply	other threads:[~2010-05-10 23:48 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-07  0:27 [PATCH v2 2/6] gatchat: Add g_at_chat_send_full Andrzej Zaborowski
2010-05-10 19:13 ` Denis Kenzior
2010-05-10 22:45   ` Andrzej Zaborowski
2010-05-10 23:48     ` Denis Kenzior [this message]

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