Re: [PATCH v7] [media] vimc: Virtual Media Controller core, capture and sensor

[Mauro Carvalho Chehab <mchehab@s-opensource.com>]

Em Mon, 27 Mar 2017 12:19:51 -0300
Helen Koike <helen.koike@collabora.co.uk> escreveu:

> Hi Sakari,
> 
> On 2017-03-26 10:31 AM, Sakari Ailus wrote:
> > Hi Helen,
> >
> > ...  
> >> +static int vimc_cap_enum_input(struct file *file, void *priv,
> >> +			       struct v4l2_input *i)
> >> +{
> >> +	/* We only have one input */
> >> +	if (i->index > 0)
> >> +		return -EINVAL;
> >> +
> >> +	i->type = V4L2_INPUT_TYPE_CAMERA;
> >> +	strlcpy(i->name, "VIMC capture", sizeof(i->name));
> >> +
> >> +	return 0;
> >> +}
> >> +
> >> +static int vimc_cap_g_input(struct file *file, void *priv, unsigned int *i)
> >> +{
> >> +	/* We only have one input */
> >> +	*i = 0;
> >> +	return 0;
> >> +}
> >> +
> >> +static int vimc_cap_s_input(struct file *file, void *priv, unsigned int i)
> >> +{
> >> +	/* We only have one input */
> >> +	return i ? -EINVAL : 0;
> >> +}  
> >
> > You can drop the input IOCTLs altogether here. If you had e.g. a TV
> > tuner, it'd be the TV tuner driver's responsibility to implement them.
> >  
> 
> input IOCTLs seems to be mandatory from v4l2-compliance when capability 
> V4L2_CAP_VIDEO_CAPTURE is set (which is the case):
> 
> https://git.linuxtv.org/v4l-utils.git/tree/utils/v4l2-compliance/v4l2-test-input-output.cpp#n418
> 
> https://git.linuxtv.org/v4l-utils.git/tree/utils/v4l2-compliance/v4l2-compliance.cpp#n989

The V4L2 spec doesn't actually define what's mandatory and what's
optional. The idea that was agreed on one of the media summits
were to define a set of profiles for different device types,
matching the features required by existing applications to work,
but this was never materialized.

So, my understanding is that any driver can implement
any V4L2 ioctl.

Yet, some applications require enum/get/set inputs, or otherwise
they wouldn't work. It is too late to change this behavior. 
So, either the driver or the core should implement those
ioctls, in order to avoid breaking backward-compatibility.

Regards,

Thanks,
Mauro