lkml.org 
[lkml]   [2011]   [Nov]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[062/264] USB: option: convert Huawei K3765, K4505, K4605 reservered interface to blacklist
    3.1-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Dan Williams <dcbw@redhat.com>

    commit 0d905fd5ece4ab65e8407c450077744e1c8f661b upstream.

    That's what the blacklist is for...

    Signed-off-by: Dan Williams <dcbw@redhat.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

    ---
    drivers/usb/serial/option.c | 79 +++++++++++++++++++++++---------------------
    1 file changed, 43 insertions(+), 36 deletions(-)

    --- a/drivers/usb/serial/option.c
    +++ b/drivers/usb/serial/option.c
    @@ -495,6 +495,10 @@ static const struct option_blacklist_inf
    .sendsetup = BIT(0) | BIT(1) | BIT(2),
    };

    +static const struct option_blacklist_info huawei_cdc12_blacklist = {
    + .reserved = BIT(1) | BIT(2),
    +};
    +
    static const struct usb_device_id option_ids[] = {
    { USB_DEVICE(OPTION_VENDOR_ID, OPTION_PRODUCT_COLT) },
    { USB_DEVICE(OPTION_VENDOR_ID, OPTION_PRODUCT_RICOLA) },
    @@ -592,12 +596,15 @@ static const struct usb_device_id option
    { USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_E143D, 0xff, 0xff, 0xff) },
    { USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_E143E, 0xff, 0xff, 0xff) },
    { USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_E143F, 0xff, 0xff, 0xff) },
    - { USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_K4505, 0xff, 0xff, 0xff) },
    - { USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_K3765, 0xff, 0xff, 0xff) },
    + { USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_K4505, 0xff, 0xff, 0xff),
    + .driver_info = (kernel_ulong_t) &huawei_cdc12_blacklist },
    + { USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_K3765, 0xff, 0xff, 0xff),
    + .driver_info = (kernel_ulong_t) &huawei_cdc12_blacklist },
    { USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_ETS1220, 0xff, 0xff, 0xff) },
    { USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_E14AC, 0xff, 0xff, 0xff) },
    { USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_K3806, 0xff, 0xff, 0xff) },
    - { USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_K4605, 0xff, 0xff, 0xff) },
    + { USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_K4605, 0xff, 0xff, 0xff),
    + .driver_info = (kernel_ulong_t) &huawei_cdc12_blacklist },
    { USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_K3770, 0xff, 0x02, 0x31) },
    { USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_K3770, 0xff, 0x02, 0x32) },
    { USB_DEVICE_AND_INTERFACE_INFO(HUAWEI_VENDOR_ID, HUAWEI_PRODUCT_K3771, 0xff, 0x02, 0x31) },
    @@ -1207,10 +1214,35 @@ static void __exit option_exit(void)
    module_init(option_init);
    module_exit(option_exit);

    +static bool is_blacklisted(const u8 ifnum, enum option_blacklist_reason reason,
    + const struct option_blacklist_info *blacklist)
    +{
    + unsigned long num;
    + const unsigned long *intf_list;
    +
    + if (blacklist) {
    + if (reason == OPTION_BLACKLIST_SENDSETUP)
    + intf_list = &blacklist->sendsetup;
    + else if (reason == OPTION_BLACKLIST_RESERVED_IF)
    + intf_list = &blacklist->reserved;
    + else {
    + BUG_ON(reason);
    + return false;
    + }
    +
    + for_each_set_bit(num, intf_list, MAX_BL_NUM + 1) {
    + if (num == ifnum)
    + return true;
    + }
    + }
    + return false;
    +}
    +
    static int option_probe(struct usb_serial *serial,
    const struct usb_device_id *id)
    {
    struct usb_wwan_intf_private *data;
    +
    /* D-Link DWM 652 still exposes CD-Rom emulation interface in modem mode */
    if (serial->dev->descriptor.idVendor == DLINK_VENDOR_ID &&
    serial->dev->descriptor.idProduct == DLINK_PRODUCT_DWM_652 &&
    @@ -1223,14 +1255,14 @@ static int option_probe(struct usb_seria
    serial->interface->cur_altsetting->desc.bInterfaceClass != 0xff)
    return -ENODEV;

    - /* Don't bind network interfaces on Huawei K3765, K4505 & K4605 */
    - if (serial->dev->descriptor.idVendor == HUAWEI_VENDOR_ID &&
    - (serial->dev->descriptor.idProduct == HUAWEI_PRODUCT_K3765 ||
    - serial->dev->descriptor.idProduct == HUAWEI_PRODUCT_K4505 ||
    - serial->dev->descriptor.idProduct == HUAWEI_PRODUCT_K4605) &&
    - (serial->interface->cur_altsetting->desc.bInterfaceNumber == 1 ||
    - serial->interface->cur_altsetting->desc.bInterfaceNumber == 2))
    - return -ENODEV;
    + /* Don't bind reserved interfaces (like network ones) which often have
    + * the same class/subclass/protocol as the serial interfaces. Look at
    + * the Windows driver .INF files for reserved interface numbers.
    + */
    + if (is_blacklisted(
    + serial->interface->cur_altsetting->desc.bInterfaceNumber,
    + OPTION_BLACKLIST_RESERVED_IF,
    + (const struct option_blacklist_info *) id->driver_info))

    /* Don't bind network interface on Samsung GT-B3730, it is handled by a separate module */
    if (serial->dev->descriptor.idVendor == SAMSUNG_VENDOR_ID &&
    @@ -1239,7 +1271,6 @@ static int option_probe(struct usb_seria
    return -ENODEV;

    data = serial->private = kzalloc(sizeof(struct usb_wwan_intf_private), GFP_KERNEL);
    -
    if (!data)
    return -ENOMEM;
    data->send_setup = option_send_setup;
    @@ -1248,30 +1279,6 @@ static int option_probe(struct usb_seria
    return 0;
    }

    -static bool is_blacklisted(const u8 ifnum, enum option_blacklist_reason reason,
    - const struct option_blacklist_info *blacklist)
    -{
    - unsigned long num;
    - const unsigned long *intf_list;
    -
    - if (blacklist) {
    - if (reason == OPTION_BLACKLIST_SENDSETUP)
    - intf_list = &blacklist->sendsetup;
    - else if (reason == OPTION_BLACKLIST_RESERVED_IF)
    - intf_list = &blacklist->reserved;
    - else {
    - BUG_ON(reason);
    - return false;
    - }
    -
    - for_each_set_bit(num, intf_list, MAX_BL_NUM + 1) {
    - if (num == ifnum)
    - return true;
    - }
    - }
    - return false;
    -}
    -
    static void option_instat_callback(struct urb *urb)
    {
    int err;



    \
     
     \ /
      Last update: 2011-11-10 05:27    [W:6.859 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site