Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753468AbeADP7u (ORCPT + 1 other); Thu, 4 Jan 2018 10:59:50 -0500 Received: from mail-wm0-f65.google.com ([74.125.82.65]:36805 "EHLO mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752902AbeADP7r (ORCPT ); Thu, 4 Jan 2018 10:59:47 -0500 X-Google-Smtp-Source: ACJfBov3uz/U9xVc2ml2Fil2mwfSLqvQ4F47U4vbmRm+7nvo5/Ub+PpMr9AdyY+IneQuI7ILOrEAPHYYDt2XdwFDzPE= MIME-Version: 1.0 In-Reply-To: <20180103213433.GA189303@bhelgaas-glaptop.roam.corp.google.com> References: <20180102175356.7999-1-romain.perier@gmail.com> <20180102221724.GE6211@bhelgaas-glaptop.roam.corp.google.com> <20180103213433.GA189303@bhelgaas-glaptop.roam.corp.google.com> From: Romain Perier Date: Thu, 4 Jan 2018 16:59:44 +0100 Message-ID: Subject: Re: [PATCH v17 0/4] Replace PCI pool by DMA pool API To: Bjorn Helgaas Cc: Jens Axboe , Andrew Morton , Dan Williams , Vinod Koul , Jeff Kirsher , Aviad Krawczyk , "James E.J. Bottomley" , "Martin K. Petersen" , linux-scsi@vger.kernel.org, Bjorn Helgaas , linux-pci@vger.kernel.org, dmaengine@vger.kernel.org, "David S. Miller" , netdev , Linux Kernel Mailing List , Greg Kroah-Hartman Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: Hi, Ok, will do. Thanks ! Romain 2018-01-03 22:34 GMT+01:00 Bjorn Helgaas : > On Tue, Jan 02, 2018 at 04:17:24PM -0600, Bjorn Helgaas wrote: >> On Tue, Jan 02, 2018 at 06:53:52PM +0100, Romain Perier wrote: >> > The current PCI pool API are simple macro functions direct expanded to >> > the appropriate dma pool functions. The prototypes are almost the same >> > and semantically, they are very similar. I propose to use the DMA pool >> > API directly and get rid of the old API. >> > >> > This set of patches, replaces the old API by the dma pool API >> > and remove the defines. >> > ... >> >> > Romain Perier (4): >> > block: DAC960: Replace PCI pool old API >> > net: e100: Replace PCI pool old API >> > hinic: Replace PCI pool old API >> > PCI: Remove PCI pool macro functions >> > >> > drivers/block/DAC960.c | 38 +++++++++++------------ >> > drivers/block/DAC960.h | 4 +-- >> > drivers/net/ethernet/huawei/hinic/hinic_hw_cmdq.c | 10 +++--- >> > drivers/net/ethernet/huawei/hinic/hinic_hw_cmdq.h | 2 +- >> > drivers/net/ethernet/intel/e100.c | 12 +++---- >> > include/linux/pci.h | 9 ------ >> > 6 files changed, 32 insertions(+), 43 deletions(-) >> >> Applied to pci/misc for v4.16, thanks! > > Oops, my mistake. I can't remove the macros themselves yet because > some of the uses were removed by patches that were applied via other > trees, and those patches are not in my tree. > > To avoid ordering dependencies during the merge window, I dropped the > "PCI: Remove PCI pool macro functions" patch. Please repost that > after all the removals have made it into Linus' tree. > > Bjorn