[Intel-gfx] [PATCH 9/9] drm/i915: Don't populate plane->plane for cursors and sprites
Ville Syrjälä
ville.syrjala at linux.intel.com
Tue Nov 8 15:40:47 UTC 2016
On Tue, Nov 08, 2016 at 03:30:33PM +0000, Chris Wilson wrote:
> On Tue, Nov 08, 2016 at 04:47:19PM +0200, ville.syrjala at linux.intel.com wrote:
> > From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> >
> > With plane->plane now purely reserved for the primary planes, let's
> > not even populate it for cursors and sprites. Let's switch the type
> > to enum plane as well since it's no longer being abused for anything
> > else.
>
> Also suggests not starting enum plane from zero.
That would complicate the DSPCNTR() etc. macros a bit. Also I think for
gen2/3 I'll be wanting to index watermark related structs and whatnot
with enum plane rather than enum plane_id since the planes<->pipe
mapping isn't fixed. So a non-zero base might make life a bit more
difficult.
--
Ville Syrjälä
Intel OTC
More information about the Intel-gfx
mailing list