Re: [PATCH] videodev2: Set vb2_rect's width and height as unsigned

2013-11-06 Thread Laurent Pinchart
Hi Ricardo, Thank you for the patch. On Wednesday 06 November 2013 13:49:25 Ricardo Ribalda Delgado wrote: > As addressed on the media summit 2013, there is no reason for the width > and height to be signed. > > Therefore this patch is an attempt to convert those fields into unsigned. > > Signe

Re: [PATCH] videodev2: Set vb2_rect's width and height as unsigned

2013-11-06 Thread Ricardo Ribalda Delgado
Hello Sakari. Thanks for your comments. I have fixed them, but I will wait some time to get more comments, to avoid spamming the list. Thanks! On Wed, Nov 6, 2013 at 3:19 PM, Sakari Ailus wrote: > Hi Ricardo, > > Thanks for the patch. > > On Wed, Nov 06, 2013 at 01:49:25PM +0100, Ricardo Ribald

Re: [PATCH] videodev2: Set vb2_rect's width and height as unsigned

2013-11-06 Thread Sakari Ailus
Hi Ricardo, Thanks for the patch. On Wed, Nov 06, 2013 at 01:49:25PM +0100, Ricardo Ribalda Delgado wrote: ... > diff --git a/drivers/media/i2c/smiapp/smiapp-core.c > b/drivers/media/i2c/smiapp/smiapp-core.c > index ae66d91..9036e64 100644 > --- a/drivers/media/i2c/smiapp/smiapp-core.c > +++ b/d

[PATCH] videodev2: Set vb2_rect's width and height as unsigned

2013-11-06 Thread Ricardo Ribalda Delgado
As addressed on the media summit 2013, there is no reason for the width and height to be signed. Therefore this patch is an attempt to convert those fields into unsigned. Signed-off-by: Ricardo Ribalda Delgado --- drivers/media/i2c/mt9m032.c| 16 +++- drivers/med