On Tue, 9 Jun 2015, Stefan Koch wrote:

> Hi
> 
> I think whitespaces should be ok now. I have tested now three mail
> clients...
> 
> This patch series enables the interface authorization. The description
> is improved now. Is the series correct now?

This patch series is much, much more complicated than necessary.

> The first patch allows to handle the probing of drivers
>  manually.

Why do you do that?  It's completely unrelated to the problem of
authorizing USB interfaces.

> The second patch unitizes the usb set configuration process.

Mostly it just moves some code around.  It also adds some debugging
statements -- why?  The whole patch looks pointless.

> Both, the first and the second patch are independent.
> 
> And the third patch introduces the usb interface authorization. This is
> the main concern of this kernel modification.

This patch also is more complicated than it should be.  Why do you want
to unregister and then re-register the interfaces?  All you need to do
is unbind drivers and prevent other drivers from binding.

Alan Stern

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

Reply via email to