[Intel-gfx] [PATCH] drm/i915: Don't use enums for hardware engine id
Chris Wilson
chris at chris-wilson.co.uk
Tue Feb 28 16:52:34 UTC 2017
On Tue, Feb 28, 2017 at 04:43:02PM +0000, Chris Wilson wrote:
> On Tue, Feb 28, 2017 at 02:12:09PM +0000, Michal Wajdeczko wrote:
> > +/* Hardware Engine ID definitions */
> > +#define RCS_HW 0
> > +#define VCS_HW 1
> > +#define BCS_HW 2
> > +#define VECS_HW 3
> > +#define VCS2_HW 4
>
> So don't put them in the header if they may have inconsistent meanings.
Or if you do want to keep them in a header, either i915_reg.h or
intel_engine_reg.h as somewhere out of the way, and clear that they are
not meant for the rest of the bookkeeping in intel_ringbuffer.h.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
More information about the Intel-gfx
mailing list