Re: [PATCH 2/3] USB: SET SEL request definition

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

 



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 <av.tikhomirov@xxxxxxxxxxx>
> > ---
> >  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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux