Re: [PATCH] vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME

2014-10-29 Thread Mauro Carvalho Chehab
Em Wed, 29 Oct 2014 14:53:53 +0100 Hans Verkuil escreveu: > On 10/29/14 14:17, Laurent Pinchart wrote: > > Hi Mauro, > > > > On Wednesday 29 October 2014 11:05:34 Mauro Carvalho Chehab wrote: > >> Em Wed, 29 Oct 2014 14:46:55 +0200 Laurent Pinchart escreveu: > > Hmm, so you think VIDEO_MAX_F

Re: [PATCH] vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME

2014-10-29 Thread Hans Verkuil
On 10/29/14 14:17, Laurent Pinchart wrote: > Hi Mauro, > > On Wednesday 29 October 2014 11:05:34 Mauro Carvalho Chehab wrote: >> Em Wed, 29 Oct 2014 14:46:55 +0200 Laurent Pinchart escreveu: > Hmm, so you think VIDEO_MAX_FRAME should just be updated to 64? Yes. > I am a bit

Re: [PATCH] vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME

2014-10-29 Thread Laurent Pinchart
Hi Mauro, On Wednesday 29 October 2014 11:05:34 Mauro Carvalho Chehab wrote: > Em Wed, 29 Oct 2014 14:46:55 +0200 Laurent Pinchart escreveu: > > > > Hmm, so you think VIDEO_MAX_FRAME should just be updated to 64? > > > > > > Yes. > > > > > > > I am a bit afraid that that might break applications

Re: [PATCH] vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME

2014-10-29 Thread Mauro Carvalho Chehab
Em Wed, 29 Oct 2014 14:46:55 +0200 Laurent Pinchart escreveu: > > > Hmm, so you think VIDEO_MAX_FRAME should just be updated to 64? > > > > Yes. > > > > > I am a bit afraid that that might break applications (especially if there > > > are any that use bits in a 32-bit unsigned variable). > > >

Re: [PATCH] vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME

2014-10-29 Thread Laurent Pinchart
Hi Mauro, On Wednesday 29 October 2014 10:40:33 Mauro Carvalho Chehab wrote: > Em Wed, 29 Oct 2014 11:01:24 +0100 Hans Verkuil escreveu: > > On 10/29/14 10:13, Mauro Carvalho Chehab wrote: > > > Em Wed, 29 Oct 2014 09:59:56 +0100 Hans Verkuil escreveu: > > >> On 10/29/14 09:29, Mauro Carvalho Cheh

Re: [PATCH] vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME

2014-10-29 Thread Mauro Carvalho Chehab
Em Wed, 29 Oct 2014 11:01:24 +0100 Hans Verkuil escreveu: > On 10/29/14 10:13, Mauro Carvalho Chehab wrote: > > Em Wed, 29 Oct 2014 09:59:56 +0100 > > Hans Verkuil escreveu: > > > >> On 10/29/14 09:29, Mauro Carvalho Chehab wrote: > >>> Em Wed, 29 Oct 2014 08:29:08 +0100 > >>> Hans Verkuil esc

Re: [PATCH] vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME

2014-10-29 Thread Hans Verkuil
On 10/29/14 10:13, Mauro Carvalho Chehab wrote: > Em Wed, 29 Oct 2014 09:59:56 +0100 > Hans Verkuil escreveu: > >> On 10/29/14 09:29, Mauro Carvalho Chehab wrote: >>> Em Wed, 29 Oct 2014 08:29:08 +0100 >>> Hans Verkuil escreveu: >>> On 10/28/2014 07:26 PM, Mauro Carvalho Chehab wrote: >

Re: [PATCH] vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME

2014-10-29 Thread Mauro Carvalho Chehab
Em Wed, 29 Oct 2014 09:59:56 +0100 Hans Verkuil escreveu: > On 10/29/14 09:29, Mauro Carvalho Chehab wrote: > > Em Wed, 29 Oct 2014 08:29:08 +0100 > > Hans Verkuil escreveu: > > > >> On 10/28/2014 07:26 PM, Mauro Carvalho Chehab wrote: > >>> Em Fri, 10 Oct 2014 10:04:58 +0200 > >>> Hans Verkuil

Re: [PATCH] vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME

2014-10-29 Thread Hans Verkuil
On 10/29/14 09:29, Mauro Carvalho Chehab wrote: > Em Wed, 29 Oct 2014 08:29:08 +0100 > Hans Verkuil escreveu: > >> On 10/28/2014 07:26 PM, Mauro Carvalho Chehab wrote: >>> Em Fri, 10 Oct 2014 10:04:58 +0200 >>> Hans Verkuil escreveu: >>> (This patch is from Divneil except for the vivid chan

Re: [PATCH] vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME

2014-10-29 Thread Mauro Carvalho Chehab
Em Wed, 29 Oct 2014 08:29:08 +0100 Hans Verkuil escreveu: > On 10/28/2014 07:26 PM, Mauro Carvalho Chehab wrote: > > Em Fri, 10 Oct 2014 10:04:58 +0200 > > Hans Verkuil escreveu: > > > >> (This patch is from Divneil except for the vivid changes which I added. He > >> had > >> difficulties post

Re: [PATCH] vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME

2014-10-29 Thread Hans Verkuil
On 10/28/2014 07:26 PM, Mauro Carvalho Chehab wrote: > Em Fri, 10 Oct 2014 10:04:58 +0200 > Hans Verkuil escreveu: > >> (This patch is from Divneil except for the vivid changes which I added. He >> had >> difficulties posting the patch without the mailer mangling it, so I'm >> reposting >> it f

Re: [PATCH] vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME

2014-10-28 Thread Mauro Carvalho Chehab
Em Fri, 10 Oct 2014 10:04:58 +0200 Hans Verkuil escreveu: > (This patch is from Divneil except for the vivid changes which I added. He had > difficulties posting the patch without the mailer mangling it, so I'm > reposting > it for him) > > - vb2 drivers to rely on VB2_MAX_FRAME. > > - VB2_MAX

Re: [PATCH] vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME

2014-10-11 Thread Hans Verkuil
Hi Laurent, On 10/11/2014 07:21 PM, Laurent Pinchart wrote: Hi Hans, On Friday 10 October 2014 10:04:58 Hans Verkuil wrote: (This patch is from Divneil except for the vivid changes which I added. He had difficulties posting the patch without the mailer mangling it, so I'm reposting it for him)

Re: [PATCH] vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME

2014-10-11 Thread Laurent Pinchart
Hi Hans, On Friday 10 October 2014 10:04:58 Hans Verkuil wrote: > (This patch is from Divneil except for the vivid changes which I added. He > had difficulties posting the patch without the mailer mangling it, so I'm > reposting it for him) > > - vb2 drivers to rely on VB2_MAX_FRAME. > > - VB2_M

[PATCH] vb2: replace VIDEO_MAX_FRAME with VB2_MAX_FRAME

2014-10-10 Thread Hans Verkuil
(This patch is from Divneil except for the vivid changes which I added. He had difficulties posting the patch without the mailer mangling it, so I'm reposting it for him) - vb2 drivers to rely on VB2_MAX_FRAME. - VB2_MAX_FRAME bumps the value to 64 from current 32 Signed-off-by: Divneil Wadhawa