Re: [Intel-gfx] [PATCH v5 3/7] drm/i915/gt: add gt_is_root() helper

[Andi Shyti <andi.shyti@linux.intel.com>]

Hi Andrzej and Michal,

> > > The "gt_is_root(struct intel_gt *gt)" helper return true if the
> > > gt is the root gt, which means that its id is 0. Return false
> > > otherwise.
> > > 
> > > Suggested-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
> > > Signed-off-by: Andi Shyti <andi.shyti@linux.intel.com>
> > > ---
> > >   drivers/gpu/drm/i915/gt/intel_gt.h | 5 +++++
> > >   1 file changed, 5 insertions(+)
> > > 
> > > diff --git a/drivers/gpu/drm/i915/gt/intel_gt.h b/drivers/gpu/drm/i915/gt/intel_gt.h
> > > index 915d6192079b..f17f51e2d394 100644
> > > --- a/drivers/gpu/drm/i915/gt/intel_gt.h
> > > +++ b/drivers/gpu/drm/i915/gt/intel_gt.h
> > > @@ -19,6 +19,11 @@ struct drm_printer;
> > >   		  ##__VA_ARGS__);					\
> > >   } while (0)
> > > +static inline bool gt_is_root(struct intel_gt *gt)
> > > +{
> > > +	return !gt->info.id;
> > > +}
> > > +
> > we could squash this patch with prev one, where it can be used in:
> > 
> >   intel_gt_tile_cleanup(struct intel_gt *gt)
> >   {
> >   	intel_uncore_cleanup_mmio(gt->uncore);
> > 
> > -	if (gt->info.id) {
> > +	if (!gt_is_root(gt)) {
> >   		kfree(gt->uncore);
> >   		kfree(gt);
> >   	}
> >   }
> 
> It can be used in intel_gt_tile_setup as well, and then you can remove id
> var.

I will move this before, then before patch 2 and use it where you
suggested, as well.

> > 
> > or just use it this way in this patch, with that:
> > 
> > Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
> Accordingly:
> 
> Reviewed-by: Andrzej Hajda <andrzej.hajda@intel.com>

Thank you!

Andi