| Message ID | 1416905668-23029-1-git-send-email-josh.wu@atmel.com (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Headers |
Received: from mail.tu-berlin.de ([130.149.7.33]) by www.linuxtv.org with esmtp (Exim 4.72) (envelope-from <linux-media-owner@vger.kernel.org>) id 1XtBu3-0003WP-E6; Tue, 25 Nov 2014 09:55:19 +0100 X-tubIT-Incoming-IP: 209.132.180.67 Received: from vger.kernel.org ([209.132.180.67]) by mail.tu-berlin.de (exim-4.72/mailfrontend-5) with esmtp id 1XtBu0-0004Q4-8S; Tue, 25 Nov 2014 09:55:18 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750960AbaKYIyx (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Tue, 25 Nov 2014 03:54:53 -0500 Received: from eusmtp01.atmel.com ([212.144.249.243]:42558 "EHLO eusmtp01.atmel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750901AbaKYIyw (ORCPT <rfc822;linux-media@vger.kernel.org>); Tue, 25 Nov 2014 03:54:52 -0500 Received: from apsmtp01.atmel.com (10.168.254.30) by eusmtp01.atmel.com (10.161.101.31) with Microsoft SMTP Server id 14.2.347.0; Tue, 25 Nov 2014 09:54:46 +0100 Received: from melon.corp.atmel.com (10.168.254.13) by apsmtp01.corp.atmel.com (10.168.254.30) with Microsoft SMTP Server id 14.2.347.0; Tue, 25 Nov 2014 16:55:18 +0800 From: Josh Wu <josh.wu@atmel.com> To: <linux-media@vger.kernel.org> CC: <m.chehab@samsung.com>, <linux-kernel@vger.kernel.org>, <g.liakhovetski@gmx.de>, Josh Wu <josh.wu@atmel.com> Subject: [PATCH 1/2] media: v4l2-image-sizes.h: add SVGA, XGA and UXGA size definitions Date: Tue, 25 Nov 2014 16:54:27 +0800 Message-ID: <1416905668-23029-1-git-send-email-josh.wu@atmel.com> X-Mailer: git-send-email 1.9.1 MIME-Version: 1.0 Content-Type: text/plain Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-PMX-Version: 6.0.0.2142326, Antispam-Engine: 2.7.2.2107409, Antispam-Data: 2014.11.25.84821 X-PMX-Spam: Gauge=IIIIIIIII, Probability=9%, Report=' MULTIPLE_RCPTS 0.1, HTML_00_01 0.05, HTML_00_10 0.05, MSGID_ADDED_BY_MTA 0.05, BODYTEXTP_SIZE_3000_LESS 0, BODY_SIZE_1000_LESS 0, BODY_SIZE_2000_LESS 0, BODY_SIZE_5000_LESS 0, BODY_SIZE_7000_LESS 0, BODY_SIZE_900_999 0, URI_ENDS_IN_HTML 0, __ANY_URI 0, __CP_MEDIA_BODY 0, __CP_URI_IN_BODY 0, __CT 0, __CT_TEXT_PLAIN 0, __DATE_TZ_HK 0, __HAS_FROM 0, __HAS_MSGID 0, __HAS_X_MAILER 0, __HAS_X_MAILING_LIST 0, __MIME_TEXT_ONLY 0, __MIME_VERSION 0, __MULTIPLE_RCPTS_CC_X2 0, __SANE_MSGID 0, __TO_MALFORMED_2 0, __TO_NO_NAME 0, __URI_NO_WWW 0, __URI_NS ' |
Commit Message
Josh Wu
Nov. 25, 2014, 8:54 a.m. UTC
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@atmel.com>
---
include/media/v4l2-image-sizes.h | 9 +++++++++
1 file changed, 9 insertions(+)
Comments
Hi Josh, On 25/11/14 09:54, 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@atmel.com> > --- > 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 I think this should be 600. With that fixed, for both patches: Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com> > #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 */ -- Regards, Sylwester -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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@atmel.com> 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? 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-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi, Sylwester and Mauro On 11/25/2014 6:34 PM, Sylwester Nawrocki wrote: > Hi Josh, > > On 25/11/14 09:54, 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@atmel.com> >> --- >> 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 > I think this should be 600. With that fixed, for both patches: Yes, right, It should be 600. It's my bad with such terrible typo here. Hi, Mauro I saw this patch is already merged in the media_tree. But not changing the SVGA_HEIGHT to 600. Would it possible for you to re-modify this commit in the media_tree to fix the SVGA_HEIGHT as 600? Or need I resend the patch or a fix for this? Sorry for such an inconvinencie. > > Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com> Thanks a again. Best Regards, Josh Wu > >> #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 */ > -- > Regards, > Sylwester -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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@atmel.com> > 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? 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-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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@atmel.com> > > 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! 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-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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@atmel.com> >>> 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-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Josh, On Wed, 28 Jan 2015, Josh Wu wrote: > 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@atmel.com> > > > > 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? uhm, yes, sorry, I'll try to make sure not to miss it with my next pull request. Thanks for a reminder! Regards Guennadi > 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-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 */