From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751937AbcFUWlJ (ORCPT ); Tue, 21 Jun 2016 18:41:09 -0400 Received: from mail.kernel.org ([198.145.29.136]:36432 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751307AbcFUWlH (ORCPT ); Tue, 21 Jun 2016 18:41:07 -0400 Date: Tue, 21 Jun 2016 17:28:30 -0500 From: Bjorn Helgaas To: Johannes Thumshirn Cc: Bjorn Helgaas , linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Christoph Hellwig , Keith Busch , Jens Axboe , linux-nvme@lists.infradead.org, James Smart , Dick Kennedy , "James E.J. Bottomley" , "Martin K. Petersen" , linux-scsi@vger.kernel.org, Frank Haverkamp , Greg Kroah-Hartman , Jay Cliburn , Chris Snook , Jeff Kirsher , "David S. Miller" , netdev@vger.kernel.org, intel-wired-lan@lists.osuosl.org Subject: Re: [PATCH v3 0/6] Introduce pci_(request|release)_(mem|io)_regions Message-ID: <20160621222830.GF7603@localhost> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: 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 On Tue, Jun 07, 2016 at 09:44:00AM +0200, Johannes Thumshirn wrote: > The first patch in this series introduces the following 4 helper functions to > the PCI core: > > * pci_request_mem_regions() > * pci_request_io_regions() > * pci_release_mem_regions() > * pci_release_io_regions() > > which encapsulate the request and release of a PCI device's memory or I/O > bars. > > The subsequent patches convert the drivers, which use the > pci_request_selected_regions(pdev, > pci_select_bars(pdev, IORESOURCE_MEM), name); > and similar pattern to use the new interface. > > This was suggested by Christoph Hellwig in > http://lists.infradead.org/pipermail/linux-nvme/2016-May/004570.html and > tested on kernel v4.6 with NVMe. I applied all six of these to pci/resources for v4.8, thanks, Johannes. > Johannes Thumshirn (6): > PCI: Add helpers to request/release memory and I/O regions > NVMe: Use pci_(request|release)_mem_regions > lpfc: Use pci_(request|release)_mem_regions > GenWQE: Use pci_(request|release)_mem_regions > ethernet/intel: Use pci_(request|release)_mem_regions > alx: Use pci_(request|release)_mem_regions > > drivers/misc/genwqe/card_base.c | 13 +++++-------- > drivers/net/ethernet/atheros/alx/main.c | 12 +++++------- > drivers/net/ethernet/intel/e1000e/netdev.c | 6 ++---- > drivers/net/ethernet/intel/fm10k/fm10k_pci.c | 11 +++-------- > drivers/net/ethernet/intel/i40e/i40e_main.c | 9 +++------ > drivers/net/ethernet/intel/igb/igb_main.c | 10 +++------- > drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 9 +++------ > drivers/nvme/host/pci.c | 10 +++------- > drivers/scsi/lpfc/lpfc_init.c | 15 ++++---------- > include/linux/pci.h | 28 +++++++++++++++++++++++++++ From mboxrd@z Thu Jan 1 00:00:00 1970 From: helgaas@kernel.org (Bjorn Helgaas) Date: Tue, 21 Jun 2016 17:28:30 -0500 Subject: [PATCH v3 0/6] Introduce pci_(request|release)_(mem|io)_regions In-Reply-To: References: Message-ID: <20160621222830.GF7603@localhost> On Tue, Jun 07, 2016@09:44:00AM +0200, Johannes Thumshirn wrote: > The first patch in this series introduces the following 4 helper functions to > the PCI core: > > * pci_request_mem_regions() > * pci_request_io_regions() > * pci_release_mem_regions() > * pci_release_io_regions() > > which encapsulate the request and release of a PCI device's memory or I/O > bars. > > The subsequent patches convert the drivers, which use the > pci_request_selected_regions(pdev, > pci_select_bars(pdev, IORESOURCE_MEM), name); > and similar pattern to use the new interface. > > This was suggested by Christoph Hellwig in > http://lists.infradead.org/pipermail/linux-nvme/2016-May/004570.html and > tested on kernel v4.6 with NVMe. I applied all six of these to pci/resources for v4.8, thanks, Johannes. > Johannes Thumshirn (6): > PCI: Add helpers to request/release memory and I/O regions > NVMe: Use pci_(request|release)_mem_regions > lpfc: Use pci_(request|release)_mem_regions > GenWQE: Use pci_(request|release)_mem_regions > ethernet/intel: Use pci_(request|release)_mem_regions > alx: Use pci_(request|release)_mem_regions > > drivers/misc/genwqe/card_base.c | 13 +++++-------- > drivers/net/ethernet/atheros/alx/main.c | 12 +++++------- > drivers/net/ethernet/intel/e1000e/netdev.c | 6 ++---- > drivers/net/ethernet/intel/fm10k/fm10k_pci.c | 11 +++-------- > drivers/net/ethernet/intel/i40e/i40e_main.c | 9 +++------ > drivers/net/ethernet/intel/igb/igb_main.c | 10 +++------- > drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 9 +++------ > drivers/nvme/host/pci.c | 10 +++------- > drivers/scsi/lpfc/lpfc_init.c | 15 ++++---------- > include/linux/pci.h | 28 +++++++++++++++++++++++++++ From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bjorn Helgaas Date: Tue, 21 Jun 2016 17:28:30 -0500 Subject: [Intel-wired-lan] [PATCH v3 0/6] Introduce pci_(request|release)_(mem|io)_regions In-Reply-To: References: Message-ID: <20160621222830.GF7603@localhost> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: intel-wired-lan@osuosl.org List-ID: On Tue, Jun 07, 2016 at 09:44:00AM +0200, Johannes Thumshirn wrote: > The first patch in this series introduces the following 4 helper functions to > the PCI core: > > * pci_request_mem_regions() > * pci_request_io_regions() > * pci_release_mem_regions() > * pci_release_io_regions() > > which encapsulate the request and release of a PCI device's memory or I/O > bars. > > The subsequent patches convert the drivers, which use the > pci_request_selected_regions(pdev, > pci_select_bars(pdev, IORESOURCE_MEM), name); > and similar pattern to use the new interface. > > This was suggested by Christoph Hellwig in > http://lists.infradead.org/pipermail/linux-nvme/2016-May/004570.html and > tested on kernel v4.6 with NVMe. I applied all six of these to pci/resources for v4.8, thanks, Johannes. > Johannes Thumshirn (6): > PCI: Add helpers to request/release memory and I/O regions > NVMe: Use pci_(request|release)_mem_regions > lpfc: Use pci_(request|release)_mem_regions > GenWQE: Use pci_(request|release)_mem_regions > ethernet/intel: Use pci_(request|release)_mem_regions > alx: Use pci_(request|release)_mem_regions > > drivers/misc/genwqe/card_base.c | 13 +++++-------- > drivers/net/ethernet/atheros/alx/main.c | 12 +++++------- > drivers/net/ethernet/intel/e1000e/netdev.c | 6 ++---- > drivers/net/ethernet/intel/fm10k/fm10k_pci.c | 11 +++-------- > drivers/net/ethernet/intel/i40e/i40e_main.c | 9 +++------ > drivers/net/ethernet/intel/igb/igb_main.c | 10 +++------- > drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 9 +++------ > drivers/nvme/host/pci.c | 10 +++------- > drivers/scsi/lpfc/lpfc_init.c | 15 ++++---------- > include/linux/pci.h | 28 +++++++++++++++++++++++++++