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

2013-11-11 Thread Hans Verkuil
Hi Ricardo, A few comments below... On 11/06/2013 05:44 PM, 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: Ricard

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

2013-11-06 Thread Sakari Ailus
Hi Ricardo, Thanks for the update. (Dropping LKML again.) On Wed, Nov 06, 2013 at 05:44:38PM +0100, Ricardo Ribalda Delgado wrote: > As addressed on the media summit 2013, there is no reason for the width s/addressed/discussed/ ? > and height to be signed. > > Therefore this patch is an attemp

[PATCH v4] 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 --- v3: Comments by Sakari -Update also doc v2: Comments by Sakari Ailus and Laurent