On Thu 13 September 2012 04:37:27 Laurent Pinchart wrote: > Hi Hans, > > Thanks for the patch. > > On Friday 07 September 2012 15:29:25 Hans Verkuil wrote: > > From: Hans Verkuil <hans.verkuil@xxxxxxxxx> > > > > Signed-off-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> > > --- > > drivers/media/pci/ivtv/ivtv-streams.c | 3 +++ > > drivers/media/pci/zoran/zoran_card.c | 4 ++++ > > drivers/media/platform/coda.c | 1 + > > drivers/media/platform/davinci/vpbe_display.c | 1 + > > drivers/media/platform/davinci/vpif_display.c | 1 + > > drivers/media/platform/m2m-deinterlace.c | 1 + > > drivers/media/platform/mem2mem_testdev.c | 1 + > > drivers/media/platform/mx2_emmaprp.c | 1 + > > drivers/media/platform/omap/omap_vout.c | 1 + > > drivers/media/platform/omap3isp/ispvideo.c | 1 + > > drivers/media/platform/s5p-fimc/fimc-m2m.c | 1 + > > drivers/media/platform/s5p-g2d/g2d.c | 1 + > > drivers/media/platform/s5p-jpeg/jpeg-core.c | 1 + > > drivers/media/platform/s5p-mfc/s5p_mfc.c | 1 + > > drivers/media/platform/s5p-tv/mixer_video.c | 1 + > > drivers/media/platform/sh_vou.c | 1 + > > drivers/media/usb/uvc/uvc_driver.c | 2 ++ > > 17 files changed, 23 insertions(+) > > > > diff --git a/drivers/media/pci/ivtv/ivtv-streams.c > > b/drivers/media/pci/ivtv/ivtv-streams.c index f08ec17..1d0e04a 100644 > > --- a/drivers/media/pci/ivtv/ivtv-streams.c > > +++ b/drivers/media/pci/ivtv/ivtv-streams.c > > @@ -223,6 +223,9 @@ static int ivtv_prep_dev(struct ivtv *itv, int type) > > > > s->vdev->num = num; > > s->vdev->v4l2_dev = &itv->v4l2_dev; > > + if (ivtv_stream_info[type].buf_type == V4L2_BUF_TYPE_VIDEO_OUTPUT || > > + ivtv_stream_info[type].buf_type == V4L2_BUF_TYPE_VBI_OUTPUT) > > + s->vdev->vfl_dir = VFL_DIR_TX; > > I think drivers should set VFL_DIR_RX explicitly instead of relying on it > being equal to 0. If we change the value later for any reason this > implementation would break. I can do that for those drivers like ivtv where you have output and input devices, but I'm not going to change this for all drivers, because that would be a substantial amount of work for little gain. Regards, Hans -- 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