Re: [PATCH] xhci-pci: Allow host runtime PM as default for Intel Alder Lake xHCI

2021-04-07 Thread Mathias Nyman
On 7.4.2021 2.35, Azhar Shaikh wrote:
> From: Abhijeet Rao 
> 
> In the same way as Intel Tiger Lake TCSS (Type-C Subsystem) the Alder Lake
> TCSS xHCI needs to be runtime suspended whenever possible to allow the
> TCSS hardware block to enter D3cold and thus save energy.
> 
> Signed-off-by: Abhijeet Rao 
> Signed-off-by: Nikunj A. Dadhania 
> Signed-off-by: Azhar Shaikh 
> ---
>  drivers/usb/host/xhci-pci.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)

Thanks, Added to queue

-Mathias


[PATCH] xhci-pci: Allow host runtime PM as default for Intel Alder Lake xHCI

2021-04-06 Thread Azhar Shaikh
From: Abhijeet Rao 

In the same way as Intel Tiger Lake TCSS (Type-C Subsystem) the Alder Lake
TCSS xHCI needs to be runtime suspended whenever possible to allow the
TCSS hardware block to enter D3cold and thus save energy.

Signed-off-by: Abhijeet Rao 
Signed-off-by: Nikunj A. Dadhania 
Signed-off-by: Azhar Shaikh 
---
 drivers/usb/host/xhci-pci.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c
index 5bbccc9a0179..a858add8436c 100644
--- a/drivers/usb/host/xhci-pci.c
+++ b/drivers/usb/host/xhci-pci.c
@@ -57,6 +57,7 @@
 #define PCI_DEVICE_ID_INTEL_CML_XHCI   0xa3af
 #define PCI_DEVICE_ID_INTEL_TIGER_LAKE_XHCI0x9a13
 #define PCI_DEVICE_ID_INTEL_MAPLE_RIDGE_XHCI   0x1138
+#define PCI_DEVICE_ID_INTEL_ALDER_LAKE_XHCI0x461e
 
 #define PCI_DEVICE_ID_AMD_PROMONTORYA_40x43b9
 #define PCI_DEVICE_ID_AMD_PROMONTORYA_30x43ba
@@ -243,7 +244,8 @@ static void xhci_pci_quirks(struct device *dev, struct 
xhci_hcd *xhci)
 pdev->device == PCI_DEVICE_ID_INTEL_TITAN_RIDGE_DD_XHCI ||
 pdev->device == PCI_DEVICE_ID_INTEL_ICE_LAKE_XHCI ||
 pdev->device == PCI_DEVICE_ID_INTEL_TIGER_LAKE_XHCI ||
-pdev->device == PCI_DEVICE_ID_INTEL_MAPLE_RIDGE_XHCI))
+pdev->device == PCI_DEVICE_ID_INTEL_MAPLE_RIDGE_XHCI ||
+pdev->device == PCI_DEVICE_ID_INTEL_ALDER_LAKE_XHCI))
xhci->quirks |= XHCI_DEFAULT_PM_RUNTIME_ALLOW;
 
if (pdev->vendor == PCI_VENDOR_ID_ETRON &&
-- 
2.17.1