> -----Original Message-----
> From: dri-devel <[email protected]> On Behalf Of
> [email protected]
> Sent: Monday, October 28, 2024 3:13 PM
> To: Jani Nikula <[email protected]>
> Cc: Vivi, Rodrigo <[email protected]>; De Marchi, Lucas
> <[email protected]>; Thomas <[email protected]>;
> Maarten Lankhorst <[email protected]>; Maxime Ripard
> <[email protected]>; Thomas Zimmermann <[email protected]>;
> David Airlie <[email protected]>; Simona Vetter <[email protected]>; intel-
> [email protected]; [email protected]; dri-
> [email protected]; [email protected]; pengfuyuan
> <[email protected]>
> Subject: [PATCH] drm/xe/hdcp: Fix logic errors
>
> From: pengfuyuan <[email protected]>
>
> Here the gsc struct null pointer check should use '||' instead of '&&'.
>
> Fix the following patches:
> drm/xe/hdcp: Check GSC structure validity
>
This fix has already been sent
https://patchwork.freedesktop.org/series/140291/
Regards,
Suraj Kandpal
> Signed-off-by: pengfuyuan <[email protected]>
> ---
> 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
