Em Sat, 5 Jan 2013 14:34:04 +0100
Hans Verkuil <hverk...@xs4all.nl> escreveu:
> On Sat January 5 2013 03:54:44 Mauro Carvalho Chehab wrote:
> > Hans/Devin,
> >
> > Em Fri, 4 Jan 2013 15:59:40 -0500
> > Devin Heitmueller <dheitmuel...@kernellabs.com> escreveu:
> >
> > > TRY_FMT should not return an error if a pixelformat is unsupported.
> > > Instead just
> > > pick a common pixelformat.
> > >
> > > Also the bytesperline calculation was incorrect: it used the old width
> > > instead of
> > > the provided with, and it miscalculated the bytesperline value for the
> > > depth == 12
> > > case.
> > >
> > > Signed-off-by: Hans Verkuil <hans.verk...@cisco.com>
> > > Signed-off-by: Devin Heitmueller <dheitmuel...@kernellabs.com>
> > > ---
> > > drivers/media/usb/em28xx/em28xx-video.c | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/media/usb/em28xx/em28xx-video.c
> > > b/drivers/media/usb/em28xx/em28xx-video.c
> > > index a91a248..7c09b55 100644
> > > --- a/drivers/media/usb/em28xx/em28xx-video.c
> > > +++ b/drivers/media/usb/em28xx/em28xx-video.c
> > > @@ -821,7 +821,7 @@ static int vidioc_try_fmt_vid_cap(struct file *file,
> > > void *priv,
> > > if (!fmt) {
> > > em28xx_videodbg("Fourcc format (%08x) invalid.\n",
> > > f->fmt.pix.pixelformat);
> > > - return -EINVAL;
> > > + fmt = format_by_fourcc(V4L2_PIX_FMT_YUYV);
> >
> > This change has the potential of causing userspace regressions, so,
> > for now, I won't apply such change.
>
> Good!
>
> > We need to discuss it better, before risk breaking things, and likely fix
> > applications first.
>
> Absolutely. I also want to change this test in v4l2-compliance from 'failure'
> to 'warning' for the time being.
Sounds reasonable for me.
Regards,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
