From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758676AbcFAP0q (ORCPT ); Wed, 1 Jun 2016 11:26:46 -0400 Received: from mx2.suse.de ([195.135.220.15]:36524 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751570AbcFAP0n (ORCPT ); Wed, 1 Jun 2016 11:26:43 -0400 Date: Wed, 1 Jun 2016 17:26:40 +0200 From: Johannes Thumshirn To: Jeff Kirsher Cc: Christoph Hellwig , Bjorn Helgaas , linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Jay Cliburn , Chris Snook , "David S. Miller" , netdev@vger.kernel.org, intel-wired-lan@lists.osuosl.org Subject: Re: [PATCH 5/5] ethernet: Use pci_(request|release)_mem_regions Message-ID: <20160601152640.tzhn5itms73q5fbw@c203.arch.suse.de> References: <3f24b687c31dea31a03cd3a529de790e942dd5bb.1464694724.git.jthumshirn@suse.de> <20160601075956.GD11923@infradead.org> <20160601085112.ve6k7bzspapjsswf@c203.arch.suse.de> <1464793005.2960.3.camel@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1464793005.2960.3.camel@intel.com> User-Agent: Mutt/1.6.1-neo () Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jun 01, 2016 at 07:56:45AM -0700, Jeff Kirsher wrote: > On Wed, 2016-06-01 at 10:51 +0200, Johannes Thumshirn wrote: > > On Wed, Jun 01, 2016 at 12:59:56AM -0700, Christoph Hellwig wrote: > > > On Tue, May 31, 2016 at 02:05:13PM +0200, Johannes Thumshirn wrote: > > > > Now that we do have pci_request_mem_regions() and > > pci_release_mem_regions() at > > > > hand, use it in the ethernet drivers. > > >  > > > This should probably be one patch per driver. > > > > I though if I do one patch per subsystem it'll be less a hassle for the > > individual maintainers, but if the netdev people want it as split up, > > I'll be > > doing it of cause. > > Since almost all the changes are to Intel wired LAN drivers, if you just > split off the atheros change into a separate patch, I would be happy.  Then > you could keep just one patch to change all the Intel drivers. That should be doable. I'll send a v2 with all requested changes tomorrow. -- Johannes Thumshirn Storage jthumshirn@suse.de +49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johannes Thumshirn Date: Wed, 1 Jun 2016 17:26:40 +0200 Subject: [Intel-wired-lan] [PATCH 5/5] ethernet: Use pci_(request|release)_mem_regions In-Reply-To: <1464793005.2960.3.camel@intel.com> References: <3f24b687c31dea31a03cd3a529de790e942dd5bb.1464694724.git.jthumshirn@suse.de> <20160601075956.GD11923@infradead.org> <20160601085112.ve6k7bzspapjsswf@c203.arch.suse.de> <1464793005.2960.3.camel@intel.com> Message-ID: <20160601152640.tzhn5itms73q5fbw@c203.arch.suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: intel-wired-lan@osuosl.org List-ID: On Wed, Jun 01, 2016 at 07:56:45AM -0700, Jeff Kirsher wrote: > On Wed, 2016-06-01 at 10:51 +0200, Johannes Thumshirn wrote: > > On Wed, Jun 01, 2016 at 12:59:56AM -0700, Christoph Hellwig wrote: > > > On Tue, May 31, 2016 at 02:05:13PM +0200, Johannes Thumshirn wrote: > > > > Now that we do have pci_request_mem_regions() and > > pci_release_mem_regions() at > > > > hand, use it in the ethernet drivers. > > >? > > > This should probably be one patch per driver. > > > > I though if I do one patch per subsystem it'll be less a hassle for the > > individual maintainers, but if the netdev people want it as split up, > > I'll be > > doing it of cause. > > Since almost all the changes are to Intel wired LAN drivers, if you just > split off the atheros change into a separate patch, I would be happy. ?Then > you could keep just one patch to change all the Intel drivers. That should be doable. I'll send a v2 with all requested changes tomorrow. -- Johannes Thumshirn Storage jthumshirn at suse.de +49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 N?rnberg GF: Felix Imend?rffer, Jane Smithard, Graham Norton HRB 21284 (AG N?rnberg) Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850