Re: [PATCH 0/3] Add WCL as subplatform from PTL

[Gustavo Sousa <gustavo.sousa@intel.com>]

Quoting Gustavo Sousa (2025-09-15 10:40:31-03:00)
>Quoting Dnyaneshwar Bhadane (2025-09-11 17:55:39-03:00)
>>- Add WCL as subplatform and update the definition struct. 
>>- Update condition required to distinguish WCL C10 PHY selection
>>on port B.
>
>I have added comments in individual patches. More general comments
>below:
>
>- The title of the series is a bit innacurate as it just refers to the
>  prelimiary work for the final objective of the series. I would simply
>  re-use the original patch's subject because (i) it would help people
>  see this series as a next version (considering that you used v2) of
>  the originally single-patch series; and (ii) it is more accurate
>  regaring the final objective of this set of changes.
>
>- This series should also be sent to intel-xe@lists.freedesktop.org,
>  since we are touching stuff related to hardware enabled in the xe
>  driver.

Huh... I just realized now that this was also sent[1] to
intel-xe@lists.freedesktop.org, but as a different mail thread and has
already gotten some feedback from Jani. Usually, we sent a single series
(same thread) to both recipients (intel-gfx and intel-xe). Sending them
as separate threads is not necesary and make reviews "fragmented".

[1] https://lore.kernel.org/intel-xe/20250911210514.389014-1-dnyaneshwar.bhadane@intel.com/

>
>--
>Gustavo Sousa
>
>>
>>Dnyaneshwar Bhadane (3):
>>  drm/pcids: Split PTL pciids group to make wcl subplatform
>>  drm/i915/display: Add definition for wcl as subplatform
>>  drm/i915/xe3: Restrict PTL intel_encoder_is_c10phy() to only PHY A
>>
>> drivers/gpu/drm/i915/display/intel_cx0_phy.c        | 11 ++++-------
>> drivers/gpu/drm/i915/display/intel_display_device.c | 13 +++++++++++++
>> drivers/gpu/drm/i915/display/intel_display_device.h |  4 +++-
>> drivers/gpu/drm/xe/xe_pci.c                         |  1 +
>> include/drm/intel/pciids.h                          |  4 +++-
>> 5 files changed, 24 insertions(+), 9 deletions(-)
>>
>>-- 
>>2.51.0
>>