From: Greg KH <greg@kroah.com>
To: fangxiaozhi 00110321 <huananhu@huawei.com>
Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
zihan@huawei.com, zhangqianchun@huawei.com
Subject: Re: PATCH 10/8]linux-usb: To support more Huawei data card product IDs, fix kernel 2.6.27-rc7
Date: Fri, 10 Oct 2008 14:15:59 -0700 [thread overview]
Message-ID: <20081010211559.GC17701@kroah.com> (raw)
In-Reply-To: <f9029a2044fb8.44fb8f9029a20@huawei.com>
On Wed, Oct 08, 2008 at 12:02:46PM +0800, fangxiaozhi 00110321 wrote:
> From: fangxiaozhi <huananhu@huawei.com>
>
> 1. This patch is based on the kernel of 2.6.27-rc7
> 2. 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 to declare the unusual device for new Huawei data card devices
> in unusual_devs.h and to declare more new product IDs in option.c.
> 3. To modify the data value and length in the function of
> usb_stor_huawei_e220_init in initializers.c
> That's because based on 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.
Can you make the changes Alan suggested?
I'll hold off for a new version after you have integrated that change.
thanks,
greg k-h
next prev parent reply other threads:[~2008-10-10 21:23 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-08 4:02 PATCH 10/8]linux-usb: To support more Huawei data card product IDs, fix kernel 2.6.27-rc7 fangxiaozhi 00110321
2008-10-10 21:15 ` Greg KH [this message]
2008-10-11 4:20 ` PATCH 10/8]linux-usb: To support more Huawei data card productIDs, " Franko Fang
2008-10-11 13:58 ` Alan Stern
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20081010211559.GC17701@kroah.com \
--to=greg@kroah.com \
--cc=huananhu@huawei.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=zhangqianchun@huawei.com \
--cc=zihan@huawei.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox