From: =?gb2312?B?TWVuZyBaaGFuZyAo1cXDyCk=?= <Meng.Zhang@mediatek.com>
To: Alan Stern <stern@rowland.harvard.edu>
CC: Josua Dietze <digidietze@draisberghof.de>,
        Matthew Dharm <mdharm-usb@one-eyed-alien.net>,
        Phil Dibowitz <phil@ipom.com>, USB list <linux-usb@vger.kernel.org>,
        USB Storage list <usb-storage@lists.one-eyed-alien.net>,
        Kernel development list <linux-kernel@vger.kernel.org>
Date: Wed, 29 Feb 2012 08:25:43 +0800
Subject: RE: [PATCH] USB: Mass Storage: Add MediaTek MT6276M device entry
Thread-Topic: [PATCH] USB: Mass Storage: Add MediaTek MT6276M device entry
Thread-Index: Acz2Lg8g7y7/cNIQSqKett6b7EMcfwASKsCw
Message-ID: <0D3EFCAF29FAA54FB91286E54DF1085C23048506EF@MTKMBS30.mediatek.inc>
References: <4F4C998D.4000805@mediatek.com>
 <Pine.LNX.4.44L0.1202281028350.1456-100000@iolanthe.rowland.org>
In-Reply-To: <Pine.LNX.4.44L0.1202281028350.1456-100000@iolanthe.rowland.org>
Accept-Language: en-US
Content-Language: zh-CN
acceptlanguage: en-US
Content-Type: text/plain; charset="gb2312"
MIME-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Content-Transfer-Encoding: 8bit
Content-Length: 2031
Lines: 48

Hi Alan,

As far as I know, many other devices used this init function in unusual_devs.h for their "ZeroCD" feature, and I think this is more convenient than let device user install utilities like usb_modeswitch and input command manually. 

Thanks.

BR.
Meng Zhang

-----Original Message-----
From: linux-usb-owner@vger.kernel.org [mailto:linux-usb-owner@vger.kernel.org] On Behalf Of Alan Stern
Sent: Tuesday, February 28, 2012 11:31 PM
To: Meng Zhang (????)
Cc: Josua Dietze; Matthew Dharm; Phil Dibowitz; USB list; USB Storage list; Kernel development list
Subject: Re: [PATCH] USB: Mass Storage: Add MediaTek MT6276M device entry

On Tue, 28 Feb 2012, Meng Zhang wrote:

> from: Meng Zhang <meng.zhang@mediatek.com>
> 
> 1. Add entrys for mediatek MT6276M in unusual_devs.h
> 
> 2. Add init function for mediatek MT6276M to switch to COM port mode
> 
> 3. usb-devices outputs after running with this patch:
> T:  Bus=02 Lev=01 Prnt=01 Port=05 Cnt=01 Dev#= 48 Spd=480 MxCh= 0
> D:  Ver= 2.00 Cls=00(>ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs=  1
> P:  Vendor=0e8d ProdID=00a1 Rev=02.00
> S:  Manufacturer=MediaTek Inc
> S:  Product=Product
> C:  #Ifs= 3 Cfg#= 1 Atr=80 MxPwr=500mA
> I:  If#= 0 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=02 Prot=01 Driver=option
> I:  If#= 1 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
> I:  If#= 2 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=00 Prot=00 Driver=option
> 
> 4. patch based on branch:
> 	git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> 
> 
> Signed-off-by: Meng Zhang <meng.zhang@mediatek.com>

Nowadays, aren't such things handled in userspace by the usb_modeswitch program rather than in the kernel driver?

Alan Stern

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@vger.kernel.org More majordomo info at  http://vger.kernel.org/majordomo-info.html
????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m????????????I?