Re: [PATCH] USB: core: remove CONFIG_USB_DEBUG usage

2013-12-20 Thread Greg Kroah-Hartman
On Fri, Dec 20, 2013 at 09:47:53PM -0500, Alan Stern wrote: > On Fri, 20 Dec 2013, Greg Kroah-Hartman wrote: > > > > > /* Check that the pipe's type matches the endpoint's type */ > > > > if (usb_pipetype(urb->pipe) != pipetypes[xfertype]) > > > > > > It looks as though it ought t

Re: [PATCH] USB: core: remove CONFIG_USB_DEBUG usage

2013-12-20 Thread Alan Stern
On Fri, 20 Dec 2013, Greg Kroah-Hartman wrote: > > > /* Check that the pipe's type matches the endpoint's type */ > > > if (usb_pipetype(urb->pipe) != pipetypes[xfertype]) > > > > It looks as though it ought to be possible to make that check: > > if (unlikely(xfertype != urb->pipe->valid_

RE: [PATCH] USB: core: remove CONFIG_USB_DEBUG usage

2013-12-20 Thread David Laight
> From: Greg Kroah-Hartman ... > > ... > > > /* Check that the pipe's type matches the endpoint's type */ > > > if (usb_pipetype(urb->pipe) != pipetypes[xfertype]) > > > > It looks as though it ought to be possible to make that check: > > if (unlikely(xfertype != urb->pipe->valid_xfertype))

Re: [PATCH] USB: core: remove CONFIG_USB_DEBUG usage

2013-12-20 Thread Greg Kroah-Hartman
On Fri, Dec 20, 2013 at 10:04:00AM -, David Laight wrote: > > From: Greg Kroah-Hartman > > CONFIG_USB_DEBUG is going away, so remove the few places in the USB core > > that relied on them. > > > > This means that we always now do the "debug" checks for every urb > > submitted, which is a good

RE: [PATCH] USB: core: remove CONFIG_USB_DEBUG usage

2013-12-20 Thread David Laight
> From: Greg Kroah-Hartman > CONFIG_USB_DEBUG is going away, so remove the few places in the USB core > that relied on them. > > This means that we always now do the "debug" checks for every urb > submitted, which is a good idea, as who knows how many driver bugs we > have been ignoring when peopl