Re: [PATCH] Staging: comedi: Use USB_ENDPOINT_DIR_MASK instead of USB_DIR_IN

[Greg KH <gregkh@linuxfoundation.org>]

On Tue, Sep 20, 2016 at 12:30:19AM +0530, Sandhya Bankar wrote:
> USB_ENDPOINT_DIR_MASK and USB_DIR_IN have the same value, 
> but USB_ENDPOINT_DIR_MASK is more correct for use as a mask.
> 
> Signed-off-by: Sandhya Bankar <bankarsandhya512@gmail.com>
> ---
>  drivers/staging/comedi/drivers/dt9812.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/staging/comedi/drivers/dt9812.c b/drivers/staging/comedi/drivers/dt9812.c
> index 7ebca86..40a7d97 100644
> --- a/drivers/staging/comedi/drivers/dt9812.c
> +++ b/drivers/staging/comedi/drivers/dt9812.c
> @@ -676,7 +676,7 @@ static int dt9812_find_endpoints(struct comedi_device *dev)
>  			dir = USB_DIR_IN;
>  			break;
>  		}
> -		if ((ep->bEndpointAddress & USB_DIR_IN) != dir) {
> +		if ((ep->bEndpointAddress & USB_ENDPOINT_DIR_MASK) != dir) {

This is messy, and I don't think that this code will ever "normally" be
triggered.  Can you rewrite this loop to properly check for the endpoint
type using the functions you used last time, and not trigger off of the
endpoint number?  That would be a much cleaner and saner fix for this.

thanks,

greg k-h