Re: [PATCH 1/2] media: v4l2-image-sizes.h: add SVGA, XGA and UXGA size definitions

From: Josh Wu
Date: Tue Jan 27 2015 - 21:09:37 EST


Hi, Guennadi

On 11/28/2014 4:13 AM, Guennadi Liakhovetski wrote:
Hi Josh,

On Thu, 27 Nov 2014, Josh Wu wrote:

Hi, Guennadi

On 11/26/2014 6:23 AM, Guennadi Liakhovetski wrote:
Hi Josh,

On Tue, 25 Nov 2014, Josh Wu wrote:

Add SVGA, UXGA and XGA size definitions to v4l2-image-sizes.h.
The definitions are sorted by alphabet order.

Signed-off-by: Josh Wu <josh.wu@xxxxxxxxx>
Thanks for your patches. I'm ok with these two, but the second of them
depends on the first one, and the first one wouldn't (normally) be going
via the soc-camera tree. Mauro, how would you prefer to handle this?
Should I pick up and push to you both of them or postpone #2 until the
next merge window?
The first patch is already merged in the media_tree. If the soc-camera tree
will be merged to the media_tree, then there should have no dependency issue.
Am I understanding correct?
Yes, then it should be ok!

Just checking the status of this patch. I don't found this patch in media's tree or soc_camera's tree.
Could you take this patch in your tree?

Best Regards,
Josh Wu


Thanks
Guennadi

Best Regards,
Josh Wu

Thanks
Guennadi

---
include/media/v4l2-image-sizes.h | 9 +++++++++
1 file changed, 9 insertions(+)

diff --git a/include/media/v4l2-image-sizes.h
b/include/media/v4l2-image-sizes.h
index 10daf92..c70c917 100644
--- a/include/media/v4l2-image-sizes.h
+++ b/include/media/v4l2-image-sizes.h
@@ -25,10 +25,19 @@
#define QVGA_WIDTH 320
#define QVGA_HEIGHT 240
+#define SVGA_WIDTH 800
+#define SVGA_HEIGHT 680
+
#define SXGA_WIDTH 1280
#define SXGA_HEIGHT 1024
#define VGA_WIDTH 640
#define VGA_HEIGHT 480
+#define UXGA_WIDTH 1600
+#define UXGA_HEIGHT 1200
+
+#define XGA_WIDTH 1024
+#define XGA_HEIGHT 768
+
#endif /* _IMAGE_SIZES_H */
--
1.9.1


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/