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