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.verk...@cisco.com>
> >
> > Signed-off-by: Hans Verkuil <hans.verk...@cisco.com>
> > ---
> > 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 majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
