* [PATCH] media: davinci: vpbe: align the buffers size to page page size boundary
@ 2013-04-16 11:10 Prabhakar lad
2013-04-16 11:40 ` Sergei Shtylyov
0 siblings, 1 reply; 2+ messages in thread
From: Prabhakar lad @ 2013-04-16 11:10 UTC (permalink / raw)
To: LMML
Cc: Mauro Carvalho Chehab, DLOS, LKML, Lad, Prabhakar,
Laurent Pinchart, Hans Verkuil
From: Lad, Prabhakar <prabhakar.csengg@gmail.com>
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 page size boundary for both
capture and display driver so the it pass the check.
Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Hans Verkuil <hans.verkuil@cisco.com>
Cc: Mauro Carvalho Chehab <mchehab@redhat.com>
---
drivers/media/platform/davinci/vpbe_display.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/media/platform/davinci/vpbe_display.c b/drivers/media/platform/davinci/vpbe_display.c
index 1802f11..ff8fe76 100644
--- a/drivers/media/platform/davinci/vpbe_display.c
+++ b/drivers/media/platform/davinci/vpbe_display.c
@@ -258,7 +258,7 @@ vpbe_buffer_queue_setup(struct vb2_queue *vq, const struct v4l2_format *fmt,
*nbuffers = layer->numbuffers = VPBE_DEFAULT_NUM_BUFS;
*nplanes = 1;
- sizes[0] = layer->pix_fmt.sizeimage;
+ sizes[0] = PAGE_ALIGN(layer->pix_fmt.sizeimage);
alloc_ctxs[0] = layer->alloc_ctx;
return 0;
--
1.7.4.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] media: davinci: vpbe: align the buffers size to page page size boundary
2013-04-16 11:10 [PATCH] media: davinci: vpbe: align the buffers size to page page size boundary Prabhakar lad
@ 2013-04-16 11:40 ` Sergei Shtylyov
0 siblings, 0 replies; 2+ messages in thread
From: Sergei Shtylyov @ 2013-04-16 11:40 UTC (permalink / raw)
To: Prabhakar lad
Cc: LMML, DLOS, Mauro Carvalho Chehab, LKML, Hans Verkuil,
Laurent Pinchart
On 16-04-2013 15:10, Prabhakar lad wrote:
> From: Lad, Prabhakar <prabhakar.csengg@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 mmap, the application were failing to
> mmap the buffers.
> This patch aligns the the buffer size to page size boundary for both
> capture and display driver so the it pass the check.
> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: Hans Verkuil <hans.verkuil@cisco.com>
> Cc: Mauro Carvalho Chehab <mchehab@redhat.com>
WBR, Sergei
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-04-16 11:41 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-16 11:10 [PATCH] media: davinci: vpbe: align the buffers size to page page size boundary Prabhakar lad
2013-04-16 11:40 ` Sergei Shtylyov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).