From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 0D9D1C624DA for ; Wed, 2 Sep 2026 19:59:01 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id B8A0A10E642; Wed, 2 Sep 2026 19:59:00 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="MYuU+fA/"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.10]) by gabe.freedesktop.org (Postfix) with ESMTPS id 1CE0A10E642 for ; Wed, 2 Sep 2026 19:59:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1788379141; x=1819915141; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=NVt0fb2MoetjXz1U2yBTGoizfeBY1zD4cgRkfgStdHE=; b=MYuU+fA/+N4NrSxkWFBWp0yAlhUwPkc3cVHSZLBvx6578pVHMKQY60ut w87V0tkXHSY9e/19vzsn59MRfSUv0qMF82NI8iUs7ND9nuPBIxMdnqlUP JsBHch8w5iKU9NNNyR6miNqM1fnvlHaxOsrOIQZiqpR0k5S1Y39iuiyas uziWyXzTakwy6DB37pKV7DeAZbnbKTtd450sagd3UdkycPj/+7++97wMt a8Y2M2pat07iCeKYlWXPie8HhxuRi/xIM0rda4FfOHYpgSGA5QxuFAFNh 4swZXssvfT26efTKI1bIOGSrxdALTdweYB1lj4fRQZOMKAFYbmty6AADJ A==; X-CSE-ConnectionGUID: 3MTzPEojQm2yDxUVCPQZqQ== X-CSE-MsgGUID: YLY050AKTIugd6efcbaztQ== X-IronPort-AV: E=McAfee;i="6800,10657,11894"; a="106216470" X-IronPort-AV: E=Sophos;i="6.25,258,1779174000"; d="scan'208";a="106216470" Received: from fmviesa007.fm.intel.com ([10.60.135.147]) by orvoesa102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Sep 2026 12:59:00 -0700 X-CSE-ConnectionGUID: cElFDSDhSdKvZR492+yb8Q== X-CSE-MsgGUID: u04QOtweTOC96odi5x2uEw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.25,258,1779174000"; d="scan'208";a="266265700" Received: from osgc-sh-dragon.sh.intel.com ([10.239.81.44]) by fmviesa007-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Sep 2026 12:58:58 -0700 From: Jonathan Cavitt To: intel-xe@lists.freedesktop.org Cc: alex.zuo@intel.com, jonathan.cavitt@intel.com, michal.wajdeczko@intel.com, raag.jadav@intel.com, andi.shyti@linux.intel.com, jani.nikula@intel.com Subject: [PATCH v6 2/3] drm/xe/heci: Use xe print functions in xe_heci_gsc.c Date: Thu, 3 Sep 2026 03:58:50 +0800 Message-ID: <20260902195851.306669-3-jonathan.cavitt@intel.com> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260902195851.306669-1-jonathan.cavitt@intel.com> References: <20260902195851.306669-1-jonathan.cavitt@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: intel-xe@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel Xe graphics driver List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-xe-bounces@lists.freedesktop.org Sender: "Intel-xe" Update xe_heci_gsc.c to use the xe error reporting helper functions in xe_printk.h instead of directly calling the associated drm print functions from drm_print.h. While we're here, we should update the error messages to print the error codes in a more friendly way. v2: - Update the error messages to print the error codes in a more friendly way (Wajdeczko) v3: - Update heci_gsc_add_device() error messages (Wajdeczko) Signed-off-by: Jonathan Cavitt Cc: Michal Wajdeczko --- drivers/gpu/drm/xe/xe_heci_gsc.c | 21 ++++++++++----------- 1 file changed, 10 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/xe/xe_heci_gsc.c b/drivers/gpu/drm/xe/xe_heci_gsc.c index 5af8903e10af2..194849edec116 100644 --- a/drivers/gpu/drm/xe/xe_heci_gsc.c +++ b/drivers/gpu/drm/xe/xe_heci_gsc.c @@ -8,12 +8,11 @@ #include #include -#include - #include "xe_device_types.h" #include "xe_heci_gsc.h" #include "regs/xe_gsc_regs.h" #include "xe_platform_types.h" +#include "xe_printk.h" #include "xe_survivability_mode.h" #define GSC_BAR_LENGTH 0x00000FFC @@ -112,13 +111,13 @@ static int heci_gsc_irq_setup(struct xe_device *xe) heci_gsc->irq = irq_alloc_desc(0); if (heci_gsc->irq < 0) { - drm_err(&xe->drm, "gsc irq error %d\n", heci_gsc->irq); + xe_err(xe, "GSC: irq allocation failed (%pe)\n", ERR_PTR(heci_gsc->irq)); return heci_gsc->irq; } ret = heci_gsc_irq_init(heci_gsc->irq); if (ret < 0) - drm_err(&xe->drm, "gsc irq init failed %d\n", ret); + xe_err(xe, "GSC: irq initialization failed (%pe)\n", ERR_PTR(ret)); return ret; } @@ -151,7 +150,7 @@ static int heci_gsc_add_device(struct xe_device *xe, const struct heci_gsc_def * ret = auxiliary_device_init(aux_dev); if (ret < 0) { - drm_err(&xe->drm, "gsc aux init failed %d\n", ret); + xe_err(xe, "GSC: aux device init failed (%pe)\n", ERR_PTR(ret)); kfree(adev); return ret; } @@ -159,7 +158,7 @@ static int heci_gsc_add_device(struct xe_device *xe, const struct heci_gsc_def * heci_gsc->adev = adev; /* needed by the notifier */ ret = auxiliary_device_add(aux_dev); if (ret < 0) { - drm_err(&xe->drm, "gsc aux add failed %d\n", ret); + xe_err(xe, "GSC: aux device add failed (%pe)\n", ERR_PTR(ret)); heci_gsc->adev = NULL; /* adev will be freed with the put_device() and .release sequence */ @@ -190,7 +189,7 @@ int xe_heci_gsc_init(struct xe_device *xe) } if (!def || !def->name) { - drm_warn(&xe->drm, "HECI is not implemented!\n"); + xe_warn(xe, "HECI is not implemented!\n"); return 0; } @@ -215,7 +214,7 @@ void xe_heci_gsc_irq_handler(struct xe_device *xe, u32 iir) return; if (!xe->info.has_heci_gscfi) { - drm_warn_once(&xe->drm, "GSC irq: not supported"); + xe_warn_once(xe, "GSC: unexpected irq %#x\n", iir); return; } @@ -224,7 +223,7 @@ void xe_heci_gsc_irq_handler(struct xe_device *xe, u32 iir) ret = generic_handle_irq_safe(xe->heci_gsc.irq); if (ret) - drm_err_ratelimited(&xe->drm, "error handling GSC irq: %d\n", ret); + xe_err_ratelimited(xe, "GSC: irq handling failed (%pe)\n", ERR_PTR(ret)); } void xe_heci_csc_irq_handler(struct xe_device *xe, u32 iir) @@ -235,7 +234,7 @@ void xe_heci_csc_irq_handler(struct xe_device *xe, u32 iir) return; if (!xe->info.has_heci_cscfi) { - drm_warn_once(&xe->drm, "CSC irq: not supported"); + xe_warn_once(xe, "CSC: unexpected irq %#x\n", iir); return; } @@ -244,5 +243,5 @@ void xe_heci_csc_irq_handler(struct xe_device *xe, u32 iir) ret = generic_handle_irq_safe(xe->heci_gsc.irq); if (ret) - drm_err_ratelimited(&xe->drm, "error handling GSC irq: %d\n", ret); + xe_err_ratelimited(xe, "CSC: irq handling failed (%pe)\n", ERR_PTR(ret)); } -- 2.53.0