First connection attempt via F3507g fails

please try this patch against mm:

just apply it on top of the unpacked mm package for testing and build it.

diff --git a/plugins/mm-modem-mbm.c b/plugins/mm-modem-mbm.c
index 6dfba3a..3c9e2be 100644
--- a/plugins/mm-modem-mbm.c
+++ b/plugins/mm-modem-mbm.c
@@ -709,6 +709,10 @@ grab_port (MMModem *modem,
         regex = g_regex_new ("\\r\\n\\+CIEV: (\\d),(\\d)\\r\\n", G_REGEX_RAW | G_REGEX_OPTIMIZE, 0, NULL);
         mm_serial_port_add_unsolicited_msg_handler (MM_SERIAL_PORT (port), regex, mbm_ciev_received, modem, NULL);
         g_regex_unref (regex);
+
+ regex = g_regex_new ("\\r\\n\\ *ESTKSMENU: .* \\r\\n", G_REGEX_RAW | G_REGEX_OPTIMIZE, 0, NULL);
+ mm_serial_port_add_unsolicited_msg_handler (MM_SERIAL_PORT (port), regex, NULL, modem, NULL);
+ g_regex_unref (regex);
     }

     return TRUE;

fyc:

apt-get source modemmanager; cd modemmanager*/; patch -p1 /tmp/416*.patch; sudo apt-get build-dep modemmanager; debuild -b

err ... patch -p1 < /tmp416*.patch ;)

committed: http://cgit.freedesktop.org/ModemManager/ModemManager/commit/?id=180ad403b8773607bd3edf624983196cc47db755

should be in todays dailies: https://edge.launchpad.net/~network-manager/+archive/trunk

please confirm.

On Fri, Aug 21, 2009 at 07:44:57PM -0000, Alexander Sack wrote:
> committed:
> http://cgit.freedesktop.org/ModemManager/ModemManager/commit/?id=180ad403b8773607bd3edf624983196cc47db755
>
> should be in todays dailies: https://edge.launchpad.net/~network-
> manager/+archive/trunk

i mean the dailies that get kicked of 7am UTC ... or around that time.

 - Alexander

Hi

Alexander Sack wrote:
> please try this patch against mm:

I applied the patch, did debuild -b, dpkg -i on the resulting deb,
killed modem-manager, network-manager restarted it.
I then rfkill'd the wwan modem, and then enabled it again and I get the
same errors as before.
I'm pretty sure that I applied the patch correctly, and it looks like
you even committed it to the source that I got.

Cheers,
--
Chris Jones
  <email address hidden>
   www.canonical.com

On Mon, Aug 24, 2009 at 09:53:26PM -0000, Chris Jones wrote:
> Hi
>
> Alexander Sack wrote:
> > please try this patch against mm:
>
> I applied the patch, did debuild -b, dpkg -i on the resulting deb,
> killed modem-manager, network-manager restarted it.
> I then rfkill'd the wwan modem, and then enabled it again and I get the
> same errors as before.
> I'm pretty sure that I applied the patch correctly, and it looks like
> you even committed it to the source that I got.
>

Yep. Just came back from vac. IIRC, I wrongly added a
whitespace. remove the trailing whitespace in the string i gave you
(before the final linefeeds).

So more like

regex = g_regex_new ("\\r\\n\\ *ESTKSMENU: .*\\r\\n", G_REGEX_RAW | G_REGEX_OPTIMIZE, 0, NULL);
                                            ^^^

Can you try that?

 - Alexander

there are a few more bugs in the initialization sequence. I am currently working on a more complete patch that does not make this module so fragile. except something to land on the daily ppas soon.

This bug was fixed in the package modemmanager - 0.2.git.20090909t142854.64b7be7-0ubuntu1

---------------
modemmanager (0.2.git.20090909t142854.64b7be7-0ubuntu1) karmic; urgency=low

  New upstream snapshot 2009-09-09 14:28:54 (GMT)
  + commit: 64b7be7460c3a9dbb3c9981de5754b330a8d2cdd
  + fix LP: #414604 - Modem Manager setting bad baud rate, breaking pppd
    connections.
  + fix LP: #416418 - First connection attempt via F3507g fails
  + fix LP: #424603 - Not closing modems when disconnected

  [ Alexander Sack <email address hidden> ]
  * address MIR comments (LP: #410259)
    + cleanup debian/copyright
    + set debhelper compat level to 5, matching debian/control
      - add debian/compat
    + remove not used patches
      - delete debian/patches/ftbfs_hardy_glib_include_signal_h.patch
    + use proper automake lower and upper version bounds matching the
      versions referred to in debian/rules
      - update debian/control
    + don't redefine cdbs variables DEB_SOURCE, DEB_VERSION and
      DEB_UPSTREAM_VERSION; in turn move DEB_MAJOR_VERSION definition
      below the cdbs includes and use DEB_SOURCE_PACKAGE in GET_SOURCE
      function
      - update debian/rules
    + document why we touch aclocal.m4 in pre-build::
      - update debian/rules
    + use Ubuntu Network Manager Team as Maintainer and add <email address hidden>
      to Uploaders
      - update debian/control

 -- Alexander Sack <email address hidden> Mon, 14 Sep 2009 11:38:05 +0200