linux-kernel - Re: [PATCH 1/1] usb: serial: option: Adding support for Telit LE922 PID 0x1045

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAGRyCJEX=FJJS3r85tNZ_b3i4PbqFhNX2j8skmTg9ksCw=5LMg@mail.gmail.com>
Date:	Mon, 29 Feb 2016 14:30:21 +0100
From:	Daniele Palmas <dnlplm@...il.com>
To:	Johan Hovold <johan@...nel.org>
Cc:	linux-usb <linux-usb@...r.kernel.org>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/1] usb: serial: option: Adding support for Telit LE922
 PID 0x1045

Hi Johan,

2016-02-28 16:36 GMT+01:00 Johan Hovold <johan@...nel.org>:
> On Fri, Feb 26, 2016 at 01:35:27PM +0100, Daniele Palmas wrote:
>> This patch adds support for 0x1045 PID of Telit LE922.
>>
>> Signed-off-by: Daniele Palmas <dnlplm@...il.com>
>> ---
>>  drivers/usb/serial/option.c | 8 ++++++++
>>  1 file changed, 8 insertions(+)
>>
>> diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c
>> index db86e51..4be2c7c 100644
>> --- a/drivers/usb/serial/option.c
>> +++ b/drivers/usb/serial/option.c
>> @@ -270,6 +270,7 @@ static void option_instat_callback(struct urb *urb);
>>  #define TELIT_PRODUCT_UE910_V2                       0x1012
>>  #define TELIT_PRODUCT_LE922_USBCFG0          0x1042
>>  #define TELIT_PRODUCT_LE922_USBCFG3          0x1043
>> +#define TELIT_PRODUCT_LE922_USBCFG5          0x1045
>>  #define TELIT_PRODUCT_LE920                  0x1200
>>  #define TELIT_PRODUCT_LE910                  0x1201
>>
>> @@ -627,6 +628,11 @@ static const struct option_blacklist_info telit_le922_blacklist_usbcfg3 = {
>>       .reserved = BIT(1) | BIT(2) | BIT(3),
>>  };
>>
>> +static const struct option_blacklist_info telit_le922_blacklist_usbcfg5 = {
>> +     .sendsetup = BIT(2),
>> +     .reserved = BIT(0) | BIT(1) | BIT(3) | BIT(8) | BIT(9) | BIT(10),
>> +};
>> +
>>  static const struct usb_device_id option_ids[] = {
>>       { USB_DEVICE(OPTION_VENDOR_ID, OPTION_PRODUCT_COLT) },
>>       { USB_DEVICE(OPTION_VENDOR_ID, OPTION_PRODUCT_RICOLA) },
>> @@ -1176,6 +1182,8 @@ static const struct usb_device_id option_ids[] = {
>>               .driver_info = (kernel_ulong_t)&telit_le922_blacklist_usbcfg0 },
>>       { USB_DEVICE(TELIT_VENDOR_ID, TELIT_PRODUCT_LE922_USBCFG3),
>>               .driver_info = (kernel_ulong_t)&telit_le922_blacklist_usbcfg3 },
>> +     { USB_DEVICE(TELIT_VENDOR_ID, TELIT_PRODUCT_LE922_USBCFG5),
>> +             .driver_info = (kernel_ulong_t)&telit_le922_blacklist_usbcfg5 },
>
> Please use USB_DEVICE_INTERFACE_CLASS to keep the blacklist entry
> shorter (perhaps you can then even reuse an existing one).

Thanks for the review!

I will submit a v2 patch.

Regards,
Daniele

>
> Thanks,
> Johan