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 [email protected], > 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 [email protected], 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/[email protected]/ > >-- >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 >>
