Re: Problem detecting/using wwan0 modem

[Denis Kenzior <denkenz@gmail.com>]

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

Hi Michael,

>> The rules seem to be OK, so the udev detection code
>> in plugins/udev.c:add_mbm is not finding your modem.
>>
>> Try starting oFono with
>>
>> ofonod -n -d 'plugins/udev*' (sorry, I messed this up earlier)
>>
>> and report the output
>>
>   meego$ /usr/sbin/ofonod -n -d 'plugins/udev*'
>   ofonod[811]: oFono version 0.48
>   ofonod[811]: plugins/udev.c:add_modem() /devices/pci0000:00/0000:00:1d.7/usb1/1-5/1-5:1.1/tty/ttyACM0 (mbm)
>   ofonod[811]: plugins/udev.c:add_mbm() desc: HP hs2330 Mobile Broadband Module Data Modem
>   ofonod[811]: plugins/udev.c:add_modem() /devices/pci0000:00/0000:00:1d.7/usb1/1-5/1-5:1.3/tty/ttyACM1 (mbm)
>   ofonod[811]: plugins/udev.c:add_mbm() desc: HP hs2330 Mobile Broadband Module Modem
>   ofonod[811]: plugins/udev.c:add_modem() /devices/pci0000:00/0000:00:1d.7/usb1/1-5/1-5:1.6/net/wwan0 (mbm)
>   ofonod[811]: plugins/udev.c:add_mbm() desc: HP hs2330 Mobile Broadband Module Network Adapter
>   ofonod[811]: plugins/udev.c:add_modem() /devices/pci0000:00/0000:00:1d.7/usb1/1-5/1-5:1.9/tty/ttyACM2 (mbm)
>   ofonod[811]: plugins/udev.c:add_mbm() desc: HP hs2330 Mobile Broadband Module NMEA
>   ofonod[811]: Exit
> 
> ...sorry I think before I mistyped the MeeGo 1.2 (current) ofono
> version 0.48 as 0.58 incorrectly.
> 
> But the output shown above is rather good, so I'm at a loss
> to what could be wrong. Any ideas? Is connman relevant here?
> 

Based on the output above, the udev plugin needed to be modified to
detect the HP version of the MBM modems.  The change can be found in
commit: 6a2847f7aba5d84b06940ee29300d4b6fd31d682.  I'm also attaching
this patch for your reference.  Please report back if this solves your
issue.

Regards,
-Denis

[-- Attachment #2: 0001-udev-Tweak-detection-rules-for-HP-HS2330-cards.patch --]
[-- Type: text/plain, Size: 2007 bytes --]

>From 6a2847f7aba5d84b06940ee29300d4b6fd31d682 Mon Sep 17 00:00:00 2001
From: Denis Kenzior <denkenz@gmail.com>
Date: Mon, 27 Jun 2011 12:17:22 -0500
Subject: [PATCH] udev: Tweak detection rules for HP HS2330 cards

---
 plugins/udev.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/plugins/udev.c b/plugins/udev.c
index 1580b02..8ea937c 100644
--- a/plugins/udev.c
+++ b/plugins/udev.c
@@ -122,6 +122,7 @@ static void add_mbm(struct ofono_modem *modem,
 			g_str_has_suffix(desc, "Minicard Modem 2") ||
 			g_str_has_suffix(desc, "Mini-Card Modem") ||
 			g_str_has_suffix(desc, "Broadband Modem") ||
+			g_str_has_suffix(desc, "Module Modem") ||
 			g_str_has_suffix(desc, "Broadband USB Modem"))) {
 		devnode = udev_device_get_devnode(udev_device);
 
@@ -132,11 +133,13 @@ static void add_mbm(struct ofono_modem *modem,
 	} else if (registered == 0 &&
 			(g_str_has_suffix(desc, "Minicard Data Modem") ||
 			g_str_has_suffix(desc, "Mini-Card Data Modem") ||
+			g_str_has_suffix(desc, "Module Data Modem") ||
 			g_str_has_suffix(desc, "Broadband Data Modem"))) {
 		devnode = udev_device_get_devnode(udev_device);
 		ofono_modem_set_string(modem, DATA_DEVICE, devnode);
 	} else if (g_str_has_suffix(desc, "Minicard GPS Port") ||
 			g_str_has_suffix(desc, "Mini-Card GPS Port") ||
+			g_str_has_suffix(desc, "Module NMEA") ||
 			g_str_has_suffix(desc, "Broadband GPS Port")) {
 		devnode = udev_device_get_devnode(udev_device);
 		ofono_modem_set_string(modem, GPS_DEVICE, devnode);
@@ -144,6 +147,7 @@ static void add_mbm(struct ofono_modem *modem,
 			(g_str_has_suffix(desc, "Minicard Network Adapter") ||
 			g_str_has_suffix(desc, "Mini-Card Network Adapter") ||
 			g_str_has_suffix(desc, "Broadband Network Adapter") ||
+			g_str_has_suffix(desc, "Module Network Adapter") ||
 			g_str_has_suffix(desc, "Minicard NetworkAdapter"))) {
 		devnode = get_property(udev_device, "INTERFACE");
 		ofono_modem_set_string(modem, NETWORK_INTERFACE, devnode);
-- 
1.7.3.4