[PATCH RFC 2/8] DRM: Armada: Add Armada DRM driver

[daniel@ffwll.ch (Daniel Vetter)]

On Mon, Jun 10, 2013 at 9:59 PM, Rob Clark <robdclark@gmail.com> wrote:
> On Mon, Jun 10, 2013 at 1:06 PM, Russell King - ARM Linux
> <linux@arm.linux.org.uk> wrote:
>> On Mon, Jun 10, 2013 at 11:57:32AM -0400, Rob Clark wrote:
>>> On Sun, Jun 9, 2013 at 3:29 PM, Russell King
>>> <rmk+kernel@arm.linux.org.uk> wrote:
>>> > This patch adds support for the pair of LCD controllers on the Marvell
>>> > Armada 510 SoCs.  This driver supports:
>>> > - multiple contiguous scanout buffers for video and graphics
>>> > - shm backed cacheable buffer objects for X pixmaps for Vivante GPU
>>> >   acceleration
>>> > - dual lcd0 and lcd1 crt operation
>>> > - video overlay on each LCD crt
>>>
>>> Any particular reason for not exposing the overlays as drm_plane's?
>>> That is probably something that should change before merging the
>>> driver.
>>
>> Only through not understanding much of DRM when I started this.
>> Provided DRM planes can do everything that I'm already doing with
>> the overlay support, then yes.  Otherwise, I want to stick with this
>> which is modelled after the i915 overlay interface.

Oh i915 overlays aren't a good reason ;-) I've done those way back
when drm didn't have any plane infrastructure and pretty much as my
very contribution. So totally lacked any clue.

If I can scrap together a bit of time I want to port the legacy
overlay code over to drm planes (and remap the current ioctl interface
to the drm plane interface for backwards compat). But atm that's
crushed under a giant pile of other todo items.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch