Re: [v4l-dvb-maintainer] [PATCH] flexcop-usb.c: fix "&& 0xff" typosand ...

[Michael Krufky]

Alexey Dobriyan wrote:
> .. fix debug printk. Why, oh why, one would want to do
> 
> 	(u16 & 0xff) << 8
> 
> and print it with %02x format?
> 
> Signed-off-by: Alexey Dobriyan <adobriyan@xxxxxxxxx>

I spoke to Patrick in irc and confirmed the fix...

Acked-by: Patrick Boettcher <pb@xxxxxxxxxxx>
Signed-off-by: Michael Krufky <mkrufky@xxxxxxxxxxx>

> ---
> 
>  drivers/media/dvb/b2c2/flexcop-usb.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --- a/drivers/media/dvb/b2c2/flexcop-usb.c
> +++ b/drivers/media/dvb/b2c2/flexcop-usb.c
> @@ -246,7 +246,7 @@ static int flexcop_usb_i2c_req(struct fl
>  	wIndex = (chipaddr << 8 ) | addr;
>  
>  	deb_i2c("i2c %2d: %02x %02x %02x %02x %02x %02x\n",func,request_type,req,
> -			((wValue && 0xff) << 8),wValue >> 8,((wIndex && 0xff) << 8),wIndex >> 8);
> +		wValue & 0xff, wValue >> 8, wIndex & 0xff, wIndex >> 8);
>  
>  	len = usb_control_msg(fc_usb->udev,pipe,
>  			req,
> 
> 
> _______________________________________________
> v4l-dvb-maintainer mailing list
> v4l-dvb-maintainer@xxxxxxxxxxx
> http://www.linuxtv.org/cgi-bin/mailman/listinfo/v4l-dvb-maintainer


-- 
Michael Krufky

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/