On 20/01/14 13:45, Hans Verkuil wrote: > From: Hans Verkuil <hans.verkuil@xxxxxxxxx> > > This patch implements initial support for complex types. > > For the most part the changes are fairly obvious (basic support for is_ptr > types, the type_is_int function is replaced by a is_int bitfield, and > v4l2_query_ext_ctrl is added), but one change needs more explanation: > > The v4l2_ctrl struct adds a 'new' field and a 'stores' array at the end > of the struct. This is in preparation for future patches where each control > can have multiple configuration stores. The idea is that stores[0] is the current > control value, stores[1] etc. are the control values for each configuration store > and the 'new' value can be accessed through 'stores[-1]', i.e. the 'new' field. > However, for now only stores[-1] and stores[0] is used. I guess it implies an assumption that (maximum) number of configuration stores is known before creating the control ? Regarding the negative array indexes, I guess I would just stick with using the 'new' field :-) > These new fields use the v4l2_ctrl_ptr union, which is a pointer to a control > value. > > Note that these two new fields are not yet actually used. > > Signed-off-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> Reviewed-by: Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx> -- Regards, Sylwester -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
