On Mon, Jan 13, 2020 at 10:27 PM Gerd Hoffmann <kraxel@xxxxxxxxxx> wrote: > > Hi, > > > > Well, no. Tomasz Figa had splitted the devices into three groups: > > > > > > (1) requires single buffer. > > > (2) allows any layout (including the one (1) devices want). > > > (3) requires per-plane buffers. > > > > > > Category (3) devices are apparently rare and old. Both category (1)+(2) > > > devices can handle single buffers just fine. So maybe support only > > > that? > > > > From the guest implementation point of view, Linux V4L2 currently > > supports 2 cases, if used in allocation-mode (i.e. the buffers are > > allocated locally by V4L2): > > > > i) single buffer with plane offsets predetermined by the format, (can > > be handled by devices from category 1) and 2)) > > ii) per-plane buffers with planes at the beginning of their own > > buffers. (can be handled by devices from category 2) and 3)) > > > > Support for ii) is required if one wants to be able to import buffers > > with arbitrary plane offsets, so I'd consider it unavoidable. > > If you have (1) hardware you simply can't import buffers with arbitrary > plane offsets, so I'd expect software would prefer the single buffer > layout (i) over (ii), even when using another driver + dmabuf > export/import, to be able to support as much hardware as possible. > So (ii) might end up being unused in practice. > > But maybe not, was just an idea, feel free to scratch it. That's true, simple user space would often do that. However, if more devices are in the game, often some extra alignment or padding between planes is needed and that is not allowed by (1), even though all the planes are in the same buffer. My suggestion, based on the latest V4L2 discussion on unifying the UAPI of i) and ii), is that we may want to instead always specify buffers on a per-plane basis. Any additional requirements would be then validated by the host, which could check if the planes end up in the same buffer (or different buffers for (3)) and/or at the right offsets. WDYT? Best regards, Tomasz
