> -----Original Message----- > From: Muralidharan Karicheri [mailto:[email protected]] > Sent: Wednesday, February 24, 2010 5:15 AM > To: Hiremath, Vaibhav > Cc: [email protected]; [email protected]; > [email protected]; [email protected]; > Karicheri, Muralidharan > Subject: Re: [PATCH 3/9] tvp514x: add YUYV format support > > Vaibhav, > > > On Mon, Jan 4, 2010 at 9:02 AM, <[email protected]> wrote: > > From: Vaibhav Hiremath <[email protected]> > > > > > > Signed-off-by: Vaibhav Hiremath <[email protected]> > > --- > > 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 [email protected] > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > > > > > -- > Murali Karicheri > [email protected] -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
