From patchwork Wed Jul 18 11:05:18 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Laurent Pinchart X-Patchwork-Id: 1210441 Return-Path: X-Original-To: patchwork-davinci@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from bear.ext.ti.com (bear.ext.ti.com [192.94.94.41]) by patchwork1.kernel.org (Postfix) with ESMTP id 4BA223FD4F for ; Wed, 18 Jul 2012 11:07:18 +0000 (UTC) Received: from dlelxv30.itg.ti.com ([172.17.2.17]) by bear.ext.ti.com (8.13.7/8.13.7) with ESMTP id q6IB5Iw4016216; Wed, 18 Jul 2012 06:05:18 -0500 Received: from DLEE74.ent.ti.com (dlee74.ent.ti.com [157.170.170.8]) by dlelxv30.itg.ti.com (8.13.8/8.13.8) with ESMTP id q6IB5Iw0012896; Wed, 18 Jul 2012 06:05:18 -0500 Received: from dlelxv24.itg.ti.com (172.17.1.199) by DLEE74.ent.ti.com (157.170.170.8) with Microsoft SMTP Server id 14.1.323.3; Wed, 18 Jul 2012 06:05:17 -0500 Received: from linux.omap.com (dlelxs01.itg.ti.com [157.170.227.31]) by dlelxv24.itg.ti.com (8.13.8/8.13.8) with ESMTP id q6IB5Hpu000733; Wed, 18 Jul 2012 06:05:17 -0500 Received: from linux.omap.com (localhost [127.0.0.1]) by linux.omap.com (Postfix) with ESMTP id 34E1D80627; Wed, 18 Jul 2012 06:05:17 -0500 (CDT) X-Original-To: davinci-linux-open-source@linux.davincidsp.com Delivered-To: davinci-linux-open-source@linux.davincidsp.com Received: from dflp51.itg.ti.com (dflp51.itg.ti.com [128.247.22.94]) by linux.omap.com (Postfix) with ESMTP id 29E1680626 for ; Wed, 18 Jul 2012 06:05:16 -0500 (CDT) Received: from neches.ext.ti.com (neches.ext.ti.com [192.91.81.29]) by dflp51.itg.ti.com (8.13.7/8.13.8) with ESMTP id q6IB5GgS020916 for ; Wed, 18 Jul 2012 06:05:16 -0500 (CDT) Received: from psmtp.com (na3sys009amx226.postini.com [74.125.149.110]) by neches.ext.ti.com (8.13.7/8.13.7) with SMTP id q6IB5Eu2027582 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO) for ; Wed, 18 Jul 2012 06:05:15 -0500 Received: from perceval.ideasonboard.com ([95.142.166.194]) (using TLSv1) by na3sys009amx226.postini.com ([74.125.148.10]) with SMTP; Wed, 18 Jul 2012 11:05:15 GMT Received: from avalon.localnet (unknown [91.178.171.146]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 77EE7359D1; Wed, 18 Jul 2012 13:05:12 +0200 (CEST) From: Laurent Pinchart To: "Hadli, Manjunath" Subject: Re: [PATCH v4 1/2] media: add new mediabus format enums for dm365 Date: Wed, 18 Jul 2012 13:05:18 +0200 Message-ID: <41958950.qGmmsSpAPM@avalon> User-Agent: KMail/4.8.3 (Linux/3.3.8-gentoo; KDE/4.8.3; x86_64; ; ) In-Reply-To: References: <1333102154-24657-1-git-send-email-manjunath.hadli@ti.com> <1521995.bdrhyBupKO@avalon> MIME-Version: 1.0 X-pstn-neptune: 0/0/0.00/0 X-pstn-levels: (S:99.90000/99.90000 CV:99.9000 FC:95.5390 LC:95.5390 R:95.9108 P:95.9108 M:97.0282 C:98.6951 ) X-pstn-dkim: 0 skipped:not-enabled X-pstn-settings: 2 (0.5000:0.5000) s cv gt3 gt2 gt1 r p m c X-pstn-addresses: from [82/3] CC: Hans Verkuil , dlos , Sakari Ailus , LMML X-BeenThere: davinci-linux-open-source@linux.davincidsp.com X-Mailman-Version: 2.1.12 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: Errors-To: davinci-linux-open-source-bounces@linux.davincidsp.com Hi Manjunath, On Tuesday 17 July 2012 12:22:42 Hadli, Manjunath wrote: > On Tue, Jul 17, 2012 at 17:25:42, Laurent Pinchart wrote: > > On Tuesday 17 July 2012 11:41:11 Hadli, Manjunath wrote: > > > On Tue, Jul 17, 2012 at 16:26:24, Laurent Pinchart wrote: > > > > On Friday 30 March 2012 10:09:13 Hadli, Manjunath wrote: > > > > > add new enum entries for supporting the media-bus formats on dm365. > > > > > These include some bayer and some non-bayer formats. > > > > > V4L2_MBUS_FMT_YDYC8_1X16 and V4L2_MBUS_FMT_UV8_1X8 are used > > > > > internal to the hardware by the resizer. > > > > > V4L2_MBUS_FMT_SBGGR10_ALAW8_1X8 represents the bayer ALAW format > > > > > that is supported by dm365 hardware. > > > > > > > > > > Signed-off-by: Manjunath Hadli > > > > > Cc: Laurent Pinchart > > > > > Cc: Sakari Ailus > > > > > Cc: Hans Verkuil > > > > > --- > > > > > > > > > > Documentation/DocBook/media/v4l/subdev-formats.xml | 171 > > > > > ++++++++++++ > > > > > include/linux/v4l2-mediabus.h | 10 +- > > > > > 2 files changed, 179 insertions(+), 2 deletions(-) > > > > > > > > > > diff --git a/Documentation/DocBook/media/v4l/subdev-formats.xml > > > > > b/Documentation/DocBook/media/v4l/subdev-formats.xml index > > > > > 49c532e..48d92bb > > > > > 100644 > > > > > --- a/Documentation/DocBook/media/v4l/subdev-formats.xml > > > > > +++ b/Documentation/DocBook/media/v4l/subdev-formats.xml > > > > [snip] > > > > > > > @@ -965,6 +1036,56 @@ > > > > > > > > > > y1 > > > > > y0 > > > > > > > > > > > > > > > > > > > > + > > > > > > > > That's a weird one. Just out of curiosity, what's the point of > > > > transferring chroma information without luma ? > > > > > > DM365 supports this format. > > > > Right, but what is it used for ? > > Sorry about that. The Resizer in Dm365 can take only chroma and resize the > buffer. It can also take luma of course. In general it can take UV8, Y8 and > also UYVY. So UV8 is used to resize an NV buffer in two passes (first Y8 then UV8) ? > > [snip] > > > > > > > @@ -2415,6 +2536,56 @@ > > > > > > > > > > u1 > > > > > u0 > > > > > > > > > > > > > > > > > > > > + > > > > > > > > What is this beast ? We at least need a textual description, as I have > > > > no > > > > idea what the format corresponds to. > > > > > > This was discussed earlier over here > > > http://patchwork.linuxtv.org/patch/8843/ > > > > My bad, I should have remembered that. Please add a textual description of > > the format, it's not clear from the name what D and C are. > > I see no description for individual MBUS formats but a collective para on > everything together. Would you like me to add in the same or otherwise can > you point to me where I can add this description? What about the following patch ? Note that I've renamed YDYC to YDYU, as we might later need a YDYV format. (Hans, Sakari, any opinion ?) diff --git a/Documentation/DocBook/media/v4l/subdev-formats.xml b/Documentation/DocBook/media/v4l/subdev-formats.xml index 49c532e..47a485e 100644 --- a/Documentation/DocBook/media/v4l/subdev-formats.xml +++ b/Documentation/DocBook/media/v4l/subdev-formats.xml @@ -853,10 +853,15 @@ Packed YUV Formats Those data formats transfer pixel data as (possibly downsampled) Y, U - and V components. The format code is made of the following information. + and V components. Some formats include dummy bits in some of their samples + and are collectively referred to as "YDYC" (Y-Dummy-Y-Chroma) formats. + + + The format code is made of the following information. The Y, U and V components order code, as transferred on the - bus. Possible values are YUYV, UYVY, YVYU and VYUY. + bus. Possible values are YUYV, UYVY, YVYU and VYUY for formats with no + dummy bit, and YDYU, YDYV, YUYD and YVYD for YDYC formats. The number of bits per pixel component. All components are transferred on the same number of bits. Common values are 8, 10 and 12. @@ -877,7 +882,21 @@ U, Y, V, Y order will be named V4L2_MBUS_FMT_UYVY8_2X8. - The following table lisst existing packet YUV formats. + list existing packet YUV + formats and describes the organization of each pixel data in each sample. + When a format pattern is split across multiple samples each of the samples + in the pattern is described. + + The role of each bit transferred over the bus is identified by one + of the following codes. + + + yx for luma component bit number x + ux for blue chroma component bit number x + vx for red chroma component bit number x + - for non-available bits (for positions higher than the bus width) + / for dummy bits + YUV Formats @@ -2415,6 +2434,106 @@ u1u0 + + V4L2_MBUS_FMT_YDYU8_1X16 + 0x2014 + + - + - + - + - + y7 + y6 + y5 + y4 + y3 + y2 + y1 + y0 + / + / + / + / + / + / + / + / + + + + + + - + - + - + - + y7 + y6 + y5 + y4 + y3 + y2 + y1 + y0 + u7 + u6 + u5 + u4 + u3 + u2 + u1 + u0 + + + + + + - + - + - + - + y7 + y6 + y5 + y4 + y3 + y2 + y1 + y0 + / + / + / + / + / + / + / + / + + + + + + - + - + - + - + y7 + y6 + y5 + y4 + y3 + y2 + y1 + y0 + v7 + v6 + v5 + v4 + v3 + v2 + v1 + v0 + V4L2_MBUS_FMT_YUYV10_1X20 0x200d