From patchwork Mon Jun 20 19:05:07 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bjorn Helgaas X-Patchwork-Id: 9188529 X-Patchwork-Delegate: bhelgaas@google.com Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 5A3C16089D for ; Mon, 20 Jun 2016 19:05:57 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 3FB2827CCB for ; Mon, 20 Jun 2016 19:05:57 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 3370F27CF5; Mon, 20 Jun 2016 19:05:57 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 85A6827CCB for ; Mon, 20 Jun 2016 19:05:56 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752884AbcFTTFu (ORCPT ); Mon, 20 Jun 2016 15:05:50 -0400 Received: from mail.kernel.org ([198.145.29.136]:36154 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752101AbcFTTFf (ORCPT ); Mon, 20 Jun 2016 15:05:35 -0400 Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id C10A3200B4; Mon, 20 Jun 2016 19:05:10 +0000 (UTC) Received: from localhost (unknown [173.216.248.176]) (using TLSv1.2 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id BFEC72013D; Mon, 20 Jun 2016 19:05:09 +0000 (UTC) Date: Mon, 20 Jun 2016 14:05:07 -0500 From: Bjorn Helgaas To: Andy Shevchenko Cc: Bjorn Helgaas , linux-pci@vger.kernel.org, "Rafael J. Wysocki" , Mika Westerberg , Lukas Wunner Subject: Re: [PATCH] PCI: Document connection between pci_power_t and hardware PM capability Message-ID: <20160620190445.GA15674@localhost> References: <20160618180056.7267.67911.stgit@bhelgaas-glaptop2.roam.corp.google.com> <1466336986.30123.164.camel@linux.intel.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1466336986.30123.164.camel@linux.intel.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-Virus-Scanned: ClamAV using ClamSMTP Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On Sun, Jun 19, 2016 at 02:49:46PM +0300, Andy Shevchenko wrote: > On Sat, 2016-06-18 at 13:00 -0500, Bjorn Helgaas wrote: > > The dev.pme_support field, pci_pm_init(), and pci_pme_capable() depend > > on > > the fact that the pci_power_t values (PCI_D0, PCI_D1, etc.) match the > > definition of the PME_Support field of the Power Management > > Capabilities > > register in the Power Management capability (see PCI Bus Power > > Management > > spec r1.2, sec 3.2.3). > > Nice to know! Thanks. > > If it matters: > Reviewed-by: Andy Shevchenko Definitely! I updated based on Lukas' comment and added your and Mika's Reviewed-by: commit 224abb67e6eb5ac062de9239163136d5ec3155c8 Author: Bjorn Helgaas Date: Fri Jun 17 15:23:52 2016 -0500 PCI: Document connection between pci_power_t and hardware PM capability The dev.pme_support field, pci_pm_init(), pci_pme_capable(), and pci_raw_set_power_state() depend on the fact that the pci_power_t values (PCI_D0, PCI_D1, etc.) match the definition of the Capabilities PME_Support and the Control/Status PowerState fields in the Power Management capability (see PCI Bus Power Management spec r1.2, sec 3.2.3). Add a note to this effect at the pci_power_t typedef. Signed-off-by: Bjorn Helgaas Reviewed-by: Andy Shevchenko Reviewed-by: Mika Westerberg --- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/include/linux/pci.h b/include/linux/pci.h index 8597b42..0a1a9e3 100644 --- a/include/linux/pci.h +++ b/include/linux/pci.h @@ -101,6 +101,10 @@ enum { DEVICE_COUNT_RESOURCE = PCI_NUM_RESOURCES, }; +/* + * pci_power_t values must match the bits in the Capabilities PME_Support + * and Control/Status PowerState fields in the Power Management capability. + */ typedef int __bitwise pci_power_t; #define PCI_D0 ((pci_power_t __force) 0)