Re: [PATCH v3 1/2] usb: Add quirk detection based on interface information

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

On Wed, 18 Jul 2012, Laurent Pinchart wrote:

> When a whole class of devices (possibly from a specific vendor, or
> across multiple vendors) require a quirk, explictly listing all devices
> in the class make the quirks table unnecessarily large. Fix this by
> allowing matching devices based on interface information.
> Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>

> --- a/drivers/usb/core/hub.c
> +++ b/drivers/usb/core/hub.c
> @@ -2170,6 +2170,8 @@ int usb_new_device(struct usb_device *udev)
>  	udev->dev.devt = MKDEV(USB_DEVICE_MAJOR,
>  			(((udev->bus->busnum-1) * 128) + (udev->devnum-1)));
> +	usb_detect_interface_quirks(udev);
> +

This is not good, because it ignores usb_authorize_device().  That's 
why I said earlier that the interface quirks should be detected at the 
end of usb_enumerate_device().

Alan Stern

To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at

B and H Foto and Electronics Corp.

[Linux Media]     [Video for Linux]     [Linux Input]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Free Online Dating]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]     [More Archives]

Add to Google Powered by Linux