Hi Hans, Thanks for the patch! On Fri, Mar 15, 2013 at 3:57 PM, Hans Verkuil <hverkuil@xxxxxxxxx> wrote: > From: Hans Verkuil <hans.verkuil@xxxxxxxxx> > > This ioctl is defined as IOW, so pass the argument by value instead of by > reference. I could have chosen to add const instead, but this is 1) easier > to handle in drivers and 2) consistent with the s_std subdev operation. > > Signed-off-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> > --- [snip] > drivers/media/platform/davinci/vpbe.c | 8 ++++---- > drivers/media/platform/davinci/vpbe_display.c | 2 +- > drivers/media/platform/davinci/vpfe_capture.c | 12 ++++++------ > drivers/media/platform/davinci/vpif_capture.c | 6 +++--- > drivers/media/platform/davinci/vpif_display.c | 10 +++++----- [snip] > drivers/staging/media/davinci_vpfe/vpfe_video.c | 6 +++--- For the above Acked-by: Lad, Prabhakar <prabhakar.csengg@xxxxxxxxx> Cheers, --Prabhakar Lad http://www.linkedin.com/pub/prabhakar-lad/19/92b/955 -- 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
