Re: [PATCH] cdc-acm: Drop the warning for unusual capabilities

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Oct 28, 2014 at 10:12:08AM +0000, Karl Palsson wrote:
> On Tue, Oct 28, 2014 at 10:20:29AM +0100, Oliver Neukum wrote:
> > Let's drop the warning for modems with unusual capabilities,
> > the associated quirk and blacklist. They made little sense.
> > 
> [snip]
> > @@ -1796,11 +1794,6 @@ static const struct usb_device_id acm_ids[] = {
> >  
> >  	/* NOTE: non-Nokia COMM/ACM/0xff is likely MSFT RNDIS... NOT a modem! */
> >  
> > -	/* Support Lego NXT using pbLua firmware */
> > -	{ USB_DEVICE(0x0694, 0xff00),
> > -	.driver_info = NOT_A_MODEM,
> > -	},
> > -
> 
> Isn't this dropping the device too, not just the quirk?

Depending on what its descriptors look like, you may be right. The commit
introducing the quirk:

	ce126644aa10 ("USB: cdc_acm: Silence "It is not a modem." error for pbLua devices")

looks like if it was just silencing the error message of a device that
was already recognised by the driver, though.

Johan
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux