RE: [PATCH] Enable hs 2330 MBM modem

2011-02-23 Thread Li, Zhigang
Hi Marcel, Base on your request, I get the lsusb -v result, according the interface result, the "Data Modem" is ok, besides, my test result also ok. In fact, I tried "Module Data Modem", "Broadband Module Data Modem" all are don't work. > looks wrong to me. Please get the lsusb -v out for this

RE: [PATCH] Enable hs 2330 MBM modem

2011-02-16 Thread Li, Zhigang
> > I think this should be "Module Data Modem". > looks wrong to me. Please get the lsusb -v out for this modem where it > lists the interface strings. Interface Descriptor: bLength 9 bDescriptorType 4 bInterfaceNumber1 bAlternateSetting

RE: [PATCH] Enable hs 2330 MBM modem

2011-02-16 Thread Marcel Holtmann
Hi Zhigan, > > >+ g_str_has_suffix(desc, "Data Modem") || > > > I think this should be "Module Data Modem". > > This is confusing part, with my test with this modem on MeeGo image, the > Module Data Modem doesn't work and only Data Modem works. > > So I think it should be ok.

RE: [PATCH] Enable hs 2330 MBM modem

2011-02-13 Thread Li, Zhigang
Hi Marcel, > >+g_str_has_suffix(desc, "Data Modem") || > I think this should be "Module Data Modem". This is confusing part, with my test with this modem on MeeGo image, the Module Data Modem doesn't work and only Data Modem works. So I think it should be ok. Thanks Zhiga

Re: [PATCH] Enable hs 2330 MBM modem

2011-01-28 Thread Marcel Holtmann
Hi Zhigang, > diff --git a/plugins/udev.c b/plugins/udev.c > index 202b225..810049a 100644 > --- a/plugins/udev.c > +++ b/plugins/udev.c > @@ -123,6 +123,7 @@ static void add_mbm(struct ofono_modem *modem, > g_str_has_suffix(desc, "Minicard Modem 2") || >

[PATCH] Enable hs 2330 MBM modem

2011-01-27 Thread Zhigang Li
--- plugins/udev.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/plugins/udev.c b/plugins/udev.c index 202b225..810049a 100644 --- a/plugins/udev.c +++ b/plugins/udev.c @@ -123,6 +123,7 @@ static void add_mbm(struct ofono_modem *modem, g_str_ha