> -----Original Message----- > From: Muralidharan Karicheri [mailto:mkarich...@gmail.com] > Sent: Wednesday, February 24, 2010 5:15 AM > To: Hiremath, Vaibhav > Cc: linux-media@vger.kernel.org; linux-o...@vger.kernel.org; > hverk...@xs4all.nl; davinci-linux-open-sou...@linux.davincidsp.com; > Karicheri, Muralidharan > Subject: Re: [PATCH 3/9] tvp514x: add YUYV format support > > Vaibhav, > > > On Mon, Jan 4, 2010 at 9:02 AM, <hvaib...@ti.com> wrote: > > From: Vaibhav Hiremath <hvaib...@ti.com> > > > > > > Signed-off-by: Vaibhav Hiremath <hvaib...@ti.com> > > --- > > drivers/media/video/tvp514x.c | 7 +++++++ > > 1 files changed, 7 insertions(+), 0 deletions(-) > > > > diff --git a/drivers/media/video/tvp514x.c b/drivers/media/video/tvp514x.c > > index 4cf3593..b344b58 100644 > > --- a/drivers/media/video/tvp514x.c > > +++ b/drivers/media/video/tvp514x.c > > @@ -212,6 +212,13 @@ static const struct v4l2_fmtdesc tvp514x_fmt_list[] = > { > > .description = "8-bit UYVY 4:2:2 Format", > > .pixelformat = V4L2_PIX_FMT_UYVY, > > }, > > + { > > + .index = 1, > > + .type = V4L2_BUF_TYPE_VIDEO_CAPTURE, > > + .flags = 0, > > + .description = "8-bit YUYV 4:2:2 Format", > > + .pixelformat = V4L2_PIX_FMT_YUYV, > > + }, > > }; > > As per data sheet I can see only CbYCrY format output from the tvp5146 > which translate to UYVY. How are you configuring tvp to output YUYV? I > don;t see any change to the code to configure this format. > [Hiremath, Vaibhav] Yes you are right, actually this is dummy format created to support YUYV. > CCDC can switch the CbCr order and also can swap Y/C order. So if you > are achieving > this via ccdc configuration, there is no need to add this format to tvp5146 > IMO. > [Hiremath, Vaibhav] I think it makes sense to handle this is master driver, since we are handling this in CCDC. It could possible in the future TVP5146 might get used with SoC which don't have this capability.
Thanks, Vaibhav > -Murali > > > > > /** > > -- > > 1.6.2.4 > > > > -- > > 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 > > > > > > -- > Murali Karicheri > mkarich...@gmail.com -- 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
