Re: [PATCH] cdc_ether/zaurus: Move Harmony 900 from cdc_ether to zaurus

On Mon, 20 Feb 2012, David Miller wrote:

From: Scott Talbert <redacted>
Date: Sun, 19 Feb 2012 20:35:29 -0500 (EST)

quoted

The Logitech Harmony 900 remote control appears to use the pseudo-MDLM
driver, rather than the standard one.  This patch simply moves the
device so that it gets picked up by the correct driver.

Signed-off-by: Scott Talbert <redacted>

You're not moving anything, you're adding the ID to not one but two
different drivers.

That doesn't make any sense to me, so either this patch is completely
wrong or you need to rewrite your commit message to explain things
better and more accurately.

Sorry, to clarify...in the current baseline, the Harmony 900 gets picked 
up by the cdc_ether driver by this generic line in cdc_ether.c:

 	USB_INTERFACE_INFO(USB_CLASS_COMM, USB_CDC_SUBCLASS_MDLM,
 			USB_CDC_PROTO_NONE),
 	.driver_info = (unsigned long)&wwan_info,

The intent of my patch is to blacklist the Harmony 900 from cdc_ether and 
whitelist it in zaurus.

Scott