On Tuesday, January 11, 2011 23:11:48 Mauro Carvalho Chehab wrote:
> Em 07-01-2011 08:44, Hans Verkuil escreveu:
> > On Friday, January 07, 2011 10:49:30 Hans Verkuil wrote:
> 
> > So this patch series is OK to merge.
> 
> >>> The following changes since commit 
> >>> 187134a5875df20356f4dca075db29f294115a47:
> >>>   David Henningsson (1):
> >>>         [media] DVB: IR support for TechnoTrend CT-3650
> >>>
> >>> are available in the git repository at:
> >>>
> >>>   ssh://linuxtv.org/git/hverkuil/media_tree.git subdev-ctrl1
> >>>
> >>> Hans Verkuil (11):
> >>>       vivi: convert to the control framework and add test controls.
> 
> 
> Hmm.. this patch is producing a new warning:
> drivers/media/video/vivi.c:1059: warning: this decimal constant is unsigned 
> only in ISO C90
> 
> Please fix.
> Mauro.

Here it is:

vivi: fix compiler warning

Signed-off-by: Hans Verkuil <[email protected]>

Regards,

        Hans

diff --git a/drivers/media/video/vivi.c b/drivers/media/video/vivi.c
index 598acfe..6beb270 100644
--- a/drivers/media/video/vivi.c
+++ b/drivers/media/video/vivi.c
@@ -1056,8 +1056,8 @@ static const struct v4l2_ctrl_config vivi_ctrl_int32 = {
        .id = VIVI_CID_CUSTOM_BASE + 2,
        .name = "Integer 32 Bits",
        .type = V4L2_CTRL_TYPE_INTEGER,
-       .min = -2147483648,
-       .max = 2147483647,
+       .min = 0x80000000,
+       .max = 0x7f000000,
        .step = 1,
 };
 


-- 
Hans Verkuil - video4linux developer - sponsored by Cisco
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to