Hi,

Greg KH wrote:
> On Mon, Feb 06, 2012 at 05:12:33PM +0900, Anton Tikhomirov wrote:
> > Cc: Kukjin Kim <kgene.kim <at> samsung.com>
> > Cc: Greg Kroah-Hartman <gregkh <at> suse.de>
> > Cc: Felipe Balbi <balbi <at> ti.com>
> 
> What is that mess?  It belongs, with real email addresses, below your
> signed-off-by line, if at all.
> 
> > Adds SET SEL standard request definition as defined by ch9
> > of the USB3.0 specification.
> >
> > Signed-off-by: Anton Tikhomirov <[email protected]>
> > ---
> >  include/linux/usb/ch9.h |    1 +
> >  1 files changed, 1 insertions(+), 0 deletions(-)
> >
> > diff --git a/include/linux/usb/ch9.h b/include/linux/usb/ch9.h
> > index 61b2905..76ff771 100644
> > --- a/include/linux/usb/ch9.h
> > +++ b/include/linux/usb/ch9.h
> > @@ -88,6 +88,7 @@
> >  #define USB_REQ_GET_INTERFACE              0x0A
> >  #define USB_REQ_SET_INTERFACE              0x0B
> >  #define USB_REQ_SYNCH_FRAME                0x0C
> > +#define USB_REQ_SET_SEL                    0x30
> >
> >  #define USB_REQ_SET_ENCRYPTION             0x0D    /* Wireless USB */
> >  #define USB_REQ_GET_ENCRYPTION             0x0E
> 
> Why did you insert this out of order?
> 
> greg "please note my email address change as well" k-h

'Set SEL' is a standard device request according to USB3.0 spec. Wireless USB 
requests are not mentioned
in USB3.0 spec (and moreover are not in ch9 of it). So I put the definition to 
an appropriate group in my opinion.
If you insist I will move the definition below Wireless USB group with a space 
between.

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

Reply via email to