Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-18 Thread Laurent Pinchart
On Thursday 18 April 2013 11:08:28 Mauro Carvalho Chehab wrote: > Em Thu, 18 Apr 2013 15:22:16 +0200 Laurent Pinchart escreveu: > > On Thursday 18 April 2013 08:35:47 Mauro Carvalho Chehab wrote: > > > Em Thu, 18 Apr 2013 08:21:21 -0300 Mauro Carvalho Chehab escreveu: > > > > Em Thu, 18 Apr 2013

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-18 Thread Mauro Carvalho Chehab
Em Thu, 18 Apr 2013 15:22:16 +0200 Laurent Pinchart escreveu: > Hi Mauro, > > On Thursday 18 April 2013 08:35:47 Mauro Carvalho Chehab wrote: > > Em Thu, 18 Apr 2013 08:21:21 -0300 Mauro Carvalho Chehab escreveu: > > > Em Thu, 18 Apr 2013 10:17:14 +0530 Prabhakar Lad escreveu: > > > > On Tue,

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-18 Thread Laurent Pinchart
Hi Mauro, On Thursday 18 April 2013 08:35:47 Mauro Carvalho Chehab wrote: > Em Thu, 18 Apr 2013 08:21:21 -0300 Mauro Carvalho Chehab escreveu: > > Em Thu, 18 Apr 2013 10:17:14 +0530 Prabhakar Lad escreveu: > > > On Tue, Apr 16, 2013 at 4:48 PM, Laurent Pinchart wrote: > > > > Hi Prabhakar, > > >

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-18 Thread Mauro Carvalho Chehab
Em Thu, 18 Apr 2013 08:21:21 -0300 Mauro Carvalho Chehab escreveu: > Em Thu, 18 Apr 2013 10:17:14 +0530 > Prabhakar Lad escreveu: > > > Hi Marek, > > > > On Tue, Apr 16, 2013 at 4:48 PM, Laurent Pinchart > > wrote: > > > Hi Prabhakar, > > ... > > > >> *nbuffers =

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-18 Thread Mauro Carvalho Chehab
Em Thu, 18 Apr 2013 10:17:14 +0530 Prabhakar Lad escreveu: > Hi Marek, > > On Tue, Apr 16, 2013 at 4:48 PM, Laurent Pinchart > wrote: > > Hi Prabhakar, ... > >> *nbuffers = config_params.min_numbuffers; > >> > >> *nplanes = 1; > >> + size = PAGE_ALIGN(size); > > > > I wonder if

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-18 Thread Mauro Carvalho Chehab
Em Thu, 18 Apr 2013 10:17:14 +0530 Prabhakar Lad prabhakar.cse...@gmail.com escreveu: Hi Marek, On Tue, Apr 16, 2013 at 4:48 PM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi Prabhakar, ... *nbuffers = config_params.min_numbuffers; *nplanes = 1; + size =

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-18 Thread Mauro Carvalho Chehab
Em Thu, 18 Apr 2013 08:21:21 -0300 Mauro Carvalho Chehab mche...@redhat.com escreveu: Em Thu, 18 Apr 2013 10:17:14 +0530 Prabhakar Lad prabhakar.cse...@gmail.com escreveu: Hi Marek, On Tue, Apr 16, 2013 at 4:48 PM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-18 Thread Laurent Pinchart
Hi Mauro, On Thursday 18 April 2013 08:35:47 Mauro Carvalho Chehab wrote: Em Thu, 18 Apr 2013 08:21:21 -0300 Mauro Carvalho Chehab escreveu: Em Thu, 18 Apr 2013 10:17:14 +0530 Prabhakar Lad escreveu: On Tue, Apr 16, 2013 at 4:48 PM, Laurent Pinchart wrote: Hi Prabhakar, ...

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-18 Thread Mauro Carvalho Chehab
Em Thu, 18 Apr 2013 15:22:16 +0200 Laurent Pinchart laurent.pinch...@ideasonboard.com escreveu: Hi Mauro, On Thursday 18 April 2013 08:35:47 Mauro Carvalho Chehab wrote: Em Thu, 18 Apr 2013 08:21:21 -0300 Mauro Carvalho Chehab escreveu: Em Thu, 18 Apr 2013 10:17:14 +0530 Prabhakar Lad

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-18 Thread Laurent Pinchart
On Thursday 18 April 2013 11:08:28 Mauro Carvalho Chehab wrote: Em Thu, 18 Apr 2013 15:22:16 +0200 Laurent Pinchart escreveu: On Thursday 18 April 2013 08:35:47 Mauro Carvalho Chehab wrote: Em Thu, 18 Apr 2013 08:21:21 -0300 Mauro Carvalho Chehab escreveu: Em Thu, 18 Apr 2013 10:17:14

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-17 Thread Prabhakar Lad
Hi Marek, On Tue, Apr 16, 2013 at 4:48 PM, Laurent Pinchart wrote: > Hi Prabhakar, > > (CC'ing Marek) > > On Tuesday 16 April 2013 16:24:30 Prabhakar lad wrote: >> From: Lad, Prabhakar >> >> with recent commit with id 068a0df76023926af958a336a78bef60468d2033 >> which adds add length check for

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-17 Thread Prabhakar Lad
Hi Marek, On Tue, Apr 16, 2013 at 4:48 PM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi Prabhakar, (CC'ing Marek) On Tuesday 16 April 2013 16:24:30 Prabhakar lad wrote: From: Lad, Prabhakar prabhakar.cse...@gmail.com with recent commit with id

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-16 Thread Sergei Shtylyov
Hello. On 16-04-2013 14:54, Prabhakar lad wrote: From: Lad, Prabhakar with recent commit with id 068a0df76023926af958a336a78bef60468d2033 Please also specify the summary line of that commit in parens (or however you like). which adds add length check for mmap, the application were

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-16 Thread Laurent Pinchart
Hi Prabhakar, (CC'ing Marek) On Tuesday 16 April 2013 16:24:30 Prabhakar lad wrote: > From: Lad, Prabhakar > > with recent commit with id 068a0df76023926af958a336a78bef60468d2033 > which adds add length check for mmap, the application were failing to > mmap the buffers. > > This patch aligns

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-16 Thread Prabhakar Lad
Hi Hans, On Tue, Apr 16, 2013 at 4:24 PM, Prabhakar lad wrote: > From: Lad, Prabhakar > > with recent commit with id 068a0df76023926af958a336a78bef60468d2033 > which adds add length check for mmap, the application were failing to > mmap the buffers. > > This patch aligns the the buffer size to

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-16 Thread Prabhakar Lad
Hi Hans, On Tue, Apr 16, 2013 at 4:24 PM, Prabhakar lad prabhakar.cse...@gmail.com wrote: From: Lad, Prabhakar prabhakar.cse...@gmail.com with recent commit with id 068a0df76023926af958a336a78bef60468d2033 which adds add length check for mmap, the application were failing to mmap the

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-16 Thread Laurent Pinchart
Hi Prabhakar, (CC'ing Marek) On Tuesday 16 April 2013 16:24:30 Prabhakar lad wrote: From: Lad, Prabhakar prabhakar.cse...@gmail.com with recent commit with id 068a0df76023926af958a336a78bef60468d2033 which adds add length check for mmap, the application were failing to mmap the buffers.

Re: [PATCH v2] media: davinci: vpif: align the buffers size to page page size boundary

2013-04-16 Thread Sergei Shtylyov
Hello. On 16-04-2013 14:54, Prabhakar lad wrote: From: Lad, Prabhakar prabhakar.cse...@gmail.com with recent commit with id 068a0df76023926af958a336a78bef60468d2033 Please also specify the summary line of that commit in parens (or however you like). which adds add length check for