* [PATCH] pxa_camera: Fix YUV format handling.
@ 2008-11-06 23:04 Robert Jarzmik
2008-11-06 23:52 ` Guennadi Liakhovetski
0 siblings, 1 reply; 8+ messages in thread
From: Robert Jarzmik @ 2008-11-06 23:04 UTC (permalink / raw)
To: video4linux-list, g.liakhovetski
Allows all YUV formats on pxa interface. Even if PXA capture
interface expects data in UYVY format, we allow all formats
considering the pxa bus is not making any translation.
For the special YUV planar format, we translate the pixel
format asked to the sensor to VYUY, which is the bus byte
order necessary (out of the sensor) for the pxa to make the
correct translation.
Signed-off-by: Robert Jarzmik <robert.jarzmik@free.fr>
---
drivers/media/video/pxa_camera.c | 15 +++++++++++++++
1 files changed, 15 insertions(+), 0 deletions(-)
diff --git a/drivers/media/video/pxa_camera.c b/drivers/media/video/pxa_camera.c
index eb6be58..863e0df 100644
--- a/drivers/media/video/pxa_camera.c
+++ b/drivers/media/video/pxa_camera.c
@@ -862,7 +862,15 @@ static int pxa_camera_set_bus_param(struct soc_camera_device *icd, __u32 pixfmt)
case V4L2_PIX_FMT_YUV422P:
pcdev->channels = 3;
cicr1 |= CICR1_YCBCR_F;
+ /*
+ * Normally, pxa bus wants as input VYUY format.
+ * We allow all YUV formats, as no translation is used, and the
+ * YUV stream is just passed through without any transformation.
+ */
+ case V4L2_PIX_FMT_UYVY:
+ case V4L2_PIX_FMT_VYUY:
case V4L2_PIX_FMT_YUYV:
+ case V4L2_PIX_FMT_YVYU:
cicr1 |= CICR1_COLOR_SP_VAL(2);
break;
case V4L2_PIX_FMT_RGB555:
@@ -907,6 +915,13 @@ static int pxa_camera_try_bus_param(struct soc_camera_device *icd, __u32 pixfmt)
static int pxa_camera_set_fmt_cap(struct soc_camera_device *icd,
__u32 pixfmt, struct v4l2_rect *rect)
{
+ /*
+ * The YUV 4:2:2 planar format is translated by the pxa assuming its
+ * input (ie. camera device output) is VYUV.
+ * We fix the pixel format asked to the camera device.
+ */
+ if (pixfmt == V4L2_PIX_FMT_YUV422P)
+ pixfmt = V4L2_PIX_FMT_VYUY;
return icd->ops->set_fmt_cap(icd, pixfmt, rect);
}
--
1.5.6.5
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] pxa_camera: Fix YUV format handling.
2008-11-06 23:04 [PATCH] pxa_camera: Fix YUV format handling Robert Jarzmik
@ 2008-11-06 23:52 ` Guennadi Liakhovetski
2008-11-07 7:10 ` Antonio Ospite
2008-11-07 17:14 ` Robert Jarzmik
0 siblings, 2 replies; 8+ messages in thread
From: Guennadi Liakhovetski @ 2008-11-06 23:52 UTC (permalink / raw)
To: Robert Jarzmik; +Cc: video4linux-list
On Fri, 7 Nov 2008, Robert Jarzmik wrote:
> Allows all YUV formats on pxa interface. Even if PXA capture
> interface expects data in UYVY format, we allow all formats
Here you call it UYVY, and I agree with it, however, in the comment in the
patch you call it VYUY, which, I think is less natural.
> considering the pxa bus is not making any translation.
>
> For the special YUV planar format, we translate the pixel
> format asked to the sensor to VYUY, which is the bus byte
Now again VYUY... In your patch for mt9m111 you do
+ case V4L2_PIX_FMT_UYVY:
+ mt9m111->swap_yuv_y_chromas = 0;
+ mt9m111->swap_yuv_cb_cr = 0;
i.e., you call mt9m111's default format "UYVY", and again, I think, this
is logical. If both datasheets are correct, this is also the format
expected by the PXA, so, you should translate YUV planar to UYVY. Have you
tested it?
And one more comment below.
> order necessary (out of the sensor) for the pxa to make the
> correct translation.
>
> Signed-off-by: Robert Jarzmik <robert.jarzmik@free.fr>
> ---
> drivers/media/video/pxa_camera.c | 15 +++++++++++++++
> 1 files changed, 15 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/media/video/pxa_camera.c b/drivers/media/video/pxa_camera.c
> index eb6be58..863e0df 100644
> --- a/drivers/media/video/pxa_camera.c
> +++ b/drivers/media/video/pxa_camera.c
> @@ -862,7 +862,15 @@ static int pxa_camera_set_bus_param(struct soc_camera_device *icd, __u32 pixfmt)
> case V4L2_PIX_FMT_YUV422P:
> pcdev->channels = 3;
> cicr1 |= CICR1_YCBCR_F;
> + /*
> + * Normally, pxa bus wants as input VYUY format.
> + * We allow all YUV formats, as no translation is used, and the
> + * YUV stream is just passed through without any transformation.
> + */
Do I understand it right from the pxa270 datasheet, that UYVY would be the
only possible format if it were used for overlay2? Then I would mention
this here as well.
> + case V4L2_PIX_FMT_UYVY:
> + case V4L2_PIX_FMT_VYUY:
> case V4L2_PIX_FMT_YUYV:
> + case V4L2_PIX_FMT_YVYU:
> cicr1 |= CICR1_COLOR_SP_VAL(2);
> break;
> case V4L2_PIX_FMT_RGB555:
> @@ -907,6 +915,13 @@ static int pxa_camera_try_bus_param(struct soc_camera_device *icd, __u32 pixfmt)
> static int pxa_camera_set_fmt_cap(struct soc_camera_device *icd,
> __u32 pixfmt, struct v4l2_rect *rect)
> {
> + /*
> + * The YUV 4:2:2 planar format is translated by the pxa assuming its
> + * input (ie. camera device output) is VYUV.
> + * We fix the pixel format asked to the camera device.
> + */
> + if (pixfmt == V4L2_PIX_FMT_YUV422P)
> + pixfmt = V4L2_PIX_FMT_VYUY;
> return icd->ops->set_fmt_cap(icd, pixfmt, rect);
> }
>
> --
> 1.5.6.5
>
So, let's just get the naming consistent. Are you also planning to update
your "Add new pixel format VYUY 16 bits wide" patch as requested by Hans
Verkuil? Then you could put all these patches in a patch series to make it
easier to manage them:-)
Also, I would _at the very least_ give credit to Antonio Ospite for
reporting the problem and suggesting a first fix in your patch for
mt9m111. Eventually we would also like to have a Tested-by from him.
Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] pxa_camera: Fix YUV format handling.
2008-11-06 23:52 ` Guennadi Liakhovetski
@ 2008-11-07 7:10 ` Antonio Ospite
2008-11-07 17:14 ` Robert Jarzmik
1 sibling, 0 replies; 8+ messages in thread
From: Antonio Ospite @ 2008-11-07 7:10 UTC (permalink / raw)
To: video4linux-list; +Cc: video4linux-list
[-- Attachment #1.1: Type: text/plain, Size: 984 bytes --]
On Fri, 7 Nov 2008 00:52:12 +0100 (CET)
Guennadi Liakhovetski <g.liakhovetski@gmx.de> wrote:
>
> So, let's just get the naming consistent. Are you also planning to update
> your "Add new pixel format VYUY 16 bits wide" patch as requested by Hans
> Verkuil? Then you could put all these patches in a patch series to make it
> easier to manage them:-)
>
> Also, I would _at the very least_ give credit to Antonio Ospite for
> reporting the problem and suggesting a first fix in your patch for
> mt9m111. Eventually we would also like to have a Tested-by from him.
>
If you can provide a patch series it will be easier to test for me.
Thanks,
Antonio Ospite
--
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
A: Top-posting.
Q: What is the most annoying thing in e-mail?
Web site: http://www.studenti.unina.it/~ospite
Public key: http://www.studenti.unina.it/~ospite/aopubkey.asc
[-- Attachment #1.2: Type: application/pgp-signature, Size: 197 bytes --]
[-- Attachment #2: Type: text/plain, Size: 164 bytes --]
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] pxa_camera: Fix YUV format handling.
2008-11-06 23:52 ` Guennadi Liakhovetski
2008-11-07 7:10 ` Antonio Ospite
@ 2008-11-07 17:14 ` Robert Jarzmik
2008-11-16 22:23 ` [PATCH] Add new pixel format VYUY 16 bits wide Guennadi Liakhovetski
1 sibling, 1 reply; 8+ messages in thread
From: Robert Jarzmik @ 2008-11-07 17:14 UTC (permalink / raw)
To: Guennadi Liakhovetski; +Cc: video4linux-list
Guennadi Liakhovetski <g.liakhovetski@gmx.de> writes:
> On Fri, 7 Nov 2008, Robert Jarzmik wrote:
>
>> Allows all YUV formats on pxa interface. Even if PXA capture
>> interface expects data in UYVY format, we allow all formats
>
> Here you call it UYVY, and I agree with it, however, in the comment in the
> patch you call it VYUY, which, I think is less natural.
Yes, the patch is wrong.
It's UYVY (Cb Y Cr Y). I'll amend the patch.
> i.e., you call mt9m111's default format "UYVY", and again, I think, this
> is logical. If both datasheets are correct, this is also the format
> expected by the PXA, so, you should translate YUV planar to UYVY. Have you
> tested it?
Yes, I had.
And I didn't saw the problem. I checked again today, and the image is greyish.
I was using my phone and myself as the picture model. My pullover is white, I'm
grey lately, the background is grey also, as is the phone. Thus, even if the
image looked correct, it was not.
I tried again today the planar YUV422P, but with a bright red pullover. With
either UYVY or VYUY, the colors are not displayed, all remains grey. I don't
really know if the problem comes from my image transformation (yuvsplittoppm),
or from the pxa_camera driver. I'll dig deeper this week-end.
> Do I understand it right from the pxa270 datasheet, that UYVY would be the
> only possible format if it were used for overlay2? Then I would mention
> this here as well.
As you wish.
> So, let's just get the naming consistent. Are you also planning to update
> your "Add new pixel format VYUY 16 bits wide" patch as requested by Hans
> Verkuil? Then you could put all these patches in a patch series to make it
> easier to manage them:-)
I didn't get that mail, either on direct destination or from the mailing
list. I'll look into the archives.
> Also, I would _at the very least_ give credit to Antonio Ospite for
> reporting the problem and suggesting a first fix in your patch for
> mt9m111. Eventually we would also like to have a Tested-by from him.
Yes, of course.
--
Robert
PS: I didn't mentioned it yet, but I had amended the mt9m111 patch, because
format_by_fourcc() in soc_camera.c is looking for YUV422P in mt9m111 formats,
even if pxa_camera translates the format.
This would deserve a cleaner patch, if you can think of one ...
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Add new pixel format VYUY 16 bits wide.
2008-11-07 17:14 ` Robert Jarzmik
@ 2008-11-16 22:23 ` Guennadi Liakhovetski
2008-11-16 22:33 ` [PATCH v2] " Robert Jarzmik
0 siblings, 1 reply; 8+ messages in thread
From: Guennadi Liakhovetski @ 2008-11-16 22:23 UTC (permalink / raw)
To: Hans Verkuil, Robert Jarzmik; +Cc: video4linux-list
On Fri, 7 Nov 2008, Robert Jarzmik wrote:
> Guennadi Liakhovetski <g.liakhovetski@gmx.de> writes:
[snip]
> > So, let's just get the naming consistent. Are you also planning to update
> > your "Add new pixel format VYUY 16 bits wide" patch as requested by Hans
> > Verkuil? Then you could put all these patches in a patch series to make it
> > easier to manage them:-)
> I didn't get that mail, either on direct destination or from the mailing
> list. I'll look into the archives.
Here's his mail again quoted below.
Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
On Wed, 5 Nov 2008, Hans Verkuil wrote:
> On Tuesday 04 November 2008 22:59:37 Robert Jarzmik wrote:
> > There were already 3 YUV formats defined :
> > - YUYV
> > - YVYU
> > - UYVY
> > The only left combination is VYUY, which is added in this
> > patch.
> >
> > Signed-off-by: Robert Jarzmik <robert.jarzmik@free.fr>
>
> It's fine by me, but since you are making a change anyway, can you move the
> V4L2_PIX_FMT_YVYU define up and put it after V4L2_PIX_FMT_YUYV? Then all
> four combinations are together.
>
> Reviewed-by: Hans Verkuil <hverkuil@xs4all.nl>
>
> Regards,
>
> Hans
>
> > ---
> > include/linux/videodev2.h | 1 +
> > 1 files changed, 1 insertions(+), 0 deletions(-)
> >
> > diff --git a/include/linux/videodev2.h b/include/linux/videodev2.h
> > index 4669d7e..ec311d4 100644
> > --- a/include/linux/videodev2.h
> > +++ b/include/linux/videodev2.h
> > @@ -293,6 +293,7 @@ struct v4l2_pix_format {
> > #define V4L2_PIX_FMT_YVU420 v4l2_fourcc('Y', 'V', '1', '2') /* 12 YVU 4:2:0 */
> > #define V4L2_PIX_FMT_YUYV v4l2_fourcc('Y', 'U', 'Y', 'V') /* 16 YUV 4:2:2 */
> > #define V4L2_PIX_FMT_UYVY v4l2_fourcc('U', 'Y', 'V', 'Y') /* 16 YUV 4:2:2 */
> > +#define V4L2_PIX_FMT_VYUY v4l2_fourcc('V', 'Y', 'U', 'Y') /* 16 YUV 4:2:2 */
> > #define V4L2_PIX_FMT_YUV422P v4l2_fourcc('4', '2', '2', 'P') /* 16 YVU422 planar */
> > #define V4L2_PIX_FMT_YUV411P v4l2_fourcc('4', '1', '1', 'P') /* 16 YVU411 planar */
> > #define V4L2_PIX_FMT_Y41P v4l2_fourcc('Y', '4', '1', 'P') /* 12 YUV 4:1:1 */
> > --
> > 1.5.6.5
> >
> > --
> > video4linux-list mailing list
> > Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
> > https://www.redhat.com/mailman/listinfo/video4linux-list
> >
> >
>
>
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2] Add new pixel format VYUY 16 bits wide.
2008-11-16 22:23 ` [PATCH] Add new pixel format VYUY 16 bits wide Guennadi Liakhovetski
@ 2008-11-16 22:33 ` Robert Jarzmik
2008-11-16 22:39 ` Guennadi Liakhovetski
0 siblings, 1 reply; 8+ messages in thread
From: Robert Jarzmik @ 2008-11-16 22:33 UTC (permalink / raw)
To: hverkuil, g.liakhovetski; +Cc: video4linux-list
There were already 3 YUV formats defined :
- YUYV
- YVYU
- UYVY
The only left combination is VYUY, which is added in this
patch.
As suggested by Hans Verkuil, all YUV 4:2:2 packet formats
were grouped together.
Reviewed-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Robert Jarzmik <robert.jarzmik@free.fr>
---
include/linux/videodev2.h | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/include/linux/videodev2.h b/include/linux/videodev2.h
index 4669d7e..615b05f 100644
--- a/include/linux/videodev2.h
+++ b/include/linux/videodev2.h
@@ -292,7 +292,9 @@ struct v4l2_pix_format {
#define V4L2_PIX_FMT_YVU410 v4l2_fourcc('Y', 'V', 'U', '9') /* 9 YVU 4:1:0 */
#define V4L2_PIX_FMT_YVU420 v4l2_fourcc('Y', 'V', '1', '2') /* 12 YVU 4:2:0 */
#define V4L2_PIX_FMT_YUYV v4l2_fourcc('Y', 'U', 'Y', 'V') /* 16 YUV 4:2:2 */
+#define V4L2_PIX_FMT_YVYU v4l2_fourcc('Y', 'V', 'Y', 'U') /* 16 YVU 4:2:2 */
#define V4L2_PIX_FMT_UYVY v4l2_fourcc('U', 'Y', 'V', 'Y') /* 16 YUV 4:2:2 */
+#define V4L2_PIX_FMT_VYUY v4l2_fourcc('V', 'Y', 'U', 'Y') /* 16 YUV 4:2:2 */
#define V4L2_PIX_FMT_YUV422P v4l2_fourcc('4', '2', '2', 'P') /* 16 YVU422 planar */
#define V4L2_PIX_FMT_YUV411P v4l2_fourcc('4', '1', '1', 'P') /* 16 YVU411 planar */
#define V4L2_PIX_FMT_Y41P v4l2_fourcc('Y', '4', '1', 'P') /* 12 YUV 4:1:1 */
@@ -342,7 +344,6 @@ struct v4l2_pix_format {
#define V4L2_PIX_FMT_SPCA561 v4l2_fourcc('S', '5', '6', '1') /* compressed GBRG bayer */
#define V4L2_PIX_FMT_PAC207 v4l2_fourcc('P', '2', '0', '7') /* compressed BGGR bayer */
#define V4L2_PIX_FMT_PJPG v4l2_fourcc('P', 'J', 'P', 'G') /* Pixart 73xx JPEG */
-#define V4L2_PIX_FMT_YVYU v4l2_fourcc('Y', 'V', 'Y', 'U') /* 16 YVU 4:2:2 */
/*
* F O R M A T E N U M E R A T I O N
--
1.5.6.5
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v2] Add new pixel format VYUY 16 bits wide.
2008-11-16 22:33 ` [PATCH v2] " Robert Jarzmik
@ 2008-11-16 22:39 ` Guennadi Liakhovetski
2008-11-16 22:44 ` Guennadi Liakhovetski
0 siblings, 1 reply; 8+ messages in thread
From: Guennadi Liakhovetski @ 2008-11-16 22:39 UTC (permalink / raw)
To: Robert Jarzmik; +Cc: video4linux-list
On Sun, 16 Nov 2008, Robert Jarzmik wrote:
> There were already 3 YUV formats defined :
> - YUYV
> - YVYU
> - UYVY
> The only left combination is VYUY, which is added in this
> patch.
>
> As suggested by Hans Verkuil, all YUV 4:2:2 packet formats
> were grouped together.
>
> Reviewed-by: Hans Verkuil <hverkuil@xs4all.nl>
> Signed-off-by: Robert Jarzmik <robert.jarzmik@free.fr>
Wow, that was fast!:-) As it doesn't look like we're going to receive a
reply from the Linux fourcc maintainer, I'll just pull it.
Thanks
Guennadi
> ---
> include/linux/videodev2.h | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/include/linux/videodev2.h b/include/linux/videodev2.h
> index 4669d7e..615b05f 100644
> --- a/include/linux/videodev2.h
> +++ b/include/linux/videodev2.h
> @@ -292,7 +292,9 @@ struct v4l2_pix_format {
> #define V4L2_PIX_FMT_YVU410 v4l2_fourcc('Y', 'V', 'U', '9') /* 9 YVU 4:1:0 */
> #define V4L2_PIX_FMT_YVU420 v4l2_fourcc('Y', 'V', '1', '2') /* 12 YVU 4:2:0 */
> #define V4L2_PIX_FMT_YUYV v4l2_fourcc('Y', 'U', 'Y', 'V') /* 16 YUV 4:2:2 */
> +#define V4L2_PIX_FMT_YVYU v4l2_fourcc('Y', 'V', 'Y', 'U') /* 16 YVU 4:2:2 */
> #define V4L2_PIX_FMT_UYVY v4l2_fourcc('U', 'Y', 'V', 'Y') /* 16 YUV 4:2:2 */
> +#define V4L2_PIX_FMT_VYUY v4l2_fourcc('V', 'Y', 'U', 'Y') /* 16 YUV 4:2:2 */
> #define V4L2_PIX_FMT_YUV422P v4l2_fourcc('4', '2', '2', 'P') /* 16 YVU422 planar */
> #define V4L2_PIX_FMT_YUV411P v4l2_fourcc('4', '1', '1', 'P') /* 16 YVU411 planar */
> #define V4L2_PIX_FMT_Y41P v4l2_fourcc('Y', '4', '1', 'P') /* 12 YUV 4:1:1 */
> @@ -342,7 +344,6 @@ struct v4l2_pix_format {
> #define V4L2_PIX_FMT_SPCA561 v4l2_fourcc('S', '5', '6', '1') /* compressed GBRG bayer */
> #define V4L2_PIX_FMT_PAC207 v4l2_fourcc('P', '2', '0', '7') /* compressed BGGR bayer */
> #define V4L2_PIX_FMT_PJPG v4l2_fourcc('P', 'J', 'P', 'G') /* Pixart 73xx JPEG */
> -#define V4L2_PIX_FMT_YVYU v4l2_fourcc('Y', 'V', 'Y', 'U') /* 16 YVU 4:2:2 */
>
> /*
> * F O R M A T E N U M E R A T I O N
> --
> 1.5.6.5
>
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2] Add new pixel format VYUY 16 bits wide.
2008-11-16 22:39 ` Guennadi Liakhovetski
@ 2008-11-16 22:44 ` Guennadi Liakhovetski
0 siblings, 0 replies; 8+ messages in thread
From: Guennadi Liakhovetski @ 2008-11-16 22:44 UTC (permalink / raw)
To: Robert Jarzmik; +Cc: video4linux-list
On Sun, 16 Nov 2008, Guennadi Liakhovetski wrote:
> On Sun, 16 Nov 2008, Robert Jarzmik wrote:
>
> > There were already 3 YUV formats defined :
> > - YUYV
> > - YVYU
> > - UYVY
> > The only left combination is VYUY, which is added in this
> > patch.
> >
> > As suggested by Hans Verkuil, all YUV 4:2:2 packet formats
> > were grouped together.
> >
> > Reviewed-by: Hans Verkuil <hverkuil@xs4all.nl>
> > Signed-off-by: Robert Jarzmik <robert.jarzmik@free.fr>
>
> Wow, that was fast!:-) As it doesn't look like we're going to receive a
> reply from the Linux fourcc maintainer, I'll just pull it.
Ooh... I'm too late - Mauro has already pulled your v1... Ok, then Hans
will have to reorder them himself:-) Sorry for having wasted your time.
Thanks
Guennadi
>
> > ---
> > include/linux/videodev2.h | 3 ++-
> > 1 files changed, 2 insertions(+), 1 deletions(-)
> >
> > diff --git a/include/linux/videodev2.h b/include/linux/videodev2.h
> > index 4669d7e..615b05f 100644
> > --- a/include/linux/videodev2.h
> > +++ b/include/linux/videodev2.h
> > @@ -292,7 +292,9 @@ struct v4l2_pix_format {
> > #define V4L2_PIX_FMT_YVU410 v4l2_fourcc('Y', 'V', 'U', '9') /* 9 YVU 4:1:0 */
> > #define V4L2_PIX_FMT_YVU420 v4l2_fourcc('Y', 'V', '1', '2') /* 12 YVU 4:2:0 */
> > #define V4L2_PIX_FMT_YUYV v4l2_fourcc('Y', 'U', 'Y', 'V') /* 16 YUV 4:2:2 */
> > +#define V4L2_PIX_FMT_YVYU v4l2_fourcc('Y', 'V', 'Y', 'U') /* 16 YVU 4:2:2 */
> > #define V4L2_PIX_FMT_UYVY v4l2_fourcc('U', 'Y', 'V', 'Y') /* 16 YUV 4:2:2 */
> > +#define V4L2_PIX_FMT_VYUY v4l2_fourcc('V', 'Y', 'U', 'Y') /* 16 YUV 4:2:2 */
> > #define V4L2_PIX_FMT_YUV422P v4l2_fourcc('4', '2', '2', 'P') /* 16 YVU422 planar */
> > #define V4L2_PIX_FMT_YUV411P v4l2_fourcc('4', '1', '1', 'P') /* 16 YVU411 planar */
> > #define V4L2_PIX_FMT_Y41P v4l2_fourcc('Y', '4', '1', 'P') /* 12 YUV 4:1:1 */
> > @@ -342,7 +344,6 @@ struct v4l2_pix_format {
> > #define V4L2_PIX_FMT_SPCA561 v4l2_fourcc('S', '5', '6', '1') /* compressed GBRG bayer */
> > #define V4L2_PIX_FMT_PAC207 v4l2_fourcc('P', '2', '0', '7') /* compressed BGGR bayer */
> > #define V4L2_PIX_FMT_PJPG v4l2_fourcc('P', 'J', 'P', 'G') /* Pixart 73xx JPEG */
> > -#define V4L2_PIX_FMT_YVYU v4l2_fourcc('Y', 'V', 'Y', 'U') /* 16 YVU 4:2:2 */
> >
> > /*
> > * F O R M A T E N U M E R A T I O N
> > --
> > 1.5.6.5
> >
>
> ---
> Guennadi Liakhovetski, Ph.D.
> Freelance Open-Source Software Developer
>
> --
> video4linux-list mailing list
> Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
> https://www.redhat.com/mailman/listinfo/video4linux-list
>
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2008-11-16 22:44 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-11-06 23:04 [PATCH] pxa_camera: Fix YUV format handling Robert Jarzmik
2008-11-06 23:52 ` Guennadi Liakhovetski
2008-11-07 7:10 ` Antonio Ospite
2008-11-07 17:14 ` Robert Jarzmik
2008-11-16 22:23 ` [PATCH] Add new pixel format VYUY 16 bits wide Guennadi Liakhovetski
2008-11-16 22:33 ` [PATCH v2] " Robert Jarzmik
2008-11-16 22:39 ` Guennadi Liakhovetski
2008-11-16 22:44 ` Guennadi Liakhovetski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox