On Fri, Mar 28, 2014 at 10:04:09PM +0100, Daniel Vetter wrote:
On Tue, Mar 18, 2014 at 05:22:57PM -0700, Matt Roper wrote:
Add cursor plane as a parameter to drm_crtc_init() and update all existing DRM drivers to use a helper-provided primary plane. Passing NULL for this parameter indicates that there is no hardware cursor supported by the driver and no cursor plane should be provided to userspace.
Signed-off-by: Matt Roper matthew.d.roper@intel.com
Ok, cursor planes. I've poked around in this a lot and unfortunately I don't think we can achieve nirvana :(
[...]
- For the pixel format I think it's ok to always assume RGBA.
How so? We do have hardware on Tegra that doesn't support RGBA cursors and if at all possible I'd very much like to support that as well. New generations can do RGBA but still support the old pixel format. Having the option of supporting all formats would be nice.
There was some discussion about implementing a bunch of plane properties so I think having one to enumerate a set of pixel formats wouldn't be such a big deal.
Thierry