All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rebecca Schultz Zavin <rebecca@android.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: linaro-mm-sig@lists.linaro.org, "Semwal,
	Sumit" <sumit.semwal@ti.com>,
	remi@remlab.net, pawel@osciak.com, kyungmin.park@samsung.com,
	dri-devel@lists.freedesktop.org, robdclark@gmail.com,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	airlied@redhat.com, linux-media@vger.kernel.org,
	g.liakhovetski@gmx.de, mchehab@redhat.com
Subject: Re: [Linaro-mm-sig] [PATCHv6 00/13] Integration of videobuf2 with dmabuf
Date: Mon, 4 Jun 2012 20:36:37 -0700	[thread overview]
Message-ID: <CALJcvx7zKGxGWkjXVPtrAvwSJeyOZcs33vKwALD_5nODKy3W8A@mail.gmail.com> (raw)
In-Reply-To: <201206042358.07234.hverkuil@xs4all.nl>

It probably is a fixed offset, but all the information describing it
is in userspace...  In my experience, it's always almost one of the
pre-defined formats, but then there's some extra padding, weird
alignment etc.  I'm trying to convert code that used userptr, but
where the planes were offsets into the same buffer, to use dma_buf.
Looks like I need to dig a bit deeper.

Thanks,
Rebecca


On Mon, Jun 4, 2012 at 2:58 PM, Hans Verkuil <hverkuil@xs4all.nl> wrote:
> Hi Rebecca,
>
> On Mon June 4 2012 21:34:23 Rebecca Schultz Zavin wrote:
>> I have a system where the data is planar, but the kernel drivers
>> expect to get one allocation with offsets for the planes.  I can't
>> figure out how to do that with the current dma_buf implementation.  I
>> thought I could pass the same dma_buf several times and use the
>> data_offset field of the v4l2_plane struct but it looks like that's
>> only for output.  Am I missing something?  Is this supported?
>
> v4l2_plane is typically used if the planes are allocated separately.
> If you allocate it in one go, aren't the planes then at well-defined
> offsets from the start? If so, then it is either one of the already
> pre-defined planar formats found here:
>
> http://hverkuil.home.xs4all.nl/spec/media.html#yuv-formats
>
> or you define a pixelformat specific to your own hardware that identifies
> that particular format.
>
> If it is one allocation, but there is no clear calculation based on width
> and height that gives you the start of each plane, then we do not support
> that at the moment. I believe I had a discussion about something similar
> with people from Qualcomm, but that never came to anything.
>
> That would be something to discuss on the linux-media mailinglist.
>
> Regards,
>
>        Hans

  reply	other threads:[~2012-06-05  3:36 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-23 12:10 [PATCHv6 00/13] Integration of videobuf2 with dmabuf Tomasz Stanislawski
2012-05-23 12:10 ` [PATCHv6 01/13] v4l: Add DMABUF as a memory type Tomasz Stanislawski
2012-05-28 21:12   ` Laurent Pinchart
2012-05-28 21:12     ` Laurent Pinchart
2012-05-23 12:10 ` [PATCHv6 02/13] Documentation: media: description of DMABUF importing in V4L2 Tomasz Stanislawski
2012-05-28 21:30   ` Laurent Pinchart
2012-05-23 12:10 ` [PATCHv6 03/13] v4l: vb2: add support for shared buffer (dma_buf) Tomasz Stanislawski
2012-05-23 12:10 ` [PATCHv6 04/13] v4l: vb: remove warnings about MEMORY_DMABUF Tomasz Stanislawski
2012-05-23 12:10 ` [PATCHv6 05/13] v4l: vb2-dma-contig: Shorten vb2_dma_contig prefix to vb2_dc Tomasz Stanislawski
2012-05-23 12:10 ` [PATCHv6 06/13] v4l: vb2-dma-contig: Remove unneeded allocation context structure Tomasz Stanislawski
2012-05-23 12:10 ` [PATCHv6 07/13] v4l: vb2-dma-contig: Reorder functions Tomasz Stanislawski
2012-05-23 12:10 ` [PATCHv6 08/13] v4l: vb2-dma-contig: add support for scatterlist in userptr mode Tomasz Stanislawski
2012-05-23 12:10 ` [PATCHv6 09/13] v4l: vb2: add prepare/finish callbacks to allocators Tomasz Stanislawski
2012-05-23 12:10 ` [PATCHv6 10/13] v4l: vb2-dma-contig: add prepare/finish to dma-contig allocator Tomasz Stanislawski
2012-05-23 12:10 ` [PATCHv6 11/13] v4l: vb2-dma-contig: add support for dma_buf importing Tomasz Stanislawski
2012-05-23 12:10 ` [PATCHv6 12/13] v4l: s5p-tv: mixer: support for dmabuf importing Tomasz Stanislawski
2012-05-23 12:10 ` [PATCHv6 13/13] v4l: s5p-fimc: " Tomasz Stanislawski
2012-05-28 22:25 ` [PATCHv6 00/13] Integration of videobuf2 with dmabuf Laurent Pinchart
2012-05-30 15:26   ` Semwal, Sumit
2012-06-04 19:34     ` [Linaro-mm-sig] " Rebecca Schultz Zavin
2012-06-04 20:28       ` Rob Clark
2012-06-04 20:46         ` Rebecca Schultz Zavin
2012-06-04 20:56           ` Rebecca Schultz Zavin
2012-06-04 21:04           ` Rob Clark
2012-06-04 21:58       ` Hans Verkuil
2012-06-05  3:36         ` Rebecca Schultz Zavin [this message]
2012-06-05  6:37         ` Hans Verkuil
2012-06-06  3:46       ` Laurent Pinchart
2012-06-06  8:17         ` Hans Verkuil
2012-06-07  0:52           ` Laurent Pinchart
2012-06-07  6:43             ` Hans Verkuil
2012-06-07 10:18               ` Sylwester Nawrocki

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CALJcvx7zKGxGWkjXVPtrAvwSJeyOZcs33vKwALD_5nODKy3W8A@mail.gmail.com \
    --to=rebecca@android.com \
    --cc=airlied@redhat.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=g.liakhovetski@gmx.de \
    --cc=hverkuil@xs4all.nl \
    --cc=kyungmin.park@samsung.com \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linaro-mm-sig@lists.linaro.org \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@redhat.com \
    --cc=pawel@osciak.com \
    --cc=remi@remlab.net \
    --cc=robdclark@gmail.com \
    --cc=sumit.semwal@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.