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.
Regards,
Hans
--
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
