From: "Rafael J. Wysocki" <rjw@sisk.pl> To: Jesse Barnes <jbarnes@virtuousgeek.org> Cc: Linux PCI <linux-pci@vger.kernel.org>, LKML <linux-kernel@vger.kernel.org>, ACPI Devel Maling List <linux-acpi@vger.kernel.org>, Francois Romieu <romieu@fr.zoreil.com>, pm list <linux-pm@lists.linux-foundation.org>, Matthew Garrett <mjg@redhat.com> Subject: [PATCH 1/12] PCI PM: Add function for checking PME status of devices Date: Sun, 27 Dec 2009 20:59:06 +0100 [thread overview] Message-ID: <200912272059.06718.rjw@sisk.pl> (raw) In-Reply-To: <200912272057.10443.rjw@sisk.pl> From: Rafael J. Wysocki <rjw@sisk.pl> Add function pci_check_pme_status() that will check the PME status bit of given device and clear it along with the PME enable bit. It will be necessary for PCI run-time power management. Based on a patch from Shaohua Li <shaohua.li@intel.com> Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl> --- drivers/pci/pci.c | 35 +++++++++++++++++++++++++++++++++++ drivers/pci/pci.h | 1 + 2 files changed, 36 insertions(+) Index: linux-2.6/drivers/pci/pci.h =================================================================== --- linux-2.6.orig/drivers/pci/pci.h +++ linux-2.6/drivers/pci/pci.h @@ -49,6 +49,7 @@ struct pci_platform_pm_ops { extern int pci_set_platform_pm(struct pci_platform_pm_ops *ops); extern void pci_update_current_state(struct pci_dev *dev, pci_power_t state); extern void pci_disable_enabled_device(struct pci_dev *dev); +extern bool pci_check_pme_status(struct pci_dev *dev); extern void pci_pm_init(struct pci_dev *dev); extern void platform_pci_wakeup_init(struct pci_dev *dev); extern void pci_allocate_cap_save_buffers(struct pci_dev *dev); Index: linux-2.6/drivers/pci/pci.c =================================================================== --- linux-2.6.orig/drivers/pci/pci.c +++ linux-2.6/drivers/pci/pci.c @@ -1180,6 +1180,41 @@ int pci_set_pcie_reset_state(struct pci_ } /** + * pci_check_pme_status - Check if given device has generated PME. + * @dev: Device to check. + * + * Check the PME status of the device and if set, clear it and clear PME enable + * (if set). Return 'true' if PME status and PME enable were both set or + * 'false' otherwise. + */ +bool pci_check_pme_status(struct pci_dev *dev) +{ + int pmcsr_pos; + u16 pmcsr; + bool ret = false; + + if (!dev->pm_cap) + return false; + + pmcsr_pos = dev->pm_cap + PCI_PM_CTRL; + pci_read_config_word(dev, pmcsr_pos, &pmcsr); + if (!(pmcsr & PCI_PM_CTRL_PME_STATUS)) + return false; + + /* Clear PME status. */ + pmcsr |= PCI_PM_CTRL_PME_STATUS; + if (pmcsr & PCI_PM_CTRL_PME_ENABLE) { + /* Disable PME to avoid interrupt flood. */ + pmcsr &= ~PCI_PM_CTRL_PME_ENABLE; + ret = true; + } + + pci_write_config_word(dev, pmcsr_pos, pmcsr); + + return ret; +} + +/** * pci_pme_capable - check the capability of PCI device to generate PME# * @dev: PCI device to handle. * @state: PCI state from which device will issue PME#.
WARNING: multiple messages have this Message-ID (diff)
From: "Rafael J. Wysocki" <rjw@sisk.pl> To: Jesse Barnes <jbarnes@virtuousgeek.org> Cc: Matthew Garrett <mjg@redhat.com>, Len Brown <lenb@kernel.org>, LKML <linux-kernel@vger.kernel.org>, pm list <linux-pm@lists.linux-foundation.org>, Alan Stern <stern@rowland.harvard.edu>, ACPI Devel Maling List <linux-acpi@vger.kernel.org>, Linux PCI <linux-pci@vger.kernel.org>, Oliver Neukum <oliver@neukum.org>, Bjorn Helgaas <bjorn.helgaas@hp.com>, Shaohua Li <shaohua.li@intel.com>, Francois Romieu <romieu@fr.zoreil.com> Subject: [PATCH 1/12] PCI PM: Add function for checking PME status of devices Date: Sun, 27 Dec 2009 20:59:06 +0100 [thread overview] Message-ID: <200912272059.06718.rjw@sisk.pl> (raw) In-Reply-To: <200912272057.10443.rjw@sisk.pl> From: Rafael J. Wysocki <rjw@sisk.pl> Add function pci_check_pme_status() that will check the PME status bit of given device and clear it along with the PME enable bit. It will be necessary for PCI run-time power management. Based on a patch from Shaohua Li <shaohua.li@intel.com> Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl> --- drivers/pci/pci.c | 35 +++++++++++++++++++++++++++++++++++ drivers/pci/pci.h | 1 + 2 files changed, 36 insertions(+) Index: linux-2.6/drivers/pci/pci.h =================================================================== --- linux-2.6.orig/drivers/pci/pci.h +++ linux-2.6/drivers/pci/pci.h @@ -49,6 +49,7 @@ struct pci_platform_pm_ops { extern int pci_set_platform_pm(struct pci_platform_pm_ops *ops); extern void pci_update_current_state(struct pci_dev *dev, pci_power_t state); extern void pci_disable_enabled_device(struct pci_dev *dev); +extern bool pci_check_pme_status(struct pci_dev *dev); extern void pci_pm_init(struct pci_dev *dev); extern void platform_pci_wakeup_init(struct pci_dev *dev); extern void pci_allocate_cap_save_buffers(struct pci_dev *dev); Index: linux-2.6/drivers/pci/pci.c =================================================================== --- linux-2.6.orig/drivers/pci/pci.c +++ linux-2.6/drivers/pci/pci.c @@ -1180,6 +1180,41 @@ int pci_set_pcie_reset_state(struct pci_ } /** + * pci_check_pme_status - Check if given device has generated PME. + * @dev: Device to check. + * + * Check the PME status of the device and if set, clear it and clear PME enable + * (if set). Return 'true' if PME status and PME enable were both set or + * 'false' otherwise. + */ +bool pci_check_pme_status(struct pci_dev *dev) +{ + int pmcsr_pos; + u16 pmcsr; + bool ret = false; + + if (!dev->pm_cap) + return false; + + pmcsr_pos = dev->pm_cap + PCI_PM_CTRL; + pci_read_config_word(dev, pmcsr_pos, &pmcsr); + if (!(pmcsr & PCI_PM_CTRL_PME_STATUS)) + return false; + + /* Clear PME status. */ + pmcsr |= PCI_PM_CTRL_PME_STATUS; + if (pmcsr & PCI_PM_CTRL_PME_ENABLE) { + /* Disable PME to avoid interrupt flood. */ + pmcsr &= ~PCI_PM_CTRL_PME_ENABLE; + ret = true; + } + + pci_write_config_word(dev, pmcsr_pos, pmcsr); + + return ret; +} + +/** * pci_pme_capable - check the capability of PCI device to generate PME# * @dev: PCI device to handle. * @state: PCI state from which device will issue PME#.
next prev parent reply other threads:[~2009-12-27 19:59 UTC|newest] Thread overview: 68+ messages / expand[flat|nested] mbox.gz Atom feed top 2009-12-27 19:57 [PATCH 0/12] PCI run-time PM support (rev. 2) Rafael J. Wysocki 2009-12-27 19:59 ` Rafael J. Wysocki [this message] 2009-12-27 19:59 ` [PATCH 1/12] PCI PM: Add function for checking PME status of devices Rafael J. Wysocki 2010-01-06 21:46 ` Jesse Barnes 2010-01-06 21:46 ` Jesse Barnes 2009-12-27 20:00 ` [PATCH 2/12] PCI / PM: Propagate wake-up enable for PCIe devices too Rafael J. Wysocki 2010-01-04 23:40 ` Jesse Barnes 2010-01-05 21:27 ` Rafael J. Wysocki 2010-01-05 21:27 ` Rafael J. Wysocki 2010-01-04 23:40 ` Jesse Barnes 2009-12-27 20:00 ` Rafael J. Wysocki 2009-12-27 20:01 ` [PATCH 3/12] PCI PM: PCIe PME root port service driver (rev. 5) Rafael J. Wysocki 2009-12-27 20:01 ` Rafael J. Wysocki 2010-01-06 21:53 ` Jesse Barnes 2010-01-06 21:53 ` Jesse Barnes 2009-12-27 20:02 ` [PATCH 4/12] PCI PM: Make it possible to force using INTx for PCIe PME signaling Rafael J. Wysocki 2009-12-27 20:02 ` Rafael J. Wysocki 2010-01-06 21:56 ` Jesse Barnes 2010-01-06 21:56 ` Jesse Barnes 2010-01-06 22:02 ` Matthew Garrett 2010-01-06 23:00 ` Rafael J. Wysocki 2010-01-06 23:00 ` Rafael J. Wysocki 2010-01-08 20:08 ` Len Brown 2010-01-08 20:25 ` Greg KH 2010-01-08 20:25 ` Greg KH 2010-01-08 20:08 ` Len Brown 2010-01-06 22:02 ` Matthew Garrett 2009-12-27 20:03 ` [PATCH 5/12] ACPI: Add infrastructure for refcounting GPE consumers Rafael J. Wysocki 2009-12-27 20:03 ` Rafael J. Wysocki 2009-12-27 20:03 ` [PATCH 6/12] ACPI: Add support for new refcounted GPE API to drivers Rafael J. Wysocki 2009-12-27 20:03 ` Rafael J. Wysocki 2009-12-27 20:04 ` [PATCH 7/12] ACPI: Remove old GPE API and transition code entirely to new one Rafael J. Wysocki 2009-12-27 20:04 ` Rafael J. Wysocki 2009-12-27 20:05 ` [PATCH 8/12] ACPI / PM: Add more run-time wake-up fields Rafael J. Wysocki 2010-01-08 20:39 ` Len Brown 2010-01-08 23:27 ` Rafael J. Wysocki 2010-01-08 23:27 ` Rafael J. Wysocki 2010-01-08 20:39 ` Len Brown 2009-12-27 20:05 ` Rafael J. Wysocki 2009-12-27 20:06 ` [PATCH 9/12] ACPI / PM: Introduce acpi_pm_wakeup_power() Rafael J. Wysocki 2010-01-06 22:00 ` Jesse Barnes 2010-01-06 23:11 ` Rafael J. Wysocki 2010-01-06 23:11 ` Rafael J. Wysocki 2010-01-07 21:11 ` Rafael J. Wysocki 2010-01-07 21:11 ` Rafael J. Wysocki 2010-01-06 22:00 ` Jesse Barnes 2009-12-27 20:06 ` Rafael J. Wysocki 2009-12-27 20:07 ` [PATCH 10/12] PCI / ACPI / PM: Platform support for PCI PME wake-up (rev. 6) Rafael J. Wysocki 2010-01-06 22:04 ` Jesse Barnes 2010-01-07 21:18 ` Rafael J. Wysocki 2010-01-07 21:18 ` Rafael J. Wysocki 2010-01-06 22:04 ` Jesse Barnes 2009-12-27 20:07 ` Rafael J. Wysocki 2009-12-27 20:08 ` [PATCH 11/12] PCI PM: Run-time callbacks for PCI bus type (rev. 2) Rafael J. Wysocki 2009-12-27 20:08 ` Rafael J. Wysocki 2010-01-06 22:06 ` Jesse Barnes 2010-01-06 22:06 ` Jesse Barnes 2009-12-27 20:11 ` [PATCH 12/12] PM / r8169: Add simplified run-time PM support Rafael J. Wysocki 2009-12-27 20:11 ` Rafael J. Wysocki 2010-01-01 19:06 ` [PATCH 12/12] PM / r8169: Add simplified run-time PM support (rev. 2) Rafael J. Wysocki 2010-01-01 19:06 ` Rafael J. Wysocki 2010-01-01 1:29 ` [PATCH][RFC] e1000e: Add basic runtime PM support (was: [PATCH 0/12] PCI run-time PM support (rev. 2)) Rafael J. Wysocki 2010-01-01 1:29 ` Rafael J. Wysocki 2010-01-01 1:29 ` Rafael J. Wysocki 2010-01-01 19:03 ` [PATCH][RFC] e1000e: Add basic runtime PM support (rev. 2) " Rafael J. Wysocki 2010-01-01 21:51 ` [PATCH][RFC] e1000e: Add basic runtime PM support (rev. 3) " Rafael J. Wysocki 2010-01-01 21:51 ` Rafael J. Wysocki 2010-01-01 19:03 ` [PATCH][RFC] e1000e: Add basic runtime PM support (rev. 2) " Rafael J. Wysocki
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=200912272059.06718.rjw@sisk.pl \ --to=rjw@sisk.pl \ --cc=jbarnes@virtuousgeek.org \ --cc=linux-acpi@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pci@vger.kernel.org \ --cc=linux-pm@lists.linux-foundation.org \ --cc=mjg@redhat.com \ --cc=romieu@fr.zoreil.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.