lkml.org 
[lkml]   [2014]   [Nov]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/2] media: v4l2-image-sizes.h: add SVGA, XGA and UXGA size definitions
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



\
 
 \ /
  Last update: 2014-11-26 03:41    [W:0.178 / U:0.128 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site