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.ha...@ti.com>
> Signed-off-by: Muralidharan Karicheri <m-kariche...@ti.com>
> ---
>  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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to