ugensa.c : need help with patch : tried to add Huawei Mobile E169

[Brendan Kosowski <bmk@xxxxxxxxxx>]

Device code for E169 was there (as Huawei Mobile Connect), but it was 
showing up as a single mass storage device, so I added it to the E220 
re-attach code.

My Patch has no effect. Can anyone tell my why?

Patch follows:
--- ugensa.c.orig       2009-03-28 10:12:26.000000000 +1100
+++ ugensa.c    2009-03-28 10:24:34.000000000 +1100
@@ -131,7 +131,7 @@
        { USB_DEVICE(0x1199, 0x6853) }, /* Sierra Wireless AirCard 881E */
        { USB_DEVICE(0x1199, 0x6855) }, /* Sierra Wireless AirCard 880U */
        { USB_DEVICE(0x1199, 0x6856) }, /* Sierra Wireless AirCard 881U */
-       { USB_DEVICE(0x12d1, 0x1001) }, /* Huawei Mobile Connect */
+       { USB_DEVICE(0x12d1, 0x1001) }, /* Huawei Mobile Connect/E169 */
        { USB_DEVICE(0x12d1, 0x1003) }, /* Huawei Mobile E220 */
        { USB_DEVICE(0x12d1, 0x1004) }, /* Huawei Mobile E220 */
        { USB_DEVICE(0x1410, 0x1100) }, /* Novatel Wireless Merlin 
XS620/S640 */
@@ -186,7 +186,8 @@
         */
        id = usbd_get_interface_descriptor(uaa->iface);
        if (id == NULL || id->bInterfaceClass == UICLASS_MASS) {
-               if ((uaa->vendor == 0x12d1 && uaa->product == 0x1003) ||
+               if ((uaa->vendor == 0x12d1 && uaa->product == 0x1001) ||
+                   (uaa->vendor == 0x12d1 && uaa->product == 0x1003) ||
                    (uaa->vendor == 0x12d1 && uaa->product == 0x1004)) {
                        if (uaa->nifaces == 1)
                                return UMATCH_VENDOR_IFACESUBCLASS;
@@ -219,7 +220,8 @@

        id = usbd_get_interface_descriptor(ucom->sc_iface);
        if (id == NULL || id->bInterfaceClass == UICLASS_MASS) {
-               if ((uaa->vendor == 0x12d1 && uaa->product == 0x1003) ||
+               if ((uaa->vendor == 0x12d1 && uaa->product == 0x1001) ||
+                   (uaa->vendor == 0x12d1 && uaa->product == 0x1003) ||
                    (uaa->vendor == 0x12d1 && uaa->product == 0x1004)) {
                        ugensa_e220_changemode(uaa->device);
                }