Re: [PATCH v3 2/2] v4l2: add new pixel formats supported on dm365

2012-03-06 Thread 'Sakari Ailus' 
Hi Manju,

On Mon, Mar 05, 2012 at 07:45:18AM +, Hadli, Manjunath wrote:
 On Sun, Mar 04, 2012 at 20:49:36, Sakari Ailus wrote:
  On Tue, Feb 07, 2012 at 03:35:14PM +0530, Manjunath Hadli wrote:
   add new macro V4L2_PIX_FMT_SGRBG10ALAW8 and associated formats to 
   represent Bayer format frames compressed by A-LAW algorithm, add 
   V4L2_PIX_FMT_UV8 to represent storage of C data (UV interleaved) only.
   
   Signed-off-by: Manjunath Hadli manjunath.ha...@ti.com
   Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com
   ---
.../DocBook/media/v4l/pixfmt-srggb10alaw8.xml  |   34 +++
Documentation/DocBook/media/v4l/pixfmt-uv8.xml |   62 
   
Documentation/DocBook/media/v4l/pixfmt.xml |2 +
include/linux/videodev2.h  |9 +++
4 files changed, 107 insertions(+), 0 deletions(-)  create mode 
   100644 Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml
create mode 100644 Documentation/DocBook/media/v4l/pixfmt-uv8.xml
   
   diff --git a/Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml 
   b/Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml
   new file mode 100644
   index 000..b20f525
   --- /dev/null
   +++ b/Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml
   @@ -0,0 +1,34 @@
   + refentry
   +   refmeta
   + refentrytitle
   +   V4L2_PIX_FMT_SRGGB10ALAW8 ('aRA8'),
   +   V4L2_PIX_FMT_SGBRG10ALAW8 ('aGA8'),
   +   V4L2_PIX_FMT_SGRBG10ALAW8 ('agA8'),
   +   V4L2_PIX_FMT_SBGGR10ALAW8 ('aBA8'),
   + /refentrytitle
   + manvol;
   +   /refmeta
   +   refnamediv
   + refname id=V4L2-PIX-FMT-SRGGB10ALAW8
   +   constantV4L2_PIX_FMT_SRGGB10ALAW8/constant
   + /refname
   + refname id=V4L2-PIX-FMT-SGRBG10ALAW8
   +   constantV4L2_PIX_FMT_SGRBG10ALAW8/constant
   + /refname
   + refname id=V4L2-PIX-FMT-SGBRG10ALAW8
   +   constantV4L2_PIX_FMT_SGBRG10ALAW8/constant
   + /refname
  
  The order here is different than earlier.
   I had taken a reference from your v3 patch series (v4l: Add DPCM compressed 
 formats). Do you want me to change it?

It certainly shouldn't be that way. I'll fix it for the next version.

Regards,

-- 
Sakari Ailus
e-mail: sakari.ai...@iki.fi jabber/XMPP/Gmail: sai...@retiisi.org.uk
--
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

Re: [PATCH v3 2/2] v4l2: add new pixel formats supported on dm365

2012-03-04 Thread Sakari Ailus 
Hi Manju,

Thanks for the patch.

On Tue, Feb 07, 2012 at 03:35:14PM +0530, Manjunath Hadli wrote:
 add new macro V4L2_PIX_FMT_SGRBG10ALAW8 and associated formats
 to represent Bayer format frames compressed by A-LAW algorithm,
 add V4L2_PIX_FMT_UV8 to represent storage of C data (UV interleaved)
 only.
 
 Signed-off-by: Manjunath Hadli manjunath.ha...@ti.com
 Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com
 ---
  .../DocBook/media/v4l/pixfmt-srggb10alaw8.xml  |   34 +++
  Documentation/DocBook/media/v4l/pixfmt-uv8.xml |   62 
 
  Documentation/DocBook/media/v4l/pixfmt.xml |2 +
  include/linux/videodev2.h  |9 +++
  4 files changed, 107 insertions(+), 0 deletions(-)
  create mode 100644 Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml
  create mode 100644 Documentation/DocBook/media/v4l/pixfmt-uv8.xml
 
 diff --git a/Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml 
 b/Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml
 new file mode 100644
 index 000..b20f525
 --- /dev/null
 +++ b/Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml
 @@ -0,0 +1,34 @@
 + refentry
 +   refmeta
 + refentrytitle
 +   V4L2_PIX_FMT_SRGGB10ALAW8 ('aRA8'),
 +   V4L2_PIX_FMT_SGBRG10ALAW8 ('aGA8'),
 +   V4L2_PIX_FMT_SGRBG10ALAW8 ('agA8'),
 +   V4L2_PIX_FMT_SBGGR10ALAW8 ('aBA8'),
 + /refentrytitle
 + manvol;
 +   /refmeta
 +   refnamediv
 + refname id=V4L2-PIX-FMT-SRGGB10ALAW8
 +   constantV4L2_PIX_FMT_SRGGB10ALAW8/constant
 + /refname
 + refname id=V4L2-PIX-FMT-SGRBG10ALAW8
 +   constantV4L2_PIX_FMT_SGRBG10ALAW8/constant
 + /refname
 + refname id=V4L2-PIX-FMT-SGBRG10ALAW8
 +   constantV4L2_PIX_FMT_SGBRG10ALAW8/constant
 + /refname

The order here is different than earlier.

 + refname id=V4L2-PIX-FMT-SBGGR10ALAW8
 +   constantV4L2_PIX_FMT_SBGGR10ALAW8/constant
 + /refname
 + refpurpose10-bit Bayer formats compressed to 8 bits/refpurpose
 +   /refnamediv
 +   refsect1
 + titleDescription/title
 + paraThe following four pixel formats are raw sRGB / Bayer
 + formats with 10 bits per colour compressed to 8 bits each,
 + using the A-LAW algorithm. Each colour component consumes 8
 + bits of memory. In other respects this format is similar to
 + xref linkend=V4L2-PIX-FMT-SRGGB8./xref/para
 +   /refsect1
 + /refentry
 diff --git a/Documentation/DocBook/media/v4l/pixfmt-uv8.xml 
 b/Documentation/DocBook/media/v4l/pixfmt-uv8.xml
 new file mode 100644
 index 000..e3e6b11
 --- /dev/null
 +++ b/Documentation/DocBook/media/v4l/pixfmt-uv8.xml
 @@ -0,0 +1,62 @@
 + refentry id=V4L2-PIX-FMT-UV8
 +   refmeta
 + refentrytitleV4L2_PIX_FMT_UV8  ('UV8')/refentrytitle
 + manvol;
 +   /refmeta
 +   refnamediv
 + refnameconstantV4L2_PIX_FMT_UV8/constant/refname
 + refpurposeUV plane interleaved/refpurpose
 +   /refnamediv
 +   refsect1
 + titleDescription/title
 + paraIn this format there is no Y plane, Only C plane. ie
 + (UV interleaved)/para

How about referring to CbCr instead of C?

 + example
 + title
 +   constantV4L2_PIX_FMT_UV8/constant
 +pixel image
 + /title
 +
 + formalpara
 +   titleByte Order./title
 +   paraEach cell is one byte.
 + informaltable frame=none
 + tgroup cols=5 align=center
 +   colspec align=left colwidth=2* /
 +   tbody valign=top
 + row
 +   entrystartnbsp;+nbsp;0:/entry
 +   entryCbsubscript00/subscript/entry
 +   entryCrsubscript00/subscript/entry
 +   entryCbsubscript01/subscript/entry
 +   entryCrsubscript01/subscript/entry
 + /row
 + row
 +   entrystartnbsp;+nbsp;4:/entry
 +   entryCbsubscript10/subscript/entry
 +   entryCrsubscript10/subscript/entry
 +   entryCbsubscript11/subscript/entry
 +   entryCrsubscript11/subscript/entry
 + /row
 + row
 +   entrystartnbsp;+nbsp;8:/entry
 +   entryCbsubscript20/subscript/entry
 +   entryCrsubscript20/subscript/entry
 +   entryCbsubscript21/subscript/entry
 +   entryCrsubscript21/subscript/entry
 + /row
 + row
 +   entrystartnbsp;+nbsp;12:/entry
 +   entryCbsubscript30/subscript/entry
 +   entryCrsubscript30/subscript/entry
 +   entryCbsubscript31/subscript/entry
 +   entryCrsubscript31/subscript/entry
 +

RE: [PATCH v3 2/2] v4l2: add new pixel formats supported on dm365

2012-03-04 Thread Hadli, Manjunath 
Sakari,

On Sun, Mar 04, 2012 at 20:49:36, Sakari Ailus wrote:
 Hi Manju,
 
 Thanks for the patch.
 
 On Tue, Feb 07, 2012 at 03:35:14PM +0530, Manjunath Hadli wrote:
  add new macro V4L2_PIX_FMT_SGRBG10ALAW8 and associated formats to 
  represent Bayer format frames compressed by A-LAW algorithm, add 
  V4L2_PIX_FMT_UV8 to represent storage of C data (UV interleaved) only.
  
  Signed-off-by: Manjunath Hadli manjunath.ha...@ti.com
  Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com
  ---
   .../DocBook/media/v4l/pixfmt-srggb10alaw8.xml  |   34 +++
   Documentation/DocBook/media/v4l/pixfmt-uv8.xml |   62 
  
   Documentation/DocBook/media/v4l/pixfmt.xml |2 +
   include/linux/videodev2.h  |9 +++
   4 files changed, 107 insertions(+), 0 deletions(-)  create mode 
  100644 Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml
   create mode 100644 Documentation/DocBook/media/v4l/pixfmt-uv8.xml
  
  diff --git a/Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml 
  b/Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml
  new file mode 100644
  index 000..b20f525
  --- /dev/null
  +++ b/Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml
  @@ -0,0 +1,34 @@
  +   refentry
  + refmeta
  +   refentrytitle
  + V4L2_PIX_FMT_SRGGB10ALAW8 ('aRA8'),
  + V4L2_PIX_FMT_SGBRG10ALAW8 ('aGA8'),
  + V4L2_PIX_FMT_SGRBG10ALAW8 ('agA8'),
  + V4L2_PIX_FMT_SBGGR10ALAW8 ('aBA8'),
  +   /refentrytitle
  +   manvol;
  + /refmeta
  + refnamediv
  +   refname id=V4L2-PIX-FMT-SRGGB10ALAW8
  + constantV4L2_PIX_FMT_SRGGB10ALAW8/constant
  +   /refname
  +   refname id=V4L2-PIX-FMT-SGRBG10ALAW8
  + constantV4L2_PIX_FMT_SGRBG10ALAW8/constant
  +   /refname
  +   refname id=V4L2-PIX-FMT-SGBRG10ALAW8
  + constantV4L2_PIX_FMT_SGBRG10ALAW8/constant
  +   /refname
 
 The order here is different than earlier.
  I had taken a reference from your v3 patch series (v4l: Add DPCM compressed 
formats). Do you want me to change it?

 
  +   refname id=V4L2-PIX-FMT-SBGGR10ALAW8
  + constantV4L2_PIX_FMT_SBGGR10ALAW8/constant
  +   /refname
  +   refpurpose10-bit Bayer formats compressed to 8 bits/refpurpose
  + /refnamediv
  + refsect1
  +   titleDescription/title
  +   paraThe following four pixel formats are raw sRGB / Bayer
  +   formats with 10 bits per colour compressed to 8 bits each,
  +   using the A-LAW algorithm. Each colour component consumes 8
  +   bits of memory. In other respects this format is similar to
  +   xref linkend=V4L2-PIX-FMT-SRGGB8./xref/para
  + /refsect1
  +   /refentry
  diff --git a/Documentation/DocBook/media/v4l/pixfmt-uv8.xml 
  b/Documentation/DocBook/media/v4l/pixfmt-uv8.xml
  new file mode 100644
  index 000..e3e6b11
  --- /dev/null
  +++ b/Documentation/DocBook/media/v4l/pixfmt-uv8.xml
  @@ -0,0 +1,62 @@
  +   refentry id=V4L2-PIX-FMT-UV8
  + refmeta
  +   refentrytitleV4L2_PIX_FMT_UV8  ('UV8')/refentrytitle
  +   manvol;
  + /refmeta
  + refnamediv
  +   refnameconstantV4L2_PIX_FMT_UV8/constant/refname
  +   refpurposeUV plane interleaved/refpurpose
  + /refnamediv
  + refsect1
  +   titleDescription/title
  +   paraIn this format there is no Y plane, Only C plane. ie
  +   (UV interleaved)/para
 
 How about referring to CbCr instead of C?
Ok.
 
  +   example
  +   title
  + constantV4L2_PIX_FMT_UV8/constant
  +  pixel image
  +   /title
  +
  +   formalpara
  + titleByte Order./title
  + paraEach cell is one byte.
  +   informaltable frame=none
  +   tgroup cols=5 align=center
  + colspec align=left colwidth=2* /
  + tbody valign=top
  +   row
  + entrystartnbsp;+nbsp;0:/entry
  + entryCbsubscript00/subscript/entry
  + entryCrsubscript00/subscript/entry
  + entryCbsubscript01/subscript/entry
  + entryCrsubscript01/subscript/entry
  +   /row
  +   row
  + entrystartnbsp;+nbsp;4:/entry
  + entryCbsubscript10/subscript/entry
  + entryCrsubscript10/subscript/entry
  + entryCbsubscript11/subscript/entry
  + entryCrsubscript11/subscript/entry
  +   /row
  +   row
  + entrystartnbsp;+nbsp;8:/entry
  + entryCbsubscript20/subscript/entry
  + entryCrsubscript20/subscript/entry
  + entryCbsubscript21/subscript/entry
  + entryCrsubscript21/subscript/entry
  +   /row
  +   row
  + entrystartnbsp;+nbsp;12:/entry
  + entryCbsubscript30/subscript/entry
  +

[PATCH v3 2/2] v4l2: add new pixel formats supported on dm365

2012-02-07 Thread Manjunath Hadli 
add new macro V4L2_PIX_FMT_SGRBG10ALAW8 and associated formats
to represent Bayer format frames compressed by A-LAW algorithm,
add V4L2_PIX_FMT_UV8 to represent storage of C data (UV interleaved)
only.

Signed-off-by: Manjunath Hadli manjunath.ha...@ti.com
Cc: Laurent Pinchart laurent.pinch...@ideasonboard.com
---
 .../DocBook/media/v4l/pixfmt-srggb10alaw8.xml  |   34 +++
 Documentation/DocBook/media/v4l/pixfmt-uv8.xml |   62 
 Documentation/DocBook/media/v4l/pixfmt.xml |2 +
 include/linux/videodev2.h  |9 +++
 4 files changed, 107 insertions(+), 0 deletions(-)
 create mode 100644 Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml
 create mode 100644 Documentation/DocBook/media/v4l/pixfmt-uv8.xml

diff --git a/Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml 
b/Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml
new file mode 100644
index 000..b20f525
--- /dev/null
+++ b/Documentation/DocBook/media/v4l/pixfmt-srggb10alaw8.xml
@@ -0,0 +1,34 @@
+   refentry
+ refmeta
+   refentrytitle
+ V4L2_PIX_FMT_SRGGB10ALAW8 ('aRA8'),
+ V4L2_PIX_FMT_SGBRG10ALAW8 ('aGA8'),
+ V4L2_PIX_FMT_SGRBG10ALAW8 ('agA8'),
+ V4L2_PIX_FMT_SBGGR10ALAW8 ('aBA8'),
+   /refentrytitle
+   manvol;
+ /refmeta
+ refnamediv
+   refname id=V4L2-PIX-FMT-SRGGB10ALAW8
+ constantV4L2_PIX_FMT_SRGGB10ALAW8/constant
+   /refname
+   refname id=V4L2-PIX-FMT-SGRBG10ALAW8
+ constantV4L2_PIX_FMT_SGRBG10ALAW8/constant
+   /refname
+   refname id=V4L2-PIX-FMT-SGBRG10ALAW8
+ constantV4L2_PIX_FMT_SGBRG10ALAW8/constant
+   /refname
+   refname id=V4L2-PIX-FMT-SBGGR10ALAW8
+ constantV4L2_PIX_FMT_SBGGR10ALAW8/constant
+   /refname
+   refpurpose10-bit Bayer formats compressed to 8 bits/refpurpose
+ /refnamediv
+ refsect1
+   titleDescription/title
+   paraThe following four pixel formats are raw sRGB / Bayer
+   formats with 10 bits per colour compressed to 8 bits each,
+   using the A-LAW algorithm. Each colour component consumes 8
+   bits of memory. In other respects this format is similar to
+   xref linkend=V4L2-PIX-FMT-SRGGB8./xref/para
+ /refsect1
+   /refentry
diff --git a/Documentation/DocBook/media/v4l/pixfmt-uv8.xml 
b/Documentation/DocBook/media/v4l/pixfmt-uv8.xml
new file mode 100644
index 000..e3e6b11
--- /dev/null
+++ b/Documentation/DocBook/media/v4l/pixfmt-uv8.xml
@@ -0,0 +1,62 @@
+   refentry id=V4L2-PIX-FMT-UV8
+ refmeta
+   refentrytitleV4L2_PIX_FMT_UV8  ('UV8')/refentrytitle
+   manvol;
+ /refmeta
+ refnamediv
+   refnameconstantV4L2_PIX_FMT_UV8/constant/refname
+   refpurposeUV plane interleaved/refpurpose
+ /refnamediv
+ refsect1
+   titleDescription/title
+   paraIn this format there is no Y plane, Only C plane. ie
+   (UV interleaved)/para
+   example
+   title
+ constantV4L2_PIX_FMT_UV8/constant
+  pixel image
+   /title
+
+   formalpara
+ titleByte Order./title
+ paraEach cell is one byte.
+   informaltable frame=none
+   tgroup cols=5 align=center
+ colspec align=left colwidth=2* /
+ tbody valign=top
+   row
+ entrystartnbsp;+nbsp;0:/entry
+ entryCbsubscript00/subscript/entry
+ entryCrsubscript00/subscript/entry
+ entryCbsubscript01/subscript/entry
+ entryCrsubscript01/subscript/entry
+   /row
+   row
+ entrystartnbsp;+nbsp;4:/entry
+ entryCbsubscript10/subscript/entry
+ entryCrsubscript10/subscript/entry
+ entryCbsubscript11/subscript/entry
+ entryCrsubscript11/subscript/entry
+   /row
+   row
+ entrystartnbsp;+nbsp;8:/entry
+ entryCbsubscript20/subscript/entry
+ entryCrsubscript20/subscript/entry
+ entryCbsubscript21/subscript/entry
+ entryCrsubscript21/subscript/entry
+   /row
+   row
+ entrystartnbsp;+nbsp;12:/entry
+ entryCbsubscript30/subscript/entry
+ entryCrsubscript30/subscript/entry
+ entryCbsubscript31/subscript/entry
+ entryCrsubscript31/subscript/entry
+   /row
+ /tbody
+   /tgroup
+   /informaltable
+ /para
+ /formalpara