[Intel-gfx] [PATCH 2/2] drm/i915: More use of GT specific print helpers
Andi Shyti
andi.shyti at linux.intel.com
Mon Oct 9 19:54:17 UTC 2023
Hi John,
...
> --- a/drivers/gpu/drm/i915/i915_driver.c
> +++ b/drivers/gpu/drm/i915/i915_driver.c
> @@ -71,6 +71,7 @@
> #include "gem/i915_gem_pm.h"
> #include "gt/intel_gt.h"
> #include "gt/intel_gt_pm.h"
> +#include "gt/intel_gt_print.h"
> #include "gt/intel_rc6.h"
>
> #include "pxp/intel_pxp.h"
> @@ -429,7 +430,7 @@ static int i915_pcode_init(struct drm_i915_private *i915)
> for_each_gt(gt, i915, id) {
> ret = intel_pcode_init(gt->uncore);
> if (ret) {
> - drm_err(>->i915->drm, "gt%d: intel_pcode_init failed %d\n", id, ret);
> + gt_err(gt, "intel_pcode_init failed %d\n", ret);
using gt_*() print functions in the upper layers looks a bit
wrong to me. If we need GT printing, the prints need to be done
inside the function called, in this case would be
intel_pcode_init().
Andi
More information about the Intel-gfx
mailing list