Hi Felipe, On Fri, Aug 21, 2020 at 04:11:01PM +0300, Heikki Krogerus wrote: > From: Raymond Tan <raymond.tan@xxxxxxxxx> > > Similar to some other IA platforms, Elkhart Lake too depends on the > PMU register write to request transition of Dx power state. > > Thus, we add the PCI_DEVICE_ID_INTEL_EHLLP to the list of devices that > shall execute the ACPI _DSM method during D0/D3 sequence. >a > [heikki.krogerus@xxxxxxxxxxxxxxx: included Fixes tag] > > Fixes: dbb0569de852 ("usb: dwc3: pci: Add Support for Intel Elkhart Lake Devices") > Cc: stable@xxxxxxxxxxxxxxx > Signed-off-by: Raymond Tan <raymond.tan@xxxxxxxxx> > Signed-off-by: Heikki Krogerus <heikki.krogerus@xxxxxxxxxxxxxxx> > --- > drivers/usb/dwc3/dwc3-pci.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/usb/dwc3/dwc3-pci.c b/drivers/usb/dwc3/dwc3-pci.c > index f5a61f57c74f0..242b6210380a4 100644 > --- a/drivers/usb/dwc3/dwc3-pci.c > +++ b/drivers/usb/dwc3/dwc3-pci.c > @@ -147,7 +147,8 @@ static int dwc3_pci_quirks(struct dwc3_pci *dwc) > > if (pdev->vendor == PCI_VENDOR_ID_INTEL) { > if (pdev->device == PCI_DEVICE_ID_INTEL_BXT || > - pdev->device == PCI_DEVICE_ID_INTEL_BXT_M) { > + pdev->device == PCI_DEVICE_ID_INTEL_BXT_M || > + pdev->device == PCI_DEVICE_ID_INTEL_EHLLP) { > guid_parse(PCI_INTEL_BXT_DSM_GUID, &dwc->guid); > dwc->has_dsm_for_pm = true; > } I think this has gone under your radar. Let me know if you want anything to be changed. thanks, -- heikki