[Intel-gfx] [PATCH v5 09/10] drm/i915/guc: Follow legacy register names

Teres Alexis, Alan Previn alan.previn.teres.alexis at intel.com
Fri Feb 4 18:53:26 UTC 2022


Squash will be much easier if i also squash all the prior register table additions together
Should i do that? Squash XeLP + DG2 + Gen9 and this together?

On Thu, 2022-02-03 at 11:09 -0800, Matthew Brost wrote:
> On Wed, Jan 26, 2022 at 02:48:21AM -0800, Alan Previn wrote:
> > Before we print the GuC provided register dumps, modify the
> > register tables to use string names as per the legacy error
> > capture execlist codes.
> > 
> > Signed-off-by: Alan Previn <alan.previn.teres.alexis at intel.com>
> 
> I'd just squash this to the patches early in the series where these are
> initially defined.
> 
> Matt 
> 
> > ---
> >  .../gpu/drm/i915/gt/uc/intel_guc_capture.c    | 70 +++++++++----------
> >  1 file changed, 35 insertions(+), 35 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c
> > index 2f5dc413dddc..506496058daf 100644
> > --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c
> > +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c
> > @@ -22,7 +22,7 @@
> >   *       from the engine-mmio-base
> >   */
> >  #define COMMON_BASE_GLOBAL() \
> > -	{FORCEWAKE_MT,             0,      0, "FORCEWAKE_MT"}
> > +	{FORCEWAKE_MT,             0,      0, "FORCEWAKE"}
> >  
> >  #define COMMON_GEN9BASE_GLOBAL() \
> >  	{GEN8_FAULT_TLB_DATA0,     0,      0, "GEN8_FAULT_TLB_DATA0"}, \
> > 


More information about the Intel-gfx mailing list