[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20160601152640.tzhn5itms73q5fbw@c203.arch.suse.de>
Date: Wed, 1 Jun 2016 17:26:40 +0200
From: Johannes Thumshirn <jthumshirn@...e.de>
To: Jeff Kirsher <jeffrey.t.kirsher@...el.com>
Cc: Christoph Hellwig <hch@...radead.org>,
Bjorn Helgaas <bhelgaas@...gle.com>, linux-pci@...r.kernel.org,
linux-kernel@...r.kernel.org, Jay Cliburn <jcliburn@...il.com>,
Chris Snook <chris.snook@...il.com>,
"David S. Miller" <davem@...emloft.net>, netdev@...r.kernel.org,
intel-wired-lan@...ts.osuosl.org
Subject: Re: [PATCH 5/5] ethernet: Use pci_(request|release)_mem_regions
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@...e.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
Powered by blists - more mailing lists