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

2013-11-06 Thread Sakari Ailus
Hi Ricardo, On Wed, Nov 06, 2013 at 04:56:12PM +0100, Ricardo Ribalda Delgado wrote: > It has to be done in the same patch? or on a separated patch just > changing the xml file? Good question. Now that you ask, I also realise the documentation must also be changed --- struct v4l2_rect is documen

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

2013-11-06 Thread Ricardo Ribalda Delgado
Hello Sakai It has to be done in the same patch? or on a separated patch just changing the xml file? Thanks! On Wed, Nov 6, 2013 at 4:53 PM, Sakari Ailus wrote: > On Wed, Nov 06, 2013 at 04:43:04PM +0100, Ricardo Ribalda Delgado wrote: >> As addressed on the media summit 2013, there is no reas

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

2013-11-06 Thread Sakari Ailus
On Wed, Nov 06, 2013 at 04:43:04PM +0100, 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. > > Signed-off-by: Ricardo Ribalda Delgado

[PATCH v2] 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 --- v2: Comments by Sakari Ailus and Laurent Pinchart -Fix alignment on all drivers -

[PATCH v2] 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 --- v2: Comments by Sakari Ailus and Laurent Pinchart -Fix alignment on all drivers -