Re: [PATCH] em28xx: Fix height setting on non-progressive captures

2012-08-12 Thread Mauro Carvalho Chehab
Em 03-08-2012 14:52, Ezequiel Garcia escreveu: > This was introduced on commit c2a6b54a9: > "em28xx: fix: don't do image interlacing on webcams" > It is a known bug that has already been reported several times > and confirmed by Mauro. Thanks for reminding us about that. > Tested by compilation o

Re: [PATCH] em28xx: Fix height setting on non-progressive captures

2012-08-12 Thread Mauro Carvalho Chehab
Em 04-08-2012 05:53, llar...@gmx.net escreveu: Wait a minute, unless I completely misunderstood the bug (which is possible), I think this patch is straightforward. By the look of this hunk on commit c2a6b54a: -8<---

Re: [PATCH] em28xx: Fix height setting on non-progressive captures

2012-08-04 Thread llar...@gmx.net
> >> Wait a minute, unless I completely misunderstood the bug (which is > >> possible), > >> I think this patch is straightforward. > >> > >> By the look of this hunk on commit c2a6b54a: > >> > >> -8<-- > >> diff --git a/drivers/media/video/e

Re: [PATCH] em28xx: Fix height setting on non-progressive captures

2012-08-03 Thread Ezequiel Garcia
On Fri, Aug 3, 2012 at 3:55 PM, Devin Heitmueller wrote: > On Fri, Aug 3, 2012 at 2:42 PM, Ezequiel Garcia wrote: >> Wait a minute, unless I completely misunderstood the bug (which is possible), >> I think this patch is straightforward. >> >> By the look of this hunk on commit c2a6b54a: >> >> ---

Re: [PATCH] em28xx: Fix height setting on non-progressive captures

2012-08-03 Thread Devin Heitmueller
On Fri, Aug 3, 2012 at 2:42 PM, Ezequiel Garcia wrote: > Wait a minute, unless I completely misunderstood the bug (which is possible), > I think this patch is straightforward. > > By the look of this hunk on commit c2a6b54a: > > -8<-- > diff

Re: [PATCH] em28xx: Fix height setting on non-progressive captures

2012-08-03 Thread Ezequiel Garcia
Hi Devin, Thanks for answering. On Fri, Aug 3, 2012 at 3:26 PM, Devin Heitmueller wrote: > On Fri, Aug 3, 2012 at 2:11 PM, Ezequiel Garcia wrote: >> On Fri, Aug 3, 2012 at 2:52 PM, Ezequiel Garcia >> wrote: >>> This was introduced on commit c2a6b54a9: >>> "em28xx: fix: don't do image interlac

Re: [PATCH] em28xx: Fix height setting on non-progressive captures

2012-08-03 Thread Devin Heitmueller
On Fri, Aug 3, 2012 at 2:11 PM, Ezequiel Garcia wrote: > On Fri, Aug 3, 2012 at 2:52 PM, Ezequiel Garcia wrote: >> This was introduced on commit c2a6b54a9: >> "em28xx: fix: don't do image interlacing on webcams" >> It is a known bug that has already been reported several times >> and confirmed by

Re: [PATCH] em28xx: Fix height setting on non-progressive captures

2012-08-03 Thread Ezequiel Garcia
On Fri, Aug 3, 2012 at 2:52 PM, Ezequiel Garcia wrote: > This was introduced on commit c2a6b54a9: > "em28xx: fix: don't do image interlacing on webcams" > It is a known bug that has already been reported several times > and confirmed by Mauro. > Tested by compilation only. > I wonder if it's poss