Re: [PATCH 10/15] em28xx: fix broken TRY_FMT.

2013-01-05 Thread Mauro Carvalho Chehab
Em Sat, 5 Jan 2013 14:34:04 +0100 Hans Verkuil 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 escreveu: > > > > > TRY_FMT should not return an error if a pixelformat is unsupported. > > >

Re: [PATCH 10/15] em28xx: fix broken TRY_FMT.

2013-01-05 Thread Hans Verkuil
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 escreveu: > > > TRY_FMT should not return an error if a pixelformat is unsupported. Instead > > just > > pick a common pixelformat. > > > > Also the bytesperline

Re: [PATCH 10/15] em28xx: fix broken TRY_FMT.

2013-01-04 Thread Mauro Carvalho Chehab
Hans/Devin, Em Fri, 4 Jan 2013 15:59:40 -0500 Devin Heitmueller 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 w

[PATCH 10/15] em28xx: fix broken TRY_FMT.

2013-01-04 Thread Devin Heitmueller
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: H