From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Bjorn Helgaas <bhelgaas@google.com>,
linux-pci@vger.kernel.org, Russell King <linux@arm.linux.org.uk>,
Rob Herring <rob.herring@calxeda.com>,
Thomas Gleixner <tglx@linutronix.de>,
Jason Cooper <jason@lakedaemon.net>, Andrew Lunn <andrew@lunn.ch>,
Gregory Clement <gregory.clement@free-electrons.com>,
Ezequiel Garcia <ezequiel.garcia@free-electrons.com>,
linux-arm-kernel@lists.infradead.org,
Maen Suleiman <maen@marvell.com>,
Lior Amsalem <alior@marvell.com>,
Thierry Reding <thierry.reding@gmail.com>
Subject: Re: [PATCHv7 07/13] irqdomain: add function to find a MSI irq_domain
Date: Thu, 08 Aug 2013 08:45:36 +1000 [thread overview]
Message-ID: <1375915536.12551.9.camel@pasglop> (raw)
In-Reply-To: <1375915344.12551.6.camel@pasglop>
On Thu, 2013-08-08 at 08:42 +1000, Benjamin Herrenschmidt wrote:
> On Thu, 2013-08-08 at 08:31 +1000, Benjamin Herrenschmidt wrote:
> > On Thu, 2013-08-08 at 00:04 +0200, Thomas Petazzoni wrote:
> > > Again, this has been discussed at lengths in the previous iterations,
> > > for which I already gave you all the links, as you requested in a
> > > private e-mail. It'd be great if this discussion was read seriously,
> > > because I really have the feeling we are restarting from zero on this
> > > whole MSI thing...
> >
> > Well, two things here:
> >
> > - You don't need my ack since I am not the maintainer of the irqdomain
> > code anymore, Grant is :-)
>
> Hrm, I'm being told Grant isn't anymore... I can step in and take it all
> back but you might not like the result ....
Oh but it looks like MAINTAINERS says I am :-)
Ok so from that perspective, I don't like it at all. You can try to
convince me otherwise but I don't think we need to introduce a
dependency to something like msi into the core remapper. It's already to
complex.
Why don't you move the powerpc bitmap allocator over to a generic
place ? I feel like it would be actually simpler but feel free to prove
me wrong.
Ben.
WARNING: multiple messages have this Message-ID (diff)
From: benh@kernel.crashing.org (Benjamin Herrenschmidt)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv7 07/13] irqdomain: add function to find a MSI irq_domain
Date: Thu, 08 Aug 2013 08:45:36 +1000 [thread overview]
Message-ID: <1375915536.12551.9.camel@pasglop> (raw)
In-Reply-To: <1375915344.12551.6.camel@pasglop>
On Thu, 2013-08-08 at 08:42 +1000, Benjamin Herrenschmidt wrote:
> On Thu, 2013-08-08 at 08:31 +1000, Benjamin Herrenschmidt wrote:
> > On Thu, 2013-08-08 at 00:04 +0200, Thomas Petazzoni wrote:
> > > Again, this has been discussed at lengths in the previous iterations,
> > > for which I already gave you all the links, as you requested in a
> > > private e-mail. It'd be great if this discussion was read seriously,
> > > because I really have the feeling we are restarting from zero on this
> > > whole MSI thing...
> >
> > Well, two things here:
> >
> > - You don't need my ack since I am not the maintainer of the irqdomain
> > code anymore, Grant is :-)
>
> Hrm, I'm being told Grant isn't anymore... I can step in and take it all
> back but you might not like the result ....
Oh but it looks like MAINTAINERS says I am :-)
Ok so from that perspective, I don't like it at all. You can try to
convince me otherwise but I don't think we need to introduce a
dependency to something like msi into the core remapper. It's already to
complex.
Why don't you move the powerpc bitmap allocator over to a generic
place ? I feel like it would be actually simpler but feel free to prove
me wrong.
Ben.
next prev parent reply other threads:[~2013-08-07 22:46 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-07 9:32 [PATCHv7 00/13] MSI support for Marvell EBU PCIe driver Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` [PATCHv7 01/13] PCI: use weak functions for MSI arch-specific functions Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` [PATCHv7 02/13] PCI: remove ARCH_SUPPORTS_MSI kconfig option Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` [PATCHv7 03/13] PCI: Introduce new MSI chip infrastructure Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` [PATCHv7 04/13] irqdomain: add irq_alloc_mapping() function Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` [PATCHv7 05/13] irqdomain: refactor __irq_domain_add() Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` [PATCHv7 06/13] irqdomain: add support to associate an irq_domain with a msi_chip Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` [PATCHv7 07/13] irqdomain: add function to find a MSI irq_domain Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 20:50 ` Benjamin Herrenschmidt
2013-08-07 20:50 ` Benjamin Herrenschmidt
2013-08-07 22:04 ` Thomas Petazzoni
2013-08-07 22:04 ` Thomas Petazzoni
2013-08-07 22:31 ` Benjamin Herrenschmidt
2013-08-07 22:31 ` Benjamin Herrenschmidt
2013-08-07 22:42 ` Benjamin Herrenschmidt
2013-08-07 22:42 ` Benjamin Herrenschmidt
2013-08-07 22:45 ` Benjamin Herrenschmidt [this message]
2013-08-07 22:45 ` Benjamin Herrenschmidt
2013-08-08 8:22 ` Thomas Petazzoni
2013-08-08 8:22 ` Thomas Petazzoni
2013-08-08 8:41 ` Benjamin Herrenschmidt
2013-08-08 8:41 ` Benjamin Herrenschmidt
2013-08-08 8:16 ` Thomas Petazzoni
2013-08-08 8:16 ` Thomas Petazzoni
2013-08-08 8:38 ` Benjamin Herrenschmidt
2013-08-08 8:38 ` Benjamin Herrenschmidt
2013-08-08 8:54 ` Benjamin Herrenschmidt
2013-08-08 8:54 ` Benjamin Herrenschmidt
2013-08-07 9:32 ` [PATCHv7 08/13] irqchip: armada-370-xp: properly request resources Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` [PATCHv7 09/13] irqchip: armada-370-xp: implement MSI support Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` [PATCHv7 10/13] ARM: pci: add ->add_bus() and ->remove_bus() hooks to hw_pci Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` [PATCHv7 11/13] ARM: mvebu: the MPIC now provides MSI controller features Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` [PATCHv7 12/13] PCI: mvebu: add support for MSI Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 9:32 ` [PATCHv7 13/13] ARM: mvebu: link PCIe controllers to the MSI controller Thomas Petazzoni
2013-08-07 9:32 ` Thomas Petazzoni
2013-08-07 20:23 ` [PATCHv7 00/13] MSI support for Marvell EBU PCIe driver Jason Cooper
2013-08-07 20:23 ` 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=1375915536.12551.9.camel@pasglop \
--to=benh@kernel.crashing.org \
--cc=alior@marvell.com \
--cc=andrew@lunn.ch \
--cc=bhelgaas@google.com \
--cc=ezequiel.garcia@free-electrons.com \
--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=thierry.reding@gmail.com \
--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.