Re: [PATCHv4 1/2] v4l: Add new framesamples field to struct v4l2_mbus_framefmt

2012-01-11 Thread Laurent Pinchart
Hi Sylwester and Sakari, On Sunday 01 January 2012 19:56:03 Sylwester Nawrocki wrote: > On 12/31/2011 02:16 PM, Sakari Ailus wrote: > >>> I could think of an in-kernel counterpart for v4l2_mbus_framefmt, say, > >>> v4l2_mbus_framedesc. This could then be passed from subdev to another > >>> using a

Re: [PATCHv4 1/2] v4l: Add new framesamples field to struct v4l2_mbus_framefmt

2012-01-06 Thread Sakari Ailus
Hi Sylwester, On Wed, Jan 04, 2012 at 11:51:00PM +0100, Sylwester Nawrocki wrote: > On 01/04/2012 01:21 PM, Sakari Ailus wrote: > > On Sun, Jan 01, 2012 at 07:56:03PM +0100, Sylwester Nawrocki wrote: > >> On 12/31/2011 02:16 PM, Sakari Ailus wrote: > > Something else that should probably belon

Re: [PATCHv4 1/2] v4l: Add new framesamples field to struct v4l2_mbus_framefmt

2012-01-06 Thread Sylwester Nawrocki
Hi Laurent, On 12/21/2011 01:20 AM, Laurent Pinchart wrote: > On Wednesday 14 December 2011 13:23:07 Sylwester Nawrocki wrote: >> The purpose of the new field is to allow the video pipeline elements >> to negotiate memory buffer size for compressed data frames, where >> the buffer size cannot be d

Re: [PATCHv4 1/2] v4l: Add new framesamples field to struct v4l2_mbus_framefmt

2012-01-04 Thread Sylwester Nawrocki
Hi Sakari, On 01/04/2012 01:21 PM, Sakari Ailus wrote: > On Sun, Jan 01, 2012 at 07:56:03PM +0100, Sylwester Nawrocki wrote: >> On 12/31/2011 02:16 PM, Sakari Ailus wrote: > Something else that should probably belong there is information on the > frame > format: contrary to what I've

Re: [PATCHv4 1/2] v4l: Add new framesamples field to struct v4l2_mbus_framefmt

2012-01-04 Thread Sakari Ailus
Hi Sylwester, On Sun, Jan 01, 2012 at 07:56:03PM +0100, Sylwester Nawrocki wrote: > Hi Sakari, > > On 12/31/2011 02:16 PM, Sakari Ailus wrote: > >>> I could think of an in-kernel counterpart for v4l2_mbus_framefmt, say, > >>> v4l2_mbus_framedesc. This could then be passed from subdev to another

Re: [PATCHv4 1/2] v4l: Add new framesamples field to struct v4l2_mbus_framefmt

2012-01-01 Thread Sylwester Nawrocki
Hi Sakari, On 12/31/2011 02:16 PM, Sakari Ailus wrote: >>> I could think of an in-kernel counterpart for v4l2_mbus_framefmt, say, >>> v4l2_mbus_framedesc. This could then be passed from subdev to another using >>> a new subdev op. >> >> That might be needed eventually. But I'm not a great fan in g

Re: [PATCHv4 1/2] v4l: Add new framesamples field to struct v4l2_mbus_framefmt

2011-12-31 Thread Sakari Ailus
Hi Sylwester, On Wed, Dec 28, 2011 at 06:09:17PM +0100, Sylwester Nawrocki wrote: > On 12/26/2011 01:53 PM, Sakari Ailus wrote: > > On Wed, Dec 21, 2011 at 01:20:56AM +0100, Laurent Pinchart wrote: > >> On Wednesday 14 December 2011 13:23:07 Sylwester Nawrocki wrote: > >>> The purpose of the new f

Re: [PATCHv4 1/2] v4l: Add new framesamples field to struct v4l2_mbus_framefmt

2011-12-28 Thread Sylwester Nawrocki
Hi Sakari, On 12/26/2011 01:53 PM, Sakari Ailus wrote: > On Wed, Dec 21, 2011 at 01:20:56AM +0100, Laurent Pinchart wrote: >> On Wednesday 14 December 2011 13:23:07 Sylwester Nawrocki wrote: >>> The purpose of the new field is to allow the video pipeline elements >>> to negotiate memory buffer siz

Re: [PATCHv4 1/2] v4l: Add new framesamples field to struct v4l2_mbus_framefmt

2011-12-26 Thread Sakari Ailus
Hi Laurent and Sylwester, On Wed, Dec 21, 2011 at 01:20:56AM +0100, Laurent Pinchart wrote: > Hi Sylwester, > > On Wednesday 14 December 2011 13:23:07 Sylwester Nawrocki wrote: > > The purpose of the new field is to allow the video pipeline elements > > to negotiate memory buffer size for compres

Re: [PATCHv4 1/2] v4l: Add new framesamples field to struct v4l2_mbus_framefmt

2011-12-22 Thread Sylwester Nawrocki
Hi Laurent, On 12/21/2011 01:20 AM, Laurent Pinchart wrote: > On Wednesday 14 December 2011 13:23:07 Sylwester Nawrocki wrote: >> The purpose of the new field is to allow the video pipeline elements >> to negotiate memory buffer size for compressed data frames, where >> the buffer size cannot be d

Re: [PATCHv4 1/2] v4l: Add new framesamples field to struct v4l2_mbus_framefmt

2011-12-20 Thread Laurent Pinchart
Hi Sylwester, On Wednesday 14 December 2011 13:23:07 Sylwester Nawrocki wrote: > The purpose of the new field is to allow the video pipeline elements > to negotiate memory buffer size for compressed data frames, where > the buffer size cannot be derived from pixel width and height and > the pixel

[PATCHv4 1/2] v4l: Add new framesamples field to struct v4l2_mbus_framefmt

2011-12-14 Thread Sylwester Nawrocki
The purpose of the new field is to allow the video pipeline elements to negotiate memory buffer size for compressed data frames, where the buffer size cannot be derived from pixel width and height and the pixel code. For VIDIOC_SUBDEV_S_FMT and VIDIOC_SUBDEV_G_FMT ioctls, the framesamples paramete