On 1/8/2026 6:36 AM, Rodrigo Vivi wrote: > 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 > > on both, the optional approach proposed by the first patch and on > the CI approach you took here. I've re-run CI and no regression have popped up both without [1] and with [2] the FW. [1] https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159774v1/index.html? xe 0000:00:02.0: [drm] Tile0: GT1: GSC firmware xe/ptl_gsc_1.bin not found xe 0000:00:02.0: [drm] Tile0: GT1: GSC firmware(s) can be downloaded from ... [2] https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-159774v3/index.html? xe 0000:00:02.0: [drm] Tile0: GT1: Using GSC firmware from xe/ptl_gsc_1.bin version 105.0.2.1301 I've also sent the PR for the FW to linux-firmware: https://gitlab.com/kernel-firmware/linux-firmware/-/merge_requests/858 So I'm going to go ahead and merge this. Daniele > >> Cc: Rodrigo Vivi >> Cc: Julia Filipchuk >> >> 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 >>