Commit 58041996 authored by John Harrison's avatar John Harrison

drm/i915/guc: More debug print updates - GSC firmware

Update a bunch more debug prints to use the new GT based scheme.

v2: Also change prints to use %pe for error values (MichalW).
Signed-off-by: default avatarJohn Harrison <John.C.Harrison@Intel.com>
Reviewed-by: default avatarMichal Wajdeczko <michal.wajdeczko@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230207050717.1833718-3-John.C.Harrison@Intel.com
parent 4fd4fde8
...@@ -6,6 +6,7 @@ ...@@ -6,6 +6,7 @@
#include "gt/intel_engine_pm.h" #include "gt/intel_engine_pm.h"
#include "gt/intel_gpu_commands.h" #include "gt/intel_gpu_commands.h"
#include "gt/intel_gt.h" #include "gt/intel_gt.h"
#include "gt/intel_gt_print.h"
#include "gt/intel_ring.h" #include "gt/intel_ring.h"
#include "intel_gsc_fw.h" #include "intel_gsc_fw.h"
...@@ -88,9 +89,8 @@ static int gsc_fw_load(struct intel_gsc_uc *gsc) ...@@ -88,9 +89,8 @@ static int gsc_fw_load(struct intel_gsc_uc *gsc)
i915_request_put(rq); i915_request_put(rq);
if (err) if (err)
drm_err(&gsc_uc_to_gt(gsc)->i915->drm, gt_err(gsc_uc_to_gt(gsc), "Request submission for GSC load failed %pe\n",
"Request submission for GSC load failed (%d)\n", ERR_PTR(err));
err);
return err; return err;
} }
...@@ -200,8 +200,7 @@ int intel_gsc_uc_fw_upload(struct intel_gsc_uc *gsc) ...@@ -200,8 +200,7 @@ int intel_gsc_uc_fw_upload(struct intel_gsc_uc *gsc)
/* FW is not fully operational until we enable SW proxy */ /* FW is not fully operational until we enable SW proxy */
intel_uc_fw_change_status(gsc_fw, INTEL_UC_FIRMWARE_TRANSFERRED); intel_uc_fw_change_status(gsc_fw, INTEL_UC_FIRMWARE_TRANSFERRED);
drm_info(&gt->i915->drm, "Loaded GSC firmware %s\n", gt_info(gt, "Loaded GSC firmware %s\n", gsc_fw->file_selected.path);
gsc_fw->file_selected.path);
return 0; return 0;
......
...@@ -6,6 +6,7 @@ ...@@ -6,6 +6,7 @@
#include <linux/types.h> #include <linux/types.h>
#include "gt/intel_gt.h" #include "gt/intel_gt.h"
#include "gt/intel_gt_print.h"
#include "intel_gsc_uc.h" #include "intel_gsc_uc.h"
#include "intel_gsc_fw.h" #include "intel_gsc_fw.h"
#include "i915_drv.h" #include "i915_drv.h"
...@@ -59,7 +60,6 @@ int intel_gsc_uc_init(struct intel_gsc_uc *gsc) ...@@ -59,7 +60,6 @@ int intel_gsc_uc_init(struct intel_gsc_uc *gsc)
{ {
static struct lock_class_key gsc_lock; static struct lock_class_key gsc_lock;
struct intel_gt *gt = gsc_uc_to_gt(gsc); struct intel_gt *gt = gsc_uc_to_gt(gsc);
struct drm_i915_private *i915 = gt->i915;
struct intel_engine_cs *engine = gt->engine[GSC0]; struct intel_engine_cs *engine = gt->engine[GSC0];
struct intel_context *ce; struct intel_context *ce;
struct i915_vma *vma; struct i915_vma *vma;
...@@ -81,8 +81,7 @@ int intel_gsc_uc_init(struct intel_gsc_uc *gsc) ...@@ -81,8 +81,7 @@ int intel_gsc_uc_init(struct intel_gsc_uc *gsc)
I915_GEM_HWS_GSC_ADDR, I915_GEM_HWS_GSC_ADDR,
&gsc_lock, "gsc_context"); &gsc_lock, "gsc_context");
if (IS_ERR(ce)) { if (IS_ERR(ce)) {
drm_err(&gt->i915->drm, gt_err(gt, "failed to create GSC CS ctx for FW communication\n");
"failed to create GSC CS ctx for FW communication\n");
err = PTR_ERR(ce); err = PTR_ERR(ce);
goto out_vma; goto out_vma;
} }
...@@ -98,7 +97,7 @@ int intel_gsc_uc_init(struct intel_gsc_uc *gsc) ...@@ -98,7 +97,7 @@ int intel_gsc_uc_init(struct intel_gsc_uc *gsc)
out_fw: out_fw:
intel_uc_fw_fini(&gsc->fw); intel_uc_fw_fini(&gsc->fw);
out: out:
i915_probe_error(i915, "failed with %d\n", err); gt_probe_error(gt, "GSC init failed %pe\n", ERR_PTR(err));
return err; return err;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment