All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marcel Holtmann <marcel@holtmann.org>
To: ofono@ofono.org
Subject: Re: [online-impl-v2 PATCH 00/10] implement Online property
Date: Thu, 09 Sep 2010 01:36:13 +0200	[thread overview]
Message-ID: <1283988973.6640.141.camel@localhost.localdomain> (raw)
In-Reply-To: <1283798780-16363-1-git-send-email-Pekka.Pessi@nokia.com>

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

Hi Pekka,

> Here are 2nd stab at implementing Online property with various modem
> drivers.

so I pushed most of them, but not all of them.

STE version needs testing by the STE guys, but I did run into some SIM
not ready issues with MBM when switching online/offline a few times. So
that needs to be looked into.

I did leave Palm Pre and G1 patches out. Andrew is the best candidate
for testing these. I have no idea on how they behave.

> Some more obvious errors (in using g_at_chat_send()) have been
> fixed. Huawei has been updated after the comment from Marcel.

The Huawei has more and more issues with SIM not ready now. This needs
fixing anyway. I pushed the patch anymore to make these symptoms more
clear to everybody.

> I've dropped support for the Nokia usb stick, according to the spec it
> does not implement but AT+CFUN=0 and AT+CFUN=1. (Not that I have tested
> what AT+CFUN=0 does.)

I have that stuck, but that beast is broken like crazy. The firmware has
even a bug with AT+COPS :(

> Again, I have not been able to test with most modems. All of them might
> not make sense (if the modem does not support AT+CFUN=4, for
> instance). There might be some common caveats.  E.g., if the modem does
> to wake up with AT+CFUN=4, it might work with an initial cfun command
> like "AT+CFUN=1 ; +CFUN=4". At least Calypso requires such an
> initialization command.
> 
> I'd appreciate if the people with access to various hardware could test
> them with the patches here (using test/enable-modem, test/online-modem
> and test/offline-modem, or, e.g., the connman with "ofono-refactor-v2"
> and "service-leak" patches).

I have done basic testing and so far so good. Huawei and MBM needs
further attention for sure. I am more than shocked that I actually do
have all this crap hardware on my desk :(

Regards

Marcel



  parent reply	other threads:[~2010-09-08 23:36 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-06 18:46 [online-impl-v2 PATCH 00/10] implement Online property Pekka.Pessi
2010-09-06 18:46 ` [online-impl-v2 PATCH 01/10] atgen: " Pekka.Pessi
2010-09-06 18:46   ` [online-impl-v2 PATCH 02/10] calypso: " Pekka.Pessi
2010-09-06 18:46     ` [online-impl-v2 PATCH 03/10] g1: " Pekka.Pessi
2010-09-06 18:46       ` [online-impl-v2 PATCH 04/10] hso: " Pekka.Pessi
2010-09-06 18:46         ` [online-impl-v2 PATCH 05/10] huawei: " Pekka.Pessi
2010-09-06 18:46           ` [online-impl-v2 PATCH 06/10] mbm: " Pekka.Pessi
2010-09-06 18:46             ` [online-impl-v2 PATCH 07/10] novatel: " Pekka.Pessi
2010-09-06 18:46               ` [online-impl-v2 PATCH 08/10] palmpre: " Pekka.Pessi
2010-09-06 18:46                 ` [online-impl-v2 PATCH 09/10] ste: " Pekka.Pessi
2010-09-06 18:46                   ` [online-impl-v2 PATCH 10/10] zte: " Pekka.Pessi
2010-09-08 23:30                     ` Marcel Holtmann
2010-09-08 23:30                   ` [online-impl-v2 PATCH 09/10] ste: " Marcel Holtmann
2010-09-08 23:30               ` [online-impl-v2 PATCH 07/10] novatel: " Marcel Holtmann
2010-09-08 23:30             ` [online-impl-v2 PATCH 06/10] mbm: " Marcel Holtmann
2010-09-08 23:30           ` [online-impl-v2 PATCH 05/10] huawei: " Marcel Holtmann
2010-09-08 23:30         ` [online-impl-v2 PATCH 04/10] hso: " Marcel Holtmann
2010-09-08 23:30   ` [online-impl-v2 PATCH 01/10] atgen: " Marcel Holtmann
2010-09-08 23:36 ` Marcel Holtmann [this message]
2010-09-14 18:49   ` [online-impl-v2 PATCH 00/10] " Pekka Pessi
2010-09-14 21:30     ` Marcel Holtmann
2010-09-14 20:06   ` Sjur =?unknown-8bit?q?Br=C3=A6ndeland?=
2010-09-14 21:28     ` 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=1283988973.6640.141.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.