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

From: Scott Talbert <redacted>
Date: Mon, 20 Feb 2012 19:50:57 -0500 (EST)

On Mon, 20 Feb 2012, David Miller wrote:

quoted

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.

Please freshly repost your patch with an updated commit message.