linux-kernel - Re: PATCH 10/4]linux-usb: To support more Huawei data card productIDs, fix kernel 2.6.27-rc7

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] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 25 Sep 2008 09:36:29 +0800
From:	Franko Fang <huananhu@...wei.com>
To:	Greg KH <greg@...ah.com>
Cc:	linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org,
	zihan@...wei.com, Lin.Lei@...wei.com, zhangqianchun@...wei.com
Subject: Re: PATCH 10/4]linux-usb: To support more Huawei data card productIDs,
 fix kernel 2.6.27-rc7

1. In this patch, we want to do one thing: add more Huawei product IDs into the USB driver. Then it can support  more Huawei data card devices. 
    So I make them in one patch.

2. To change usb_stor_huawei_e220_init function as follows, that's because in the USB standard, while sending SET_FETURE_D to the device, it requires the corresponding data to be zero, and its sending length also must be zero.
    In our old solution, it can be compatible with our WCDMA data card devices, but can not support our CDMA data card devices. 
    But in this new solution, it can be compatible with all of our data card devices.

    Hope that my interpreting can satify you.
-----------------------------
>> --- linux-2.6.27-rc7-orig/drivers/usb/storage/initializers.c 2008-09-22 06:29:55.000000000 +0800
>> +++ linux-2.6.27-rc7/drivers/usb/storage/initializers.c 2008-09-24 12:04:40.000000000 +0800
>> @@ -95,11 +95,11 @@ int usb_stor_huawei_e220_init(struct us_
>>  {
>>  int result;
>>  
>> - us->iobuf[0] = 0x1;
>> + us->iobuf[0] = 0x0;
>>  result = usb_stor_control_msg(us, us->send_ctrl_pipe,
>>        USB_REQ_SET_FEATURE,
>>        USB_TYPE_STANDARD | USB_RECIP_DEVICE,
>> -       0x01, 0x0, us->iobuf, 0x1, 1000);
>> +       0x01, 0x0, us->iobuf, 0x0, 1000);
-------------------------------------------------------------------------

Franko Fang
2008-9-25



----- Original Message ----- 
From: "Greg KH" <greg@...ah.com>
To: "fangxiaozhi 00110321" <huananhu@...wei.com>
Cc: <linux-usb@...r.kernel.org>; <linux-kernel@...r.kernel.org>; <zihan@...wei.com>; <Lin.Lei@...wei.com>; <zhangqianchun@...wei.com>
Sent: Wednesday, September 24, 2008 11:03 PM
Subject: Re: PATCH 10/4]linux-usb: To support more Huawei data card productIDs, fix kernel 2.6.27-rc7


> On Wed, Sep 24, 2008 at 02:08:12PM +0800, fangxiaozhi 00110321 wrote:
>> 
>>  From: fangxiaozhi <huananhu@...wei.com>
>>  
>>  1. This patch is based on the kernel of 2.6.27-rc7 2. To declare the unusual device for new Huawei data card devices 
>>  in unusual_devs.h
>>  3. To modify the data value and length in the function of 
>>  usb_stor_huawei_e220_init in initializers.c
>>  4. To declare more new product IDs in option.c.
> 
> I thought I asked you to split this up into 4 different patches?  Hm,
> maybe three should be all that is needed, I thought your original one
> did 4 different things.
> 
> And isn't your modification of the usb_stor_huawei_e220_init function
> going to break the devices that it currently works for?  Or has it
> always been wrong?
> 
> thanks,
> 
> greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/