> -----Original Message----- > From: Laurent Pinchart [mailto:laurent.pinchart@xxxxxxxxxxxxxxxx] > Sent: Wednesday, October 06, 2010 3:49 PM > To: Hiremath, Vaibhav > Cc: linux-media@xxxxxxxxxxxxxxx; sakari.ailus@xxxxxxxxxxxxxxxxxxxxxxxxxx > Subject: Re: [PATCH/RFC v3 04/11] v4l: Add 8-bit YUYV on 16-bit bus and > SGRBG10 media bus pixel codes > > Hi Vaibhav, > > On Wednesday 06 October 2010 11:19:25 Hiremath, Vaibhav wrote: > > On Tuesday, October 05, 2010 7:55 PM Laurent Pinchart wrote: > > > > > > Add the following media bus format code definitions: > > > > > > - V4L2_MBUS_FMT_SGRBG10_1X10 for 10-bit GRBG Bayer > > > - V4L2_MBUS_FMT_SGRBG10_DPCM8_1X8 for 10-bit DPCM compressed GRBG > Bayer > > > - V4L2_MBUS_FMT_YUYV16_1X16 for 8-bit YUYV on 16-bit bus > > > - V4L2_MBUS_FMT_UYVY16_1X16 for 8-bit UYVY on 16-bit bus > > > - V4L2_MBUS_FMT_YVYU16_1X16 for 8-bit YVYU on 16-bit bus > > > - V4L2_MBUS_FMT_VYUY16_1X16 for 8-bit VYUY on 16-bit bus > > The commit message should list V4L2_MBUS_FMT_*8_1X16 instead of > V4L2_MBUS_FMT_*16_1X16, my bad. > > > > Laurent I may be wrong here, but I think above definition is confusing - > > > > For me the above definition actually means, 16bits are coming on the bus > > for every cycle. > > That's correct. > > > If you are referring to OMAP3 interface here then 8->16 bit conversion > > happens inside ISP-bridge, the interface from sensor-to-CCDC is still 8 > > bit (Technically it is 10, but we are using lane shifter to get 8 bits) > > and I believe sensor is also sending one component for every cycle > (either > > UYVY or YUYV). > > That's correct as well. > > > And I believe the bridge driver is not exported to user application so > we > > should be using MBUS_FMT_UYVY8_2x8 and family. > > The V4L2_MBUS_FMT_*8_1X16 formats are used for the preview -> resizer > link, > not the sensor -> CCDC link. For the later the V4L2_MBUS_FMT_*8_2X8 are > used > instead. > [Hiremath, Vaibhav] Ok. Understood & agreed. Thanks, Vaibhav > -- > Regards, > > Laurent Pinchart -- 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
