On Thursday, December 02, 2010 13:39:07 Manjunath Hadli wrote: > This patch adds the VENC or the Video encoder, whichis responsible > for the blending of al source planes and timing generation for Video > modes like NTSC, PAL and other digital outputs. the VENC implementation > currently supports COMPOSITE and COMPONENT outputs and NTSC and PAL > resolutions through the analog DACs. The venc block is implemented > as a subdevice, allowing for additional extenal and internal encoders > of other kind to plug-in. > > Signed-off-by: Manjunath Hadli <manjunath.hadli@xxxxxx> > Signed-off-by: Muralidharan Karicheri <m-karicheri2@xxxxxx> > --- > drivers/media/video/davinci/vpbe_venc.c | 574 ++++++++++++++++++++++++++ > drivers/media/video/davinci/vpbe_venc_regs.h | 189 +++++++++ > include/media/davinci/vpbe_venc.h | 38 ++ > 3 files changed, 801 insertions(+), 0 deletions(-) > create mode 100644 drivers/media/video/davinci/vpbe_venc.c > create mode 100644 drivers/media/video/davinci/vpbe_venc_regs.h > create mode 100644 include/media/davinci/vpbe_venc.h > > diff --git a/drivers/media/video/davinci/vpbe_venc.c b/drivers/media/video/davinci/vpbe_venc.c > new file mode 100644 > index 0000000..bf30332 > --- /dev/null > +++ b/drivers/media/video/davinci/vpbe_venc.c <snip> > +static int venc_s_std_output(struct v4l2_subdev *sd, v4l2_std_id norm) > +{ > + int ret = 0; > + > + v4l2_dbg(debug, 1, sd, "venc_s_std_output\n"); > + > + if (norm & V4L2_STD_NTSC) > + ret = venc_set_ntsc(sd); > + else if (norm & V4L2_STD_PAL) Shouldn't this be V4L2_STD_525_60 and V4L2_STD_625_50 instead of NTSC/PAL? > + ret = venc_set_pal(sd); > + else > + ret = -EINVAL; > + return ret; > +} Regards, Hans -- Hans Verkuil - video4linux developer - sponsored by Cisco -- 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