[Intel-gfx] [PATCH 5/9] drm/i915: Use a define for the default priority [0]

Mika Kuoppala mika.kuoppala at linux.intel.com
Fri May 5 08:31:14 UTC 2017


Chris Wilson <chris at chris-wilson.co.uk> writes:

> On Thu, May 04, 2017 at 04:32:34PM +0300, Joonas Lahtinen wrote:
>> On ke, 2017-05-03 at 12:37 +0100, Chris Wilson wrote:
>> > Explicitly assign the default priority, and give it a name (macro).
>> > 
>> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
>> 
>> <SNIP>
>> 
>> >  	kref_init(&ctx->ref);
>> >  	list_add_tail(&ctx->link, &dev_priv->context_list);
>> >  	ctx->i915 = dev_priv;
>> > +	ctx->priority = I915_PRIORITY_DFL;
>> 
>> I915_PRIORITY_DEFAULT would work better.
>
> On the one hand I have the symmetry with MIN, DFL, MAX, on the other
> hand DFL is plain bizarre.

DEF?

[1]: http://www.abbreviations.com/abbreviation/default

-Mika


More information about the Intel-gfx mailing list