diff options
| author | Lucas De Marchi <lucas.demarchi@intel.com> | 2023-06-11 15:24:45 -0700 |
|---|---|---|
| committer | Rodrigo Vivi <rodrigo.vivi@intel.com> | 2023-12-21 11:34:14 -0500 |
| commit | 6dc3a12fb8185f98b525dbdb02fa5b810c4ff0bc (patch) | |
| tree | 569deb9caa80a5cfb581f3f661b817e100b9add6 /drivers/gpu/drm/xe | |
| parent | 90738d86650729cafb6d92191e6568d4b425b20a (diff) | |
drm/xe/guc: Normalize error messages with %#x
One of the messages was printed without 0x prefix, so it was not clear
if it was decimal or hex: make sure to add the prefix by using %#x.
While at it, normalize the other messages in the same function to follow
the same pattern.
Reviewed-by: Gustavo Sousa <gustavo.sousa@intel.com>
Link: https://lore.kernel.org/r/20230611222447.2837573-3-lucas.demarchi@intel.com
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Diffstat (limited to 'drivers/gpu/drm/xe')
| -rw-r--r-- | drivers/gpu/drm/xe/xe_guc.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/gpu/drm/xe/xe_guc.c b/drivers/gpu/drm/xe/xe_guc.c index e51d8fb4a354..43f862aaacbe 100644 --- a/drivers/gpu/drm/xe/xe_guc.c +++ b/drivers/gpu/drm/xe/xe_guc.c @@ -647,7 +647,7 @@ retry: GUC_HXG_MSG_0_ORIGIN, 50000, &reply, false); if (ret) { timeout: - drm_err(&xe->drm, "mmio request 0x%08x: no reply 0x%08x\n", + drm_err(&xe->drm, "mmio request %#x: no reply %#x\n", request[0], reply); return ret; } @@ -673,7 +673,7 @@ timeout: GUC_HXG_TYPE_NO_RESPONSE_RETRY) { u32 reason = FIELD_GET(GUC_HXG_RETRY_MSG_0_REASON, header); - drm_dbg(&xe->drm, "mmio request %#x: retrying, reason %u\n", + drm_dbg(&xe->drm, "mmio request %#x: retrying, reason %#x\n", request[0], reason); goto retry; } @@ -683,7 +683,7 @@ timeout: u32 hint = FIELD_GET(GUC_HXG_FAILURE_MSG_0_HINT, header); u32 error = FIELD_GET(GUC_HXG_FAILURE_MSG_0_ERROR, header); - drm_err(&xe->drm, "mmio request %#x: failure %x/%u\n", + drm_err(&xe->drm, "mmio request %#x: failure %#x/%#x\n", request[0], error, hint); return -ENXIO; } |