Jan Dumon wrote:
quoted text
> Paulius Zaleckas wrote:
>> Jan Dumon wrote:
>>> From: Jan Dumon <j.dumon@option.com>
>>>
>>> Add Vendor/Product ID's for new devices.
>>>
>>> Signed-off-by: Jan Dumon <j.dumon@option.com>
>>> ---
>>> diff --git a/drivers/net/usb/hso.c b/drivers/net/usb/hso.c
>>> index fe98aca..d47c508 100644
>>> --- a/drivers/net/usb/hso.c
>>> +++ b/drivers/net/usb/hso.c
>>> @@ -462,9 +462,16 @@ static const struct usb_device_id hso_ids[] = {
>>>  	{USB_DEVICE(0x0af0, 0x7701)},
>>>  	{USB_DEVICE(0x0af0, 0x7801)},
>>>  	{USB_DEVICE(0x0af0, 0x7901)},
>>> -	{USB_DEVICE(0x0af0, 0x7361)},
>>> -	{USB_DEVICE(0x0af0, 0xd057)},
>> Please add comment in description why you are removing these IDs.
> 
> No ID's are removed.
> 0xd057 is added again by the same patch, 0x7361 was twice in the list.

OK for 0xd057, but please add comment that you are removing duplicate
definition of 0x7361.

quoted text
> 
>>> +	{USB_DEVICE(0x0af0, 0x8200)},
>>> +	{USB_DEVICE(0x0af0, 0x8201)},
>>> +	{USB_DEVICE(0x0af0, 0xd035)},
>>>  	{USB_DEVICE(0x0af0, 0xd055)},
>>> +	{USB_DEVICE(0x0af0, 0xd155)},
>>> +	{USB_DEVICE(0x0af0, 0xd255)},
>>> +	{USB_DEVICE(0x0af0, 0xd057)},
>>> +	{USB_DEVICE(0x0af0, 0xd157)},
>>> +	{USB_DEVICE(0x0af0, 0xd257)},
>>> +	{USB_DEVICE(0x0af0, 0xd357)},
>>>  	{}
>>>  };
>>>  MODULE_DEVICE_TABLE(usb, hso_ids);
> 
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html