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 [email protected],
  since we are touching stuff related to hardware enabled in the xe
  driver.

--
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
>

Reply via email to