Re: [PATCH v2.2 10/10] v4l: Add 16-bit raw bayer pixel format definitions

2016-07-13 Thread Lad, Prabhakar
On Thu, Jul 7, 2016 at 7:48 AM, Sakari Ailus
 wrote:
> The formats added by this patch are:
>
> V4L2_PIX_FMT_SBGGR16
> V4L2_PIX_FMT_SGBRG16
> V4L2_PIX_FMT_SGRBG16
>
> V4L2_PIX_FMT_SRGGB16 already existed before the patch. Rework the
> documentation to match that of the other sample depths.
>
> Signed-off-by: Sakari Ailus 
> ---

Acked-by: Lad, Prabhakar 

Cheers,
--Prabhakar Lad

>  Documentation/DocBook/media/v4l/pixfmt-sbggr16.xml | 81 ---
>  Documentation/DocBook/media/v4l/pixfmt-srggb16.xml | 91 
> ++
>  Documentation/DocBook/media/v4l/pixfmt.xml |  2 +-
>  include/uapi/linux/videodev2.h |  3 +
>  4 files changed, 95 insertions(+), 82 deletions(-)
>  delete mode 100644 Documentation/DocBook/media/v4l/pixfmt-sbggr16.xml
>  create mode 100644 Documentation/DocBook/media/v4l/pixfmt-srggb16.xml
>
> diff --git a/Documentation/DocBook/media/v4l/pixfmt-sbggr16.xml 
> b/Documentation/DocBook/media/v4l/pixfmt-sbggr16.xml
> deleted file mode 100644
> index 789160565..000
> --- a/Documentation/DocBook/media/v4l/pixfmt-sbggr16.xml
> +++ /dev/null
> @@ -1,81 +0,0 @@
> -
> -  
> -V4L2_PIX_FMT_SBGGR16 ('BYR2')
> -&manvol;
> -  
> -  
> -V4L2_PIX_FMT_SBGGR16
> -Bayer RGB format
> -  
> -  
> -Description
> -
> -This format is similar to  -linkend="V4L2-PIX-FMT-SBGGR8">
> -V4L2_PIX_FMT_SBGGR8, except each pixel has
> -a depth of 16 bits. The least significant byte is stored at lower
> -memory addresses (little-endian).
> -
> -
> -  V4L2_PIX_FMT_SBGGR16 4 × 4
> -pixel image
> -
> -  
> -   Byte Order.
> -   Each cell is one byte.
> - 
> -   
> - 
> - 
> -   
> - start + 0:
> - B00low
> - B00high
> - G01low
> - G01high
> - B02low
> - B02high
> - G03low
> - G03high
> -   
> -   
> - start + 8:
> - G10low
> - G10high
> - R11low
> - R11high
> - G12low
> - G12high
> - R13low
> - R13high
> -   
> -   
> - start + 16:
> - B20low
> - B20high
> - G21low
> - G21high
> - B22low
> - B22high
> - G23low
> - G23high
> -   
> -   
> - start + 24:
> - G30low
> - G30high
> - R31low
> - R31high
> - G32low
> - G32high
> - R33low
> - R33high
> -   
> - 
> -   
> - 
> -   
> -  
> -
> -  
> -
> diff --git a/Documentation/DocBook/media/v4l/pixfmt-srggb16.xml 
> b/Documentation/DocBook/media/v4l/pixfmt-srggb16.xml
> new file mode 100644
> index 000..590266f
> --- /dev/null
> +++ b/Documentation/DocBook/media/v4l/pixfmt-srggb16.xml
> @@ -0,0 +1,91 @@
> +
> +  
> +   V4L2_PIX_FMT_SRGGB16 ('RG16'),
> +V4L2_PIX_FMT_SGRBG16 ('GR16'),
> +V4L2_PIX_FMT_SGBRG16 ('GB16'),
> +V4L2_PIX_FMT_SBGGR16 ('BYR2')
> +
> +   &manvol;
> +  
> +  
> +id="V4L2-PIX-FMT-SRGGB16">V4L2_PIX_FMT_SRGGB16
> +id="V4L2-PIX-FMT-SGRBG16">V4L2_PIX_FMT_SGRBG16
> +id="V4L2-PIX-FMT-SGBRG16">V4L2_PIX_FMT_SGBRG16
> +id="V4L2-PIX-FMT-SBGGR16">V4L2_PIX_FMT_SBGGR16
> +   16-bit Bayer formats
> +  
> +  
> +   Description
> +
> +   These four pixel formats are raw sRGB / Bayer formats with
> +16 bits per colour. Each colour component is stored in a 16-bit word.
> +Each n-pixel row contains n/2 green samples and n/2 blue or red
> +samples, with alternating red and blue rows. Bytes are stored in
> +memory in little endian order. They are conventionally described
> +as GRGR... BGBG..., RGRG... GBGB..., etc. Below is an example of one of these
> +formats:
> +
> +
> +  V4L2_PIX_FMT_SBGGR16 4 × 4
> +pixel image
> +
> +  
> +   Byte Order.
> +   Each cell is one byte.
> + 
> +   
> + 
> + 
> +   
> + start + 0:
> + B00low
> + B00high
> + G01low
> + G01high
> + B02low
> + B02high
> + G03low
> + G03high
> +   
> +   
> + start + 8:
> + G10low
> + G10high
> + R11low
> + R11high
> + G12low
> + G12high
> + R13low
> + R13high
> +   

[PATCH v2.2 10/10] v4l: Add 16-bit raw bayer pixel format definitions

2016-07-06 Thread Sakari Ailus
The formats added by this patch are:

V4L2_PIX_FMT_SBGGR16
V4L2_PIX_FMT_SGBRG16
V4L2_PIX_FMT_SGRBG16

V4L2_PIX_FMT_SRGGB16 already existed before the patch. Rework the
documentation to match that of the other sample depths.

Signed-off-by: Sakari Ailus 
---
 Documentation/DocBook/media/v4l/pixfmt-sbggr16.xml | 81 ---
 Documentation/DocBook/media/v4l/pixfmt-srggb16.xml | 91 ++
 Documentation/DocBook/media/v4l/pixfmt.xml |  2 +-
 include/uapi/linux/videodev2.h |  3 +
 4 files changed, 95 insertions(+), 82 deletions(-)
 delete mode 100644 Documentation/DocBook/media/v4l/pixfmt-sbggr16.xml
 create mode 100644 Documentation/DocBook/media/v4l/pixfmt-srggb16.xml

diff --git a/Documentation/DocBook/media/v4l/pixfmt-sbggr16.xml 
b/Documentation/DocBook/media/v4l/pixfmt-sbggr16.xml
deleted file mode 100644
index 789160565..000
--- a/Documentation/DocBook/media/v4l/pixfmt-sbggr16.xml
+++ /dev/null
@@ -1,81 +0,0 @@
-
-  
-V4L2_PIX_FMT_SBGGR16 ('BYR2')
-&manvol;
-  
-  
-V4L2_PIX_FMT_SBGGR16
-Bayer RGB format
-  
-  
-Description
-
-This format is similar to 
-V4L2_PIX_FMT_SBGGR8, except each pixel has
-a depth of 16 bits. The least significant byte is stored at lower
-memory addresses (little-endian).
-
-
-  V4L2_PIX_FMT_SBGGR16 4 × 4
-pixel image
-
-  
-   Byte Order.
-   Each cell is one byte.
- 
-   
- 
- 
-   
- start + 0:
- B00low
- B00high
- G01low
- G01high
- B02low
- B02high
- G03low
- G03high
-   
-   
- start + 8:
- G10low
- G10high
- R11low
- R11high
- G12low
- G12high
- R13low
- R13high
-   
-   
- start + 16:
- B20low
- B20high
- G21low
- G21high
- B22low
- B22high
- G23low
- G23high
-   
-   
- start + 24:
- G30low
- G30high
- R31low
- R31high
- G32low
- G32high
- R33low
- R33high
-   
- 
-   
- 
-   
-  
-
-  
-
diff --git a/Documentation/DocBook/media/v4l/pixfmt-srggb16.xml 
b/Documentation/DocBook/media/v4l/pixfmt-srggb16.xml
new file mode 100644
index 000..590266f
--- /dev/null
+++ b/Documentation/DocBook/media/v4l/pixfmt-srggb16.xml
@@ -0,0 +1,91 @@
+
+  
+   V4L2_PIX_FMT_SRGGB16 ('RG16'),
+V4L2_PIX_FMT_SGRBG16 ('GR16'),
+V4L2_PIX_FMT_SGBRG16 ('GB16'),
+V4L2_PIX_FMT_SBGGR16 ('BYR2')
+
+   &manvol;
+  
+  
+   V4L2_PIX_FMT_SRGGB16
+   V4L2_PIX_FMT_SGRBG16
+   V4L2_PIX_FMT_SGBRG16
+   V4L2_PIX_FMT_SBGGR16
+   16-bit Bayer formats
+  
+  
+   Description
+
+   These four pixel formats are raw sRGB / Bayer formats with
+16 bits per colour. Each colour component is stored in a 16-bit word.
+Each n-pixel row contains n/2 green samples and n/2 blue or red
+samples, with alternating red and blue rows. Bytes are stored in
+memory in little endian order. They are conventionally described
+as GRGR... BGBG..., RGRG... GBGB..., etc. Below is an example of one of these
+formats:
+
+
+  V4L2_PIX_FMT_SBGGR16 4 × 4
+pixel image
+
+  
+   Byte Order.
+   Each cell is one byte.
+ 
+   
+ 
+ 
+   
+ start + 0:
+ B00low
+ B00high
+ G01low
+ G01high
+ B02low
+ B02high
+ G03low
+ G03high
+   
+   
+ start + 8:
+ G10low
+ G10high
+ R11low
+ R11high
+ G12low
+ G12high
+ R13low
+ R13high
+   
+   
+ start + 16:
+ B20low
+ B20high
+ G21low
+ G21high
+ B22low
+ B22high
+ G23low
+ G23high
+   
+   
+ start + 24:
+ G30low
+ G30high
+ R31low
+ R31high
+ G32low
+ G32high
+ R33low
+ R33high
+   
+ 
+   
+ 
+   
+