Re: [Patch]nm-moderm-probe.c: add two VID and indicate usbif 0 is modem port

[Dan Williams <dcbw redhat com>]

On Wed, 2009-10-14 at 16:28 +0800, spark wrote:
>  
> 
> Add two VID to indicate usbif 0 is modem port to support 3.5G data
> cards of BANDRICH and PIRELLI 

Hi!

Could you give me a few more details about the Bandrich parts?  The only
reason that USB interface > 0 was ignored for Huawei parts was that the
Huawei firmware become confused when the host tried to talk to it on
more than one AT command channel at a time (which is completely broken
and should be fixed in the firmware).

I would hope that the Bandrich parts do not have that limitation?  None
of the other device's I've encountered have that limitation (Sierra,
Option, Novatel, ZTE, Pantech, Qualcom Gobi, Sony-Ericsson, CMOtech,
etc).

Any chance you could indicate which Bandrich and Pirelli parts had
problems probing the secondary ports?

Dan

>  
> 
> --- nm-modem-probe.c 2009-04-08 18:46:21.000000000 +0800
> 
> +++ patched/nm-modem-probe.c       2009-10-13 19:35:18.000000000 +0800
> 
> @@ -32,6 +32,8 @@
> 
>  
> 
>  #define HUAWEI_VENDOR_ID   0x12D1
> 
>  #define SIERRA_VENDOR_ID   0x1199
> 
> +#define BANDRICH_VENDOR_ID 0x1A8D
> 
> +#define PIRELLI_VENDOR_ID  0x1266
> 
>  
> 
>  #define MODEM_CAP_GSM         0x0001 /* GSM */
> 
>  #define MODEM_CAP_IS707_A     0x0002 /* CDMA Circuit Switched Data */
> 
> @@ -549,6 +551,20 @@
> 
>                goto exit;
> 
>        }
> 
>  
> 
> +      if (vid == BANDRICH_VENDOR_ID && usbif != 0) {
> 
> +              verbose ("(%s) ignoring BANDRICH USB interface #1",
> device);
> 
> +              if (export)
> 
> +                      printf ("ID_NM_MODEM_PROBED=1\n");
> 
> +              goto exit;
> 
> +      }
> 
> +
> 
> +      if (vid == PIRELLI_VENDOR_ID && usbif != 0) {
> 
> +              verbose ("(%s) ignoring PIRELLI USB interface #1",
> device);
> 
> +              if (export)
> 
> +                      printf ("ID_NM_MODEM_PROBED=1\n");
> 
> +              goto exit;
> 
> +      }
> 
> +
> 
>        verbose ("probing %s", device);
> 
>  
> 
>        /* If a delay was specified, retry opening the serial port for
> that
> 
>  
> 
> 
> _______________________________________________
> NetworkManager-list mailing list
> NetworkManager-list gnome org
> http://mail.gnome.org/mailman/listinfo/networkmanager-list