NAK: [SRU][U][PATCH 0/8] Fix TC port in legacy/static mode can't be detected due TCCOLD
Seth Forshee
seth.forshee at canonical.com
Thu Apr 30 21:29:32 UTC 2020
On Wed, Apr 22, 2020 at 03:04:12PM +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.
>
> [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.
The patches are okay for unstable, however there are some problems with
your submission. Primarily these lines:
(backported from drm-tip commit ...)
If you are going to add text to these lines, it must come after the
sha1. We have scripts which look for the '(cherry picked|backported)
from commit' strings, and if you add text in the middle it breaks these
scripts.
Also note that if these patches were only coming from a maintainer tree,
they should be sauce. However I see these patches in linux-next, so
please denote them like this:
(backported from commit ... linux-next)
Thanks,
Seth
More information about the kernel-team
mailing list