+Hans Verkuil (I think he commented on earlier revisions. Please keep anyone who commented before on CC when sending further revisions.)
On Mon, Sep 17, 2018 at 5:04 AM Zhi, Yong <[email protected]> wrote: > > Hi, Tomasz, > > Sorry for the delay in responding to your review. > > > -----Original Message----- > > From: [email protected] [mailto:linux-media- > > [email protected]] On Behalf Of Tomasz Figa > > Sent: Monday, July 2, 2018 2:50 AM > > To: Zhi, Yong <[email protected]> > > Cc: Linux Media Mailing List <[email protected]>; Sakari Ailus > > <[email protected]>; Mani, Rajmohan > > <[email protected]>; Toivonen, Tuukka > > <[email protected]>; Hu, Jerry W <[email protected]>; Zheng, > > Jian Xu <[email protected]>; Vijaykumar, Ramya > > <[email protected]> > > Subject: Re: [PATCH v6 11/12] intel-ipu3: Add v4l2 driver based on media > > framework > > > > Hi Yong, > > > > On Fri, Mar 30, 2018 at 11:15 AM Yong Zhi <[email protected]> wrote: > > [snip] > > > +static int ipu3_vidioc_enum_input(struct file *file, void *fh, > > > + struct v4l2_input *input) { > > > + if (input->index > 0) > > > + return -EINVAL; > > > + strlcpy(input->name, "camera", sizeof(input->name)); > > > + input->type = V4L2_INPUT_TYPE_CAMERA; > > > + > > > + return 0; > > > +} > > > + > > > +static int ipu3_vidioc_g_input(struct file *file, void *fh, unsigned > > > +int *input) { > > > + *input = 0; > > > + > > > + return 0; > > > +} > > > + > > > +static int ipu3_vidioc_s_input(struct file *file, void *fh, unsigned > > > +int input) { > > > + return input == 0 ? 0 : -EINVAL; } > > > + > > > +static int ipu3_vidioc_enum_output(struct file *file, void *fh, > > > + struct v4l2_output *output) { > > > + if (output->index > 0) > > > + return -EINVAL; > > > + strlcpy(output->name, "camera", sizeof(output->name)); > > > + output->type = V4L2_INPUT_TYPE_CAMERA; > > > + > > > + return 0; > > > +} > > > + > > > +static int ipu3_vidioc_g_output(struct file *file, void *fh, > > > + unsigned int *output) { > > > + *output = 0; > > > + > > > + return 0; > > > +} > > > + > > > +static int ipu3_vidioc_s_output(struct file *file, void *fh, > > > + unsigned int output) { > > > + return output == 0 ? 0 : -EINVAL; } > > > > Do we really need to implement the 6 functions above? They don't seem to > > be doing anything useful. > > > > They are here to pass v4l2-compliance test. I can add a note in next update > for their purpose. We can remove them in the future when defaults callbacks > are available for those ops. > Strange. Hans, is it really mandatory to implement dmmy output/input setting if there is no output/input switching capability? Best regards, Tomasz
