Re: [PATCH 1/6] omap3isp: video: Split format info bpp field into width and bpp

2012-06-27 Thread Sakari Ailus
Laurent Pinchart wrote: > Hi Sakari, > > Thanks for the review. > > On Wednesday 27 June 2012 14:07:51 Sakari Ailus wrote: >> Laurent Pinchart wrote: >>> The bpp field currently stores the sample width and is aligned to the >>> next multiple of 8 bits when computing data size in memory. This won'

Re: [PATCH 1/6] omap3isp: video: Split format info bpp field into width and bpp

2012-06-27 Thread Laurent Pinchart
On Wednesday 27 June 2012 13:54:30 Laurent Pinchart wrote: > Hi Sakari, > > Thanks for the review. > > On Wednesday 27 June 2012 14:07:51 Sakari Ailus wrote: > > Laurent Pinchart wrote: > > > The bpp field currently stores the sample width and is aligned to the > > > next multiple of 8 bits when

Re: [PATCH 1/6] omap3isp: video: Split format info bpp field into width and bpp

2012-06-27 Thread Laurent Pinchart
Hi Sakari, Thanks for the review. On Wednesday 27 June 2012 14:07:51 Sakari Ailus wrote: > Laurent Pinchart wrote: > > The bpp field currently stores the sample width and is aligned to the > > next multiple of 8 bits when computing data size in memory. This won't > > work anymore for YUYV8_2X8 fo

Re: [PATCH 1/6] omap3isp: video: Split format info bpp field into width and bpp

2012-06-27 Thread Sakari Ailus
Hi Laurent, Thanks for the patch. Laurent Pinchart wrote: > The bpp field currently stores the sample width and is aligned to the > next multiple of 8 bits when computing data size in memory. This won't > work anymore for YUYV8_2X8 formats. Split the bpp field into a sample > width and a bits per

[PATCH 1/6] omap3isp: video: Split format info bpp field into width and bpp

2012-06-26 Thread Laurent Pinchart
The bpp field currently stores the sample width and is aligned to the next multiple of 8 bits when computing data size in memory. This won't work anymore for YUYV8_2X8 formats. Split the bpp field into a sample width and a bits per pixel value. Signed-off-by: Laurent Pinchart --- drivers/media/v