ACK/cmnt: [SRU][F][PATCH 0/9] Fix TC port in legacy/static mode can't be detected due TCCOLD

Kamal Mostafa kamal at canonical.com
Thu Apr 23 18:23:14 UTC 2020


Comment: All the backports from drm-tip should be marked "UBUNTU: SAUCE:"
if they land in Ubuntu before landing in mainline.  (That can be handled
by the applier; no need to resubmit the patch set).
 
Acked-by: Kamal Mostafa <kamal at canonical.com>

 -Kamal

On Wed, Apr 22, 2020 at 02:55:01PM +0800, You-Sheng Yang wrote:
> BugLink: https://bugs.launchpad.net/bugs/1868936
> 
> [Impact]
> External monitor connected via a Type-C dongle when device is in PC10 may not be
> light up.
> 
> [Fix]
> Patches series https://patchwork.freedesktop.org/series/75941/ from
> intel-gfx mailing list, which has been merged in drm-tip tree with tip
> commit 0f8925090ac7. An additional commit 8a84bacba19c landed in v5.5-rc1 is
> also needed to forfill dependencies.
> 
> [Test Case]
> 1. Ensure device may enter PC10.
> 
>    $ sudo powertop --auto-tune
>    $ sudo watch cat /sys/kernel/debug/pmc_core/package_cstate_show
> 
>    Press Meta+L to lock screen for a while and unlock again, "Package
>    C10" field should have non-zero counts.
> 
> 2. Press Meta+L to lock screen again.
> 
> 3. After screen turns black and wait a few more seconds, connect external
>    monitor to a type C port via a type C dongle and wake the device.
> 
> 4. Check if external monitor is also light up as an extended/mirrored
>    screen.
> 
> [Regression Potential]
> Medium.
> 
> Imre Deak (1):
>   drm/i915: Align power domain names with port names
> 
> José Roberto de Souza (8):
>   drm/i915/display: Move out code to return the digital_port of the aux
>     ch
>   drm/i915/display: Add intel_legacy_aux_to_power_domain()
>   drm/i915/display: Split hsw_power_well_enable() into two
>   drm/i915/tc/icl: Implement TC cold sequences
>   drm/i915/tc: Skip ref held check for TC legacy aux power wells
>   drm/i915/tc/tgl: Implement TC cold sequences
>   drm/i915/tc: Catch TC users accessing FIA registers without enable aux
>   drm/i915/tc: Do not warn when aux power well of static TC ports
>     timeout
> 
>  drivers/gpu/drm/i915/display/intel_display.c  |  24 +-
>  drivers/gpu/drm/i915/display/intel_display.h  |   2 +
>  .../drm/i915/display/intel_display_power.c    | 658 +++++++++++-------
>  .../drm/i915/display/intel_display_power.h    |  41 +-
>  drivers/gpu/drm/i915/display/intel_tc.c       |  92 ++-
>  drivers/gpu/drm/i915/i915_debugfs.c           |   3 +-
>  drivers/gpu/drm/i915/i915_reg.h               |   5 +
>  7 files changed, 546 insertions(+), 279 deletions(-)
> 
> -- 
> 2.25.1
> 
> 
> -- 
> kernel-team mailing list
> kernel-team at lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team



More information about the kernel-team mailing list