Re: [PATCH 1/2] drm/i915/gen8: The WA BB framework is enabled.

[Michel Thierry <michel.thierry@intel.com>]

On 27/02/15 17:32, Michel Thierry wrote:
>
>
> On 25/02/15 17:54, Arun Siluvery wrote:
>> From: Namrta <namrta.salonie@intel.com>
>>
>> This can be used to enable WA BB infrastructure for features like
>> RC6, SSEU and in between context save/restore etc.
>> The patch which would need WA BB will have to declare the wa_bb obj
>> utilizing the function here. Update the WA BB with required commands
>> and update the address of the WA BB at appropriate place.
>>
>> Change-Id: I9cc49ae7426560215e7b6a6d10ba411caeb9321b
>> Signed-off-by: Namrta <namrta.salonie@intel.com>
>> Signed-off-by: Arun Siluvery <arun.siluvery@linux.intel.com>
>> ---
>>   drivers/gpu/drm/i915/intel_lrc.c | 32 ++++++++++++++++++++++++++++++++
>>   1 file changed, 32 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/i915/intel_lrc.c 
>> b/drivers/gpu/drm/i915/intel_lrc.c
>> index 9c851d8..47473e5 100644
>>
> Maybe add an extra line between the 2 paragraphs in the commit message.
>
> Reviewed-by: Michel Thierry <michel.thierry@intel.com>

Also, your next patch moves the function around, it'd be nice to add it 
in the right place since here, so the diffs are clearer.

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx