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. > s->vdev->fops = ivtv_stream_info[type].fops; > s->vdev->ctrl_handler = itv->v4l2_dev.ctrl_handler; > s->vdev->release = video_device_release; -- Regards, Laurent Pinchart -- 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
