Re: [PATCH 0/3] Enable GSC loading and PXP for PTL

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Wed, Jan 07, 2026 at 05:13:41PM -0800, Daniele Ceraolo Spurio wrote:
> GSC loading is required for content protection features (PXP, HDCP).
> 
> Since PTL force_probe was removed before we added the FW, the driver
> needs to keep working if the FW is not found. Therefore, the code has
> been modified to allow the GSC FW to be missing for BMG and newer HW
> (older HW platforms require GSC for power management, so on those we
> don't want to continue if the FW is missing).
> 
> Note that I haven't pushed the FW yet. This is on purpose, as I want to
> get CI with the FW missing to test that case. Once the patches have been
> reviewed and tested without the FW, I'll push the FW to intel-staging
> and re-trigger CI to make sure everything works with the FW as well.


Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

on both, the optional approach proposed by the first patch and on
the CI approach you took here.

> 
> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
> Cc: Julia Filipchuk <julia.filipchuk@intel.com>
> 
> Daniele Ceraolo Spurio (3):
>   drm/xe/gsc: Make GSC FW load optional for newer platforms
>   drm/xe/ptl: Define GSC for PTL
>   drm/xe/ptl: Enable PXP for PTL
> 
>  drivers/gpu/drm/xe/display/xe_hdcp_gsc.c |  2 +-
>  drivers/gpu/drm/xe/xe_gsc.c              | 13 +++++++------
>  drivers/gpu/drm/xe/xe_pci.c              |  1 +
>  drivers/gpu/drm/xe/xe_uc_fw.c            | 11 ++++++++---
>  4 files changed, 17 insertions(+), 10 deletions(-)
> 
> -- 
> 2.43.0
>