From: Will Deacon <will.deacon@arm.com>
To: Arnd Bergmann <arnd@arndb.de>
Cc: "linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
"bhelgaas@google.com" <bhelgaas@google.com>,
"jgunthorpe@obsidianresearch.com"
<jgunthorpe@obsidianresearch.com>,
"sthokal@xilinx.com" <sthokal@xilinx.com>
Subject: Re: [PATCH v7 0/4] Support for Generic PCI Host Controller
Date: Sun, 25 May 2014 11:43:59 +0100 [thread overview]
Message-ID: <20140525104359.GA2343@arm.com> (raw)
In-Reply-To: <201405232119.23891.arnd@arndb.de>
Hi Arnd,
On Fri, May 23, 2014 at 08:19:23PM +0100, Arnd Bergmann wrote:
> On Friday 23 May 2014, Will Deacon wrote:
> > - Removed redundant MIGHT_HAVE_PCI selections from multiplatform-enabled
> > SoCs
> >
> > - Included patch from Srikanth for generic Configuration Space access to
> > be factored out and used by other drivers
> >
> > - Fixed minor issue in I/O port allocation (I had a >= instead of a >)
> >
>
> It seems strange that you include the patch from Srikanth but then use
> a separate copy of the same code in patch 2. Is that intentional?
That was a combination of me not being sure about the status of Srikanth's
patch (the review didn't seem to conclude) but also then forgetting to move
over to using it.
However, judging by your most recent comments, I'm glad I didn't spend any
time on it!
Will
prev parent reply other threads:[~2014-05-25 10:44 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-23 16:51 [PATCH v7 0/4] Support for Generic PCI Host Controller Will Deacon
2014-05-23 16:51 ` [PATCH v7 1/4] ARM: kconfig: allow PCI support to be selected with ARCH_MULTIPLATFORM Will Deacon
2014-05-23 16:51 ` [PATCH v7 2/4] PCI: ARM: add support for generic PCI host controller Will Deacon
2014-05-23 16:51 ` [PATCH v7 3/4] MAINTAINERS: add entry for generic PCI host controller driver Will Deacon
2014-05-23 16:51 ` [PATCH v7 4/4] PCI: Generic Configuration Access Mechanism support Will Deacon
2014-05-23 19:22 ` Arnd Bergmann
2014-05-25 10:48 ` Will Deacon
2014-05-23 19:19 ` [PATCH v7 0/4] Support for Generic PCI Host Controller Arnd Bergmann
2014-05-25 10:43 ` Will Deacon [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140525104359.GA2343@arm.com \
--to=will.deacon@arm.com \
--cc=arnd@arndb.de \
--cc=bhelgaas@google.com \
--cc=jgunthorpe@obsidianresearch.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-pci@vger.kernel.org \
--cc=sthokal@xilinx.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).