[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <tencent_46D189AD021D29866D1A9806B47AB013700A@qq.com>
Date: Mon, 28 Oct 2024 17:43:14 +0800
From: 1064094935@...com
To: Jani Nikula <jani.nikula@...ux.intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@...el.com>,
Lucas De Marchi <lucas.demarchi@...el.com>,
Thomas <thomas.hellstrom@...ux.intel.com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>,
Thomas Zimmermann <tzimmermann@...e.de>,
David Airlie <airlied@...il.com>,
Simona Vetter <simona@...ll.ch>,
intel-gfx@...ts.freedesktop.org,
intel-xe@...ts.freedesktop.org,
dri-devel@...ts.freedesktop.org,
linux-kernel@...r.kernel.org,
pengfuyuan <pengfuyuan@...inos.cn>
Subject: [PATCH] drm/xe/hdcp: Fix logic errors
From: pengfuyuan <pengfuyuan@...inos.cn>
Here the gsc struct null pointer check should use '||' instead of '&&'.
Fix the following patches:
drm/xe/hdcp: Check GSC structure validity
Signed-off-by: pengfuyuan <pengfuyuan@...inos.cn>
---
drivers/gpu/drm/xe/display/xe_hdcp_gsc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/xe/display/xe_hdcp_gsc.c b/drivers/gpu/drm/xe/display/xe_hdcp_gsc.c
index 6619a40aed15..f4332f06b6c8 100644
--- a/drivers/gpu/drm/xe/display/xe_hdcp_gsc.c
+++ b/drivers/gpu/drm/xe/display/xe_hdcp_gsc.c
@@ -42,7 +42,7 @@ bool intel_hdcp_gsc_check_status(struct xe_device *xe)
struct xe_gsc *gsc = >->uc.gsc;
bool ret = true;
- if (!gsc && !xe_uc_fw_is_enabled(&gsc->fw)) {
+ if (!gsc || !xe_uc_fw_is_enabled(&gsc->fw)) {
drm_dbg_kms(&xe->drm,
"GSC Components not ready for HDCP2.x\n");
return false;
--
2.25.1
Powered by blists - more mailing lists