Em 21-05-2011 03:05, stefan.ringel@xxxxxxxx escreveu:
> From: Stefan Ringel <stefan.ringel@xxxxxxxx>
>
> fix uninitialized field, change prink to dprintk
Applied, thanks.
A quick note: please don't repeat the patch subject at the
first line of the body. It requires me to manually drop it
when importing the patch. Instead, put an useful description
about why and how are you doing such change.
Thanks,
Mauro.
>
>
> Signed-off-by: Stefan Ringel <stefan.ringel@xxxxxxxx>
> ---
> drivers/staging/tm6000/tm6000-usb-isoc.h | 2 +-
> drivers/staging/tm6000/tm6000-video.c | 5 ++++-
> 2 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/tm6000/tm6000-usb-isoc.h b/drivers/staging/tm6000/tm6000-usb-isoc.h
> index a9e61d9..084c2a8 100644
> --- a/drivers/staging/tm6000/tm6000-usb-isoc.h
> +++ b/drivers/staging/tm6000/tm6000-usb-isoc.h
> @@ -39,7 +39,7 @@ struct usb_isoc_ctl {
> int pos, size, pktsize;
>
> /* Last field: ODD or EVEN? */
> - int vfield;
> + int vfield, field;
>
> /* Stores incomplete commands */
> u32 tmp_buf;
> diff --git a/drivers/staging/tm6000/tm6000-video.c b/drivers/staging/tm6000/tm6000-video.c
> index 4802396..4264064 100644
> --- a/drivers/staging/tm6000/tm6000-video.c
> +++ b/drivers/staging/tm6000/tm6000-video.c
> @@ -334,6 +334,7 @@ static int copy_streams(u8 *data, unsigned long len,
> size = dev->isoc_ctl.size;
> pos = dev->isoc_ctl.pos;
> pktsize = dev->isoc_ctl.pktsize;
> + field = dev->isoc_ctl.field;
> }
> cpysize = (endp - ptr > size) ? size : endp - ptr;
> if (cpysize) {
> @@ -359,7 +360,8 @@ static int copy_streams(u8 *data, unsigned long len,
> /* Need some code to copy pts */
> u32 pts;
> pts = *(u32 *)ptr;
> - printk(KERN_INFO "%s: field %d, PTS %x", dev->name, field, pts);
> + dprintk(dev, V4L2_DEBUG_ISOC, "field %d, PTS %x",
> + field, pts);
> break;
> }
> }
> @@ -371,6 +373,7 @@ static int copy_streams(u8 *data, unsigned long len,
> dev->isoc_ctl.pos = pos + cpysize;
> dev->isoc_ctl.size = size - cpysize;
> dev->isoc_ctl.cmd = cmd;
> + dev->isoc_ctl.field = field;
> dev->isoc_ctl.pktsize = pktsize - (endp - ptr);
> ptr += endp - ptr;
> } else {
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
