Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753532AbeADWOy (ORCPT + 1 other); Thu, 4 Jan 2018 17:14:54 -0500 Received: from mail.kernel.org ([198.145.29.99]:45042 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752550AbeADWOw (ORCPT ); Thu, 4 Jan 2018 17:14:52 -0500 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 210372190C Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=helgaas@kernel.org Date: Thu, 4 Jan 2018 16:14:48 -0600 From: Bjorn Helgaas To: "Rafael J. Wysocki" Cc: Linux PM , Greg Kroah-Hartman , Alan Stern , Kevin Hilman , LKML , Mika Westerberg , Ulf Hansson , linux-i2c , Linux PCI , Lee Jones , Andy Shevchenko , Wolfram Sang Subject: Re: [PATCH 7/7] PCI / PM: Use SMART_SUSPEND and LEAVE_SUSPENDED flags for PCIe ports Message-ID: <20180104221448.GH189897@bhelgaas-glaptop.roam.corp.google.com> References: <7742130.AaJQIxeI1n@aspire.rjw.lan> <2436726.ykYRzVD45y@aspire.rjw.lan> <33674270.0PFIgc82r6@aspire.rjw.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <33674270.0PFIgc82r6@aspire.rjw.lan> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On Wed, Jan 03, 2018 at 01:38:27AM +0100, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > Make the PCIe port driver set DPM_FLAG_SMART_SUSPEND and > DPM_FLAG_LEAVE_SUSPENDED for the devices handled by it to benefit > from the opportunistic optimizations in the PCI layer enabled by > these flags. > > Signed-off-by: Rafael J. Wysocki Acked-by: Bjorn Helgaas Please merge this along with the rest of the series. > --- > drivers/pci/pcie/portdrv_pci.c | 3 +++ > 1 file changed, 3 insertions(+) > > Index: linux-pm/drivers/pci/pcie/portdrv_pci.c > =================================================================== > --- linux-pm.orig/drivers/pci/pcie/portdrv_pci.c > +++ linux-pm/drivers/pci/pcie/portdrv_pci.c > @@ -150,6 +150,9 @@ static int pcie_portdrv_probe(struct pci > > pci_save_state(dev); > > + dev_pm_set_driver_flags(&dev->dev, DPM_FLAG_SMART_SUSPEND | > + DPM_FLAG_LEAVE_SUSPENDED); > + > if (pci_bridge_d3_possible(dev)) { > /* > * Keep the port resumed 100ms to make sure things like >