All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Added bayer 8-bit patterns to uvcvideo
@ 2014-01-20  8:11 Edgar Thier
  2014-02-18 14:37 ` Laurent Pinchart
  0 siblings, 1 reply; 2+ messages in thread
From: Edgar Thier @ 2014-01-20  8:11 UTC (permalink / raw)
  To: linux-media

Add bayer 8-bit GUIDs to uvcvideo and
associate them with the corresponding V4L2 pixel formats.

Signed-off-by: Edgar Thier <info@edgarthier.net>
---
 drivers/media/usb/uvc/uvc_driver.c | 22 +++++++++++++++++++++-
 drivers/media/usb/uvc/uvcvideo.h   | 12 ++++++++++++
 2 files changed, 33 insertions(+), 1 deletion(-)

diff --git a/drivers/media/usb/uvc/uvc_driver.c
b/drivers/media/usb/uvc/uvc_driver.c
index c3bb250..84da426 100644
--- a/drivers/media/usb/uvc/uvc_driver.c
+++ b/drivers/media/usb/uvc/uvc_driver.c
@@ -108,11 +108,31 @@ static struct uvc_format_desc uvc_fmts[] = {
         .fcc        = V4L2_PIX_FMT_Y16,
     },
     {
-        .name        = "RGB Bayer",
+        .name        = "RGB Bayer (bggr)",
         .guid        = UVC_GUID_FORMAT_BY8,
         .fcc        = V4L2_PIX_FMT_SBGGR8,
     },
     {
+        .name        = "RGB Bayer (bggr)",
+        .guid        = UVC_GUID_FORMAT_BY8_BA81,
+        .fcc        = V4L2_PIX_FMT_SBGGR8,
+    },
+    {
+        .name        = "RGB Bayer (grbg)",
+        .guid        = UVC_GUID_FORMAT_BY8_GRBG,
+        .fcc        = V4L2_PIX_FMT_SGRBG8,
+    },
+    {
+        .name        = "RGB Bayer (gbrg)",
+        .guid        = UVC_GUID_FORMAT_BY8_GBRG,
+        .fcc        = V4L2_PIX_FMT_SGBRG8,
+    },
+    {
+        .name        = "RGB Bayer (rggb)",
+        .guid        = UVC_GUID_FORMAT_BY8_RGGB,
+        .fcc        = V4L2_PIX_FMT_SRGGB8,
+    },
+    {
         .name        = "RGB565",
         .guid        = UVC_GUID_FORMAT_RGBP,
         .fcc        = V4L2_PIX_FMT_RGB565,
diff --git a/drivers/media/usb/uvc/uvcvideo.h b/drivers/media/usb/uvc/uvcvideo.h
index 9e35982..57357d9 100644
--- a/drivers/media/usb/uvc/uvcvideo.h
+++ b/drivers/media/usb/uvc/uvcvideo.h
@@ -94,6 +94,18 @@
 #define UVC_GUID_FORMAT_BY8 \
     { 'B',  'Y',  '8',  ' ', 0x00, 0x00, 0x10, 0x00, \
      0x80, 0x00, 0x00, 0xaa, 0x00, 0x38, 0x9b, 0x71}
+#define UVC_GUID_FORMAT_BY8_BA81 \
+    { 'B',  'A',  '8',  '1', 0x00, 0x00, 0x10, 0x00, \
+     0x80, 0x00, 0x00, 0xaa, 0x00, 0x38, 0x9b, 0x71}
+#define UVC_GUID_FORMAT_BY8_GRBG \
+    { 'G',  'R',  'B',  'G', 0x00, 0x00, 0x10, 0x00, \
+     0x80, 0x00, 0x00, 0xaa, 0x00, 0x38, 0x9b, 0x71}
+#define UVC_GUID_FORMAT_BY8_GBRG \
+    { 'G',  'B',  'R',  'G', 0x00, 0x00, 0x10, 0x00, \
+     0x80, 0x00, 0x00, 0xaa, 0x00, 0x38, 0x9b, 0x71}
+#define UVC_GUID_FORMAT_BY8_RGGB \
+    { 'R',  'G',  'G',  'B', 0x00, 0x00, 0x10, 0x00, \
+     0x80, 0x00, 0x00, 0xaa, 0x00, 0x38, 0x9b, 0x71}
 #define UVC_GUID_FORMAT_RGBP \
     { 'R',  'G',  'B',  'P', 0x00, 0x00, 0x10, 0x00, \
      0x80, 0x00, 0x00, 0xaa, 0x00, 0x38, 0x9b, 0x71}
-- 
1.8.5.3

^ permalink raw reply related	[flat|nested] 2+ messages in thread
* Re: [PATCH] Added bayer 8-bit patterns to uvcvideo
  2014-01-20  8:11 [PATCH] Added bayer 8-bit patterns to uvcvideo Edgar Thier
@ 2014-02-18 14:37 ` Laurent Pinchart
  0 siblings, 0 replies; 2+ messages in thread
From: Laurent Pinchart @ 2014-02-18 14:37 UTC (permalink / raw)
  To: Edgar Thier; +Cc: linux-media

Hi Edgar,

Thank you for the patch.

On Monday 20 January 2014 09:11:28 Edgar Thier wrote:
> Add bayer 8-bit GUIDs to uvcvideo and
> associate them with the corresponding V4L2 pixel formats.
> 
> Signed-off-by: Edgar Thier <info@edgarthier.net>

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Out of curiosity, could you please send me the lsusb -v output of the camera 
you need this for ?

> ---
>  drivers/media/usb/uvc/uvc_driver.c | 22 +++++++++++++++++++++-
>  drivers/media/usb/uvc/uvcvideo.h   | 12 ++++++++++++
>  2 files changed, 33 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/media/usb/uvc/uvc_driver.c
> b/drivers/media/usb/uvc/uvc_driver.c
> index c3bb250..84da426 100644
> --- a/drivers/media/usb/uvc/uvc_driver.c
> +++ b/drivers/media/usb/uvc/uvc_driver.c
> @@ -108,11 +108,31 @@ static struct uvc_format_desc uvc_fmts[] = {
>          .fcc        = V4L2_PIX_FMT_Y16,
>      },
>      {
> -        .name        = "RGB Bayer",
> +        .name        = "RGB Bayer (bggr)",
>          .guid        = UVC_GUID_FORMAT_BY8,
>          .fcc        = V4L2_PIX_FMT_SBGGR8,

It looks like your mailer has corrupted the patch by replacing tabs with 
spaces. Could you please fix that and resubmit ? While you're at it, could you 
please prefix the commit subject with "uvcvideo: " ?

>      },
>      {
> +        .name        = "RGB Bayer (bggr)",
> +        .guid        = UVC_GUID_FORMAT_BY8_BA81,
> +        .fcc        = V4L2_PIX_FMT_SBGGR8,
> +    },
> +    {
> +        .name        = "RGB Bayer (grbg)",
> +        .guid        = UVC_GUID_FORMAT_BY8_GRBG,
> +        .fcc        = V4L2_PIX_FMT_SGRBG8,
> +    },
> +    {
> +        .name        = "RGB Bayer (gbrg)",
> +        .guid        = UVC_GUID_FORMAT_BY8_GBRG,
> +        .fcc        = V4L2_PIX_FMT_SGBRG8,
> +    },
> +    {
> +        .name        = "RGB Bayer (rggb)",
> +        .guid        = UVC_GUID_FORMAT_BY8_RGGB,
> +        .fcc        = V4L2_PIX_FMT_SRGGB8,
> +    },
> +    {
>          .name        = "RGB565",
>          .guid        = UVC_GUID_FORMAT_RGBP,
>          .fcc        = V4L2_PIX_FMT_RGB565,
> diff --git a/drivers/media/usb/uvc/uvcvideo.h
> b/drivers/media/usb/uvc/uvcvideo.h index 9e35982..57357d9 100644
> --- a/drivers/media/usb/uvc/uvcvideo.h
> +++ b/drivers/media/usb/uvc/uvcvideo.h
> @@ -94,6 +94,18 @@
>  #define UVC_GUID_FORMAT_BY8 \
>      { 'B',  'Y',  '8',  ' ', 0x00, 0x00, 0x10, 0x00, \
>       0x80, 0x00, 0x00, 0xaa, 0x00, 0x38, 0x9b, 0x71}
> +#define UVC_GUID_FORMAT_BY8_BA81 \
> +    { 'B',  'A',  '8',  '1', 0x00, 0x00, 0x10, 0x00, \
> +     0x80, 0x00, 0x00, 0xaa, 0x00, 0x38, 0x9b, 0x71}
> +#define UVC_GUID_FORMAT_BY8_GRBG \
> +    { 'G',  'R',  'B',  'G', 0x00, 0x00, 0x10, 0x00, \
> +     0x80, 0x00, 0x00, 0xaa, 0x00, 0x38, 0x9b, 0x71}
> +#define UVC_GUID_FORMAT_BY8_GBRG \
> +    { 'G',  'B',  'R',  'G', 0x00, 0x00, 0x10, 0x00, \
> +     0x80, 0x00, 0x00, 0xaa, 0x00, 0x38, 0x9b, 0x71}
> +#define UVC_GUID_FORMAT_BY8_RGGB \
> +    { 'R',  'G',  'G',  'B', 0x00, 0x00, 0x10, 0x00, \
> +     0x80, 0x00, 0x00, 0xaa, 0x00, 0x38, 0x9b, 0x71}
>  #define UVC_GUID_FORMAT_RGBP \
>      { 'R',  'G',  'B',  'P', 0x00, 0x00, 0x10, 0x00, \
>       0x80, 0x00, 0x00, 0xaa, 0x00, 0x38, 0x9b, 0x71}

-- 
Regards,

Laurent Pinchart


^ permalink raw reply	[flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-02-18 14:36 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-01-20  8:11 [PATCH] Added bayer 8-bit patterns to uvcvideo Edgar Thier
2014-02-18 14:37 ` Laurent Pinchart

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.