On Fri, 25 Oct 2024, Suraj Kandpal <suraj.kandpal@xxxxxxxxx> wrote:
> Fix the condition for gsc structure validity in
> gsc_cs_status_check(). It needs to be an OR and not an AND
> condition
This is a completely different and new patch. For future reference,
please send them in new threads, not in-reply-to the old ones. In
general, please tweak your approach slightly more towards new threads
over in-reply-to.
Thanks,
Jani.
>
> Fixes: b4224f6bae38 ("drm/xe/hdcp: Check GSC structure validity")
> Signed-off-by: Suraj Kandpal <suraj.kandpal@xxxxxxxxx>
> ---
> 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 6d34b55d72bb..7c02323e9531 100644
> --- a/drivers/gpu/drm/xe/display/xe_hdcp_gsc.c
> +++ b/drivers/gpu/drm/xe/display/xe_hdcp_gsc.c
> @@ -44,7 +44,7 @@ bool intel_hdcp_gsc_check_status(struct intel_display *display)
> bool ret = true;
> unsigned int fw_ref;
>
> - 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;
--
Jani Nikula, Intel
