From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [RESEND PATCH v3 05/11] drm: add Atmel HLCDC Display Controller support Date: Wed, 9 Jul 2014 16:02:51 +0200 Message-ID: <20140709140251.GU17271@phenom.ffwll.local> References: <1404751384-5077-1-git-send-email-boris.brezillon@free-electrons.com> <1404751384-5077-6-git-send-email-boris.brezillon@free-electrons.com> <20140708092348.4ecdbb8f@bbrezillon> <20140708163731.6693a883@bbrezillon> <20140708190820.2a94723f@bbrezillon> <20140708235124.GM31728@intel.com> <20140709091424.167b8f16@bbrezillon> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20140709091424.167b8f16@bbrezillon> Sender: linux-pwm-owner@vger.kernel.org To: Boris BREZILLON Cc: Matt Roper , Mark Rutland , "devicetree@vger.kernel.org" , Jean-Jacques Hiblot , Samuel Ortiz , Pawel Moll , linux-pwm@vger.kernel.org, Jean-Christophe Plagniol-Villard , Ian Campbell , Nicolas Ferre , "dri-devel@lists.freedesktop.org" , Tim Niemeyer , Rob Herring , Alexandre Belloni , Laurent Pinchart , Bo Shen , Kumar Gala , Thomas Petazzoni , Lee Jones , Andrew Victor List-Id: devicetree@vger.kernel.org On Wed, Jul 09, 2014 at 09:14:24AM +0200, Boris BREZILLON wrote: > Hi Matt, > > On Tue, 8 Jul 2014 16:51:24 -0700 > Matt Roper wrote: > > > Hi Boris. > > > > I haven't really looked at any of your driver in depth, but from a quick > > glance it looks like you're registering a cursor drm_plane (i.e., making > > use of the new universal plane infrastructure), but you're also > > providing an implementation of the legacy cursor ioctls (cursor_set and > > cursor_move). There's some patches working their way through the > > pipeline that should make this unnecessary and hopefully simplify your > > life a bit: > > > > http://cgit.freedesktop.org/drm-intel/commit/?h=drm-intel-nightly&id=c394c2b08e247c32ef292b75fd8b34312465f8ae > > http://cgit.freedesktop.org/drm-intel/commit/?h=drm-intel-nightly&id=b36552b32aa9c69e83a3a20bda56379fb9e52435 > > http://cgit.freedesktop.org/drm-intel/commit/?h=drm-intel-nightly&id=161d0dc1dccb17ff7a38f462c7c0d4ef8bcc5662 > > http://cgit.freedesktop.org/drm-intel/commit/?h=drm-intel-nightly&id=fc1d3e44ef7c1db93384150fdbf8948dcf949f15 > > > > The third patch there is the one that's really important for your work. > > When a driver provides a cursor plane via the universal plane interface, > > cursor_set and cursor_move are automatically implemented for you by > > drm_mode_cursor_universal() in drivers/gpu/drm/drm_crtc.c and your > > legacy handlers will never get called. drm_mode_cursor_universal() will > > take care of wrapping the bo's into a drm_framebuffer for you. > > > > When I added the universal cursor stuff, I wanted to make sure that as > > soon as a driver starts supporting universal planes it can stop > > supporting the legacy ioctls directly; otherwise handling refcounting > > when userspace switches back and forth between calling legacy ioctl's > > and calling setplane() on a cursor plane would be a nightmare. > > > > I think those patches are only available in drm-intel-nightly at the > > moment and haven't moved on to drm-next and such yet, since i915 is the > > only driver that currently has patches to make use of cursors via the > > univeral plane interface (probably landing for kernel 3.17). > > That's great news. I knew there were some work in progress on this > topic, but didn't know it was planned for 3.17. > > I'll move to this solution. As of today those patches have landed in Dave's drm-next branch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch