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

[Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 2234 bytes --]



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<rodrigo.vivi@intel.com>
>
> 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<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
>>

[-- Attachment #2: Type: text/html, Size: 4472 bytes --]