On Friday 31 July 2009 01:05:04 Patch from Jean-Francois Moine wrote:
> The patch number 12354 was added via Jean-Francois Moine <[email protected]>
> to http://linuxtv.org/hg/v4l-dvb master development tree.
>
> Kernel patches in this development tree may be modified to be backward
> compatible with older kernels. Compatibility modifications will be
> removed before inclusion into the mainstream Kernel
>
> If anyone has any objections, please let us know by sending a message to:
> Linux Media Mailing List <[email protected]>
>
> ------
>
> From: Jean-Francois Moine <[email protected]>
> gspca - vc032x: H and V flip controls added for mi13x0_soc sensors.
>
>
> Also, H/V flip default values adjusted according to the webcam IDs.
>
> Priority: normal
>
> Signed-off-by: Jean-Francois Moine <[email protected]>
>
>
> ---
>
> linux/drivers/media/video/gspca/vc032x.c | 109 +++++++++++++----------
> 1 file changed, 63 insertions(+), 46 deletions(-)
>
> diff -r c9c025650ce7 -r 266dc538f544 linux/drivers/media/video/gspca/vc032x.c
> --- a/linux/drivers/media/video/gspca/vc032x.c Mon Jul 27 10:52:27
> 2009 +0200
> +++ b/linux/drivers/media/video/gspca/vc032x.c Mon Jul 27 11:00:03
> 2009 +0200
> @@ -3121,33 +3127,44 @@
> return 0;
> }
>
> -/* for OV7660 and OV7670 only */
> +/* some sensors only */
> static void sethvflip(struct gspca_dev *gspca_dev)
> {
> struct sd *sd = (struct sd *) gspca_dev;
> - __u8 data;
> + u8 data[2], hflip, vflip;
>
> + hflip = sd->hflip;
> + if (sd->flags & FL_HFLIP)
> + hflip != hflip;
> + vflip = sd->vflip;
> + if (sd->flags & FL_VFLIP)
> + vflip != vflip;
Hi Jean-Francois,
The daily build produces this warning:
/marune/build/v4l-dvb-master/v4l/vc032x.c: In function 'sethvflip':
/marune/build/v4l-dvb-master/v4l/vc032x.c:3138: warning: statement with no
effect
/marune/build/v4l-dvb-master/v4l/vc032x.c:3141: warning: statement with no
effect
And looking at the code those warnings are correct. I think you wanted to do
'hflip = !hflip'.
Can you take a look at this?
Thanks,
Hans
--
Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
