From: Thierry Reding <thierry.reding@gmail.com>
To: Jason Cooper <jason@lakedaemon.net>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
Bjorn Helgaas <bhelgaas@google.com>,
linux-pci@vger.kernel.org, Russell King <linux@arm.linux.org.uk>,
Grant Likely <grant.likely@secretlab.ca>,
Rob Herring <rob.herring@calxeda.com>,
Thomas Gleixner <tglx@linutronix.de>,
Andrew Lunn <andrew@lunn.ch>,
Gregory Clement <gregory.clement@free-electrons.com>,
Lior Amsalem <alior@marvell.com>,
Maen Suleiman <maen@marvell.com>,
linux-arm-kernel@lists.infradead.org,
Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
Subject: Re: [PATCHv6 00/13] MSI support for Marvell EBU PCIe driver
Date: Wed, 7 Aug 2013 08:59:56 +0200 [thread overview]
Message-ID: <20130807065954.GA22906@manwe> (raw)
In-Reply-To: <20130806200256.GB19280@titan.lakedaemon.net>
[-- Attachment #1: Type: text/plain, Size: 1214 bytes --]
On Tue, Aug 06, 2013 at 04:02:56PM -0400, Jason Cooper wrote:
> Thomas,
>
> In order to get some testing in -next, I've *tentatively* applied this
> series in the following way:
>
> +---+ mvebu/msi_common (1,2,3,10)
> |
> +---+---+ mvebu/msi_irq (4-9)
> |
> +---+---+ mvebu/msi_mvebu (11-13)
>
> I had to make one change to patch #9, I protected the last two blocks
> with #ifdef CONFIG_PCI_MSI to prevent a build failure at that point.
>
> mvebu/msi_common should be appropriate for Thierry to base off, however,
> I make no guarantees about the stability of the branch until I send a
> PR for it. Code-wise, it should be fine, but the commit-ids may change
> due to adding Tested-by's, etc.
I noticed that patch 10 has my Reviewed-by, even though my comments
weren't addressed. They were about stylistic issues but I'd still like
to see them fixed now rather than having to fix them up later. Also I
think I gave my Tested-by on those patches somewhere already and I think
it's unlikely that anybody else will test them, so with the consistency
style fixes applied I don't see any reason why that branch can't be
stabilized right away.
Thierry
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: thierry.reding@gmail.com (Thierry Reding)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv6 00/13] MSI support for Marvell EBU PCIe driver
Date: Wed, 7 Aug 2013 08:59:56 +0200 [thread overview]
Message-ID: <20130807065954.GA22906@manwe> (raw)
In-Reply-To: <20130806200256.GB19280@titan.lakedaemon.net>
On Tue, Aug 06, 2013 at 04:02:56PM -0400, Jason Cooper wrote:
> Thomas,
>
> In order to get some testing in -next, I've *tentatively* applied this
> series in the following way:
>
> +---+ mvebu/msi_common (1,2,3,10)
> |
> +---+---+ mvebu/msi_irq (4-9)
> |
> +---+---+ mvebu/msi_mvebu (11-13)
>
> I had to make one change to patch #9, I protected the last two blocks
> with #ifdef CONFIG_PCI_MSI to prevent a build failure at that point.
>
> mvebu/msi_common should be appropriate for Thierry to base off, however,
> I make no guarantees about the stability of the branch until I send a
> PR for it. Code-wise, it should be fine, but the commit-ids may change
> due to adding Tested-by's, etc.
I noticed that patch 10 has my Reviewed-by, even though my comments
weren't addressed. They were about stylistic issues but I'd still like
to see them fixed now rather than having to fix them up later. Also I
think I gave my Tested-by on those patches somewhere already and I think
it's unlikely that anybody else will test them, so with the consistency
style fixes applied I don't see any reason why that branch can't be
stabilized right away.
Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130807/10c41cd0/attachment.sig>
next prev parent reply other threads:[~2013-08-07 7:00 UTC|newest]
Thread overview: 65+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-01 13:25 [PATCHv6 00/13] MSI support for Marvell EBU PCIe driver Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` [PATCHv6 01/13] PCI: use weak functions for MSI arch-specific functions Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` [PATCHv6 02/13] PCI: remove ARCH_SUPPORTS_MSI kconfig option Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 16:48 ` Bjorn Helgaas
2013-08-01 16:48 ` Bjorn Helgaas
2013-08-01 16:48 ` Bjorn Helgaas
2013-08-01 13:25 ` [PATCHv6 03/13] PCI: Introduce new MSI chip infrastructure Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` [PATCHv6 04/13] irqdomain: add irq_alloc_mapping() function Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` [PATCHv6 05/13] irqdomain: refactor __irq_domain_add() Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` [PATCHv6 06/13] irqdomain: add support to associate an irq_domain with a msi_chip Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` [PATCHv6 07/13] irqdomain: add function to find a MSI irq_domain Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` [PATCHv6 08/13] irqchip: armada-370-xp: properly request resources Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` [PATCHv6 09/13] irqchip: armada-370-xp: implement MSI support Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` [PATCHv6 10/13] ARM: pci: add ->add_bus() and ->remove_bus() hooks to hw_pci Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` [PATCHv6 11/13] ARM: mvebu: the MPIC now provides MSI controller features Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` [PATCHv6 12/13] PCI: mvebu: add support for MSI Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 13:25 ` [PATCHv6 13/13] ARM: mvebu: link PCIe controllers to the MSI controller Thomas Petazzoni
2013-08-01 13:25 ` Thomas Petazzoni
2013-08-01 16:08 ` [PATCHv6 00/13] MSI support for Marvell EBU PCIe driver Stephen Warren
2013-08-01 16:08 ` Stephen Warren
2013-08-01 16:14 ` Thomas Petazzoni
2013-08-01 16:14 ` Thomas Petazzoni
2013-08-01 16:50 ` Bjorn Helgaas
2013-08-01 16:50 ` Bjorn Helgaas
2013-08-01 17:58 ` Thomas Petazzoni
2013-08-01 17:58 ` Thomas Petazzoni
2013-08-06 18:10 ` Thomas Petazzoni
2013-08-06 18:10 ` Thomas Petazzoni
2013-08-06 22:39 ` Benjamin Herrenschmidt
2013-08-06 22:39 ` Benjamin Herrenschmidt
2013-08-07 9:03 ` Thomas Petazzoni
2013-08-07 9:03 ` Thomas Petazzoni
2013-09-18 20:23 ` Grant Likely
2013-09-18 20:23 ` Grant Likely
2013-08-06 18:22 ` Daniel Price
2013-08-06 19:30 ` Jason Cooper
2013-08-06 20:02 ` Jason Cooper
2013-08-06 20:02 ` Jason Cooper
2013-08-06 20:08 ` Stephen Warren
2013-08-06 20:08 ` Stephen Warren
2013-08-07 15:51 ` Jason Cooper
2013-08-07 15:51 ` Jason Cooper
2013-08-07 6:59 ` Thierry Reding [this message]
2013-08-07 6:59 ` Thierry Reding
2013-08-07 15:55 ` Jason Cooper
2013-08-07 15:55 ` Jason Cooper
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=20130807065954.GA22906@manwe \
--to=thierry.reding@gmail.com \
--cc=alior@marvell.com \
--cc=andrew@lunn.ch \
--cc=bhelgaas@google.com \
--cc=ezequiel.garcia@free-electrons.com \
--cc=grant.likely@secretlab.ca \
--cc=gregory.clement@free-electrons.com \
--cc=jason@lakedaemon.net \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-pci@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=maen@marvell.com \
--cc=rob.herring@calxeda.com \
--cc=tglx@linutronix.de \
--cc=thomas.petazzoni@free-electrons.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.