[SRU] [Bionic/Unstable] [PATCH 0/7] Support Power Management for Thunderbolt Controller

[Kai-Heng Feng]

at 16:25, Hui Wang <hui.wang at canonical.com> wrote:

> Looks like the patch [7/7] is the key one to implement this feature. what  
> is the relation between [1/7~6/7] and [7/7], could we drop [1/7~6/7]?

Patch [1-2] fix boot_acl, which is later used by TBT runtime power  
management.
Patch [3-6] add system-wide power management support for native PCIe TBT,  
i.e. support for s2idle/s3.
Patch 7 adds runtime power management for native PCIe TBT.

So all patches in this series are required.

Kai-Heng

>
> Regards,
> Hui.
>
> On 2018年08月28日 14:43, Kai-Heng Feng wrote:
>> BugLink: https://bugs.launchpad.net/bugs/1789358
>>
>> [Impact]
>> New TBT controllers may use Native PCIe Enumeration (i.e. just like
>> other normal PCIe device listed under lspci) instead of ACPI hotplug.
>>
>> Since TBT controller stops getting unplugged from PCIe bus, it draws
>> additional power.
>>
>> [Fix]
>> Support runtime power management for TBT controllers, so it can be put
>> to D3 to save powers.
>>
>> [Test]
>> I tested and verified this patch series can make Lenovo T480's TBT
>> controller enters PCI D3 at runtime.
>>
>> [Regression Potential]
>> Low. Almost all TBT controllers in the wild still uses ACPI hotplug,
>> this series won't affect those users.
>>
>> Mika Westerberg (7):
>>    thunderbolt: Handle NULL boot ACL entries properly
>>    thunderbolt: Notify userspace when boot_acl is changed
>>    thunderbolt: Use 64-bit DMA mask if supported by the platform
>>    thunderbolt: Do not unnecessarily call ICM get route
>>    thunderbolt: No need to take tb->lock in domain suspend/complete
>>    thunderbolt: Use correct ICM commands in system suspend
>>    thunderbolt: Add support for runtime PM
>>
>>   drivers/thunderbolt/domain.c  |  59 +++++++++---
>>   drivers/thunderbolt/icm.c     | 174 ++++++++++++++++++++++++++++------
>>   drivers/thunderbolt/nhi.c     |  46 ++++++++-
>>   drivers/thunderbolt/switch.c  |  65 ++++++++++++-
>>   drivers/thunderbolt/tb.h      |  10 ++
>>   drivers/thunderbolt/tb_msgs.h |   4 +
>>   drivers/thunderbolt/xdomain.c |  18 ++++
>>   7 files changed, 328 insertions(+), 48 deletions(-)