All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rob Herring <robh+dt@kernel.org>
To: "Pali Rohár" <pali@kernel.org>
Cc: "Lorenzo Pieralisi" <lorenzo.pieralisi@arm.com>,
	"Bjorn Helgaas" <bhelgaas@google.com>,
	"Thomas Petazzoni" <thomas.petazzoni@bootlin.com>,
	"Krzysztof Wilczyński" <kw@linux.com>,
	"Marek Behún" <kabel@kernel.org>,
	"Russell King" <rmk+kernel@armlinux.org.uk>,
	PCI <linux-pci@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	linux-arm-kernel <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 08/11] PCI: mvebu: Use child_ops API
Date: Wed, 12 Jan 2022 08:53:34 -0600	[thread overview]
Message-ID: <CAL_JsqLaB=WnZjD3UBSyLHdWC4N3V8rFS47u57NZmmB1o0j1XA@mail.gmail.com> (raw)
In-Reply-To: <20220112014356.h2pm7nc3vzsvjexq@pali>

On Tue, Jan 11, 2022 at 7:44 PM Pali Rohár <pali@kernel.org> wrote:
>
> On Wednesday 05 January 2022 09:41:51 Rob Herring wrote:
> > On Wed, Jan 5, 2022 at 9:03 AM Pali Rohár <pali@kernel.org> wrote:
> > >
> > > Split struct pci_ops between ops and child_ops. Member ops is used for
> > > accessing PCIe Root Ports via pci-bridge-emul.c driver and child_ops for
> > > accessing real PCIe cards.
> > >
> > > There is no need to mix these two struct pci_ops into one as PCI core code
> > > already provides separate callbacks via bridge->ops and bridge->child_ops.
> >
> > I had similar patches including mvebu that I never got around to
> > sending out. I pushed the branch out now at least[1].
>
> Are you going to finish your patch series and send it? Because if yes,
> I can drop this my patch in v3 and let all ->child_ops conversion for
> you.

Not any time soon.

Rob

WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh+dt@kernel.org>
To: "Pali Rohár" <pali@kernel.org>
Cc: "Lorenzo Pieralisi" <lorenzo.pieralisi@arm.com>,
	"Bjorn Helgaas" <bhelgaas@google.com>,
	"Thomas Petazzoni" <thomas.petazzoni@bootlin.com>,
	"Krzysztof Wilczyński" <kw@linux.com>,
	"Marek Behún" <kabel@kernel.org>,
	"Russell King" <rmk+kernel@armlinux.org.uk>,
	PCI <linux-pci@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	linux-arm-kernel <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 08/11] PCI: mvebu: Use child_ops API
Date: Wed, 12 Jan 2022 08:53:34 -0600	[thread overview]
Message-ID: <CAL_JsqLaB=WnZjD3UBSyLHdWC4N3V8rFS47u57NZmmB1o0j1XA@mail.gmail.com> (raw)
In-Reply-To: <20220112014356.h2pm7nc3vzsvjexq@pali>

On Tue, Jan 11, 2022 at 7:44 PM Pali Rohár <pali@kernel.org> wrote:
>
> On Wednesday 05 January 2022 09:41:51 Rob Herring wrote:
> > On Wed, Jan 5, 2022 at 9:03 AM Pali Rohár <pali@kernel.org> wrote:
> > >
> > > Split struct pci_ops between ops and child_ops. Member ops is used for
> > > accessing PCIe Root Ports via pci-bridge-emul.c driver and child_ops for
> > > accessing real PCIe cards.
> > >
> > > There is no need to mix these two struct pci_ops into one as PCI core code
> > > already provides separate callbacks via bridge->ops and bridge->child_ops.
> >
> > I had similar patches including mvebu that I never got around to
> > sending out. I pushed the branch out now at least[1].
>
> Are you going to finish your patch series and send it? Because if yes,
> I can drop this my patch in v3 and let all ->child_ops conversion for
> you.

Not any time soon.

Rob

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2022-01-12 14:53 UTC|newest]

Thread overview: 130+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-05 15:02 [PATCH 00/11] PCI: mvebu: subsystem ids, AER and INTx Pali Rohár
2022-01-05 15:02 ` Pali Rohár
2022-01-05 15:02 ` [PATCH 01/11] PCI: pci-bridge-emul: Re-arrange register tests Pali Rohár
2022-01-05 15:02   ` Pali Rohár
2022-01-05 15:02 ` [PATCH 02/11] PCI: pci-bridge-emul: Add support for PCIe extended capabilities Pali Rohár
2022-01-05 15:02   ` Pali Rohár
2022-01-05 15:02 ` [PATCH 03/11] PCI: pci-bridge-emul: Add support for PCI Bridge Subsystem Vendor ID capability Pali Rohár
2022-01-05 15:02   ` Pali Rohár
2022-01-05 15:02 ` [PATCH 04/11] dt-bindings: PCI: mvebu: Add num-lanes property Pali Rohár
2022-01-05 15:02   ` Pali Rohár
2022-01-12  1:29   ` Rob Herring
2022-01-12  1:29     ` Rob Herring
2022-01-05 15:02 ` [PATCH 05/11] PCI: mvebu: Correctly configure x1/x4 mode Pali Rohár
2022-01-05 15:02   ` Pali Rohár
2022-01-05 15:02 ` [PATCH 06/11] PCI: mvebu: Add support for PCI Bridge Subsystem Vendor ID on emulated bridge Pali Rohár
2022-01-05 15:02   ` Pali Rohár
2022-01-05 15:02 ` [PATCH 07/11] PCI: mvebu: Add support for Advanced Error Reporting registers " Pali Rohár
2022-01-05 15:02   ` Pali Rohár
2022-01-05 15:02 ` [PATCH 08/11] PCI: mvebu: Use child_ops API Pali Rohár
2022-01-05 15:02   ` Pali Rohár
2022-01-05 15:41   ` Rob Herring
2022-01-05 15:41     ` Rob Herring
2022-01-05 15:49     ` Pali Rohár
2022-01-05 15:49       ` Pali Rohár
2022-01-12  1:43     ` Pali Rohár
2022-01-12  1:43       ` Pali Rohár
2022-01-12 14:53       ` Rob Herring [this message]
2022-01-12 14:53         ` Rob Herring
2022-01-05 15:02 ` [PATCH 09/11] dt-bindings: PCI: mvebu: Update information about intx interrupts Pali Rohár
2022-01-05 15:02   ` Pali Rohár
2022-01-12  1:30   ` Rob Herring
2022-01-12  1:30     ` Rob Herring
2022-01-05 15:02 ` [PATCH 10/11] PCI: mvebu: Implement support for legacy INTx interrupts Pali Rohár
2022-01-05 15:02   ` Pali Rohár
2022-01-06 15:28   ` Marc Zyngier
2022-01-06 15:28     ` Marc Zyngier
2022-01-06 15:44     ` Pali Rohár
2022-01-06 15:44       ` Pali Rohár
2022-01-06 15:55       ` Marc Zyngier
2022-01-06 15:55         ` Marc Zyngier
2022-01-06 16:20         ` Pali Rohár
2022-01-06 16:20           ` Pali Rohár
2022-01-06 16:27           ` Marc Zyngier
2022-01-06 16:27             ` Marc Zyngier
2022-01-06 17:20             ` Marek Behún
2022-01-06 17:20               ` Marek Behún
2022-01-06 17:31               ` Marc Zyngier
2022-01-06 17:31                 ` Marc Zyngier
2022-01-07 11:50                 ` Pali Rohár
2022-01-07 11:50                   ` Pali Rohár
2022-01-07 18:53                   ` Marc Zyngier
2022-01-07 18:53                     ` Marc Zyngier
2022-01-05 15:02 ` [PATCH 11/11] ARM: dts: armada-385.dtsi: Add definitions for PCIe " Pali Rohár
2022-01-05 15:02   ` Pali Rohár
2022-01-12 15:18 ` [PATCH v2 00/11] PCI: mvebu: subsystem ids, AER and INTx Pali Rohár
2022-01-12 15:18   ` Pali Rohár
2022-01-12 15:18   ` [PATCH v2 01/11] PCI: pci-bridge-emul: Re-arrange register tests Pali Rohár
2022-01-12 15:18     ` Pali Rohár
2022-01-12 15:18   ` [PATCH v2 02/11] PCI: pci-bridge-emul: Add support for PCIe extended capabilities Pali Rohár
2022-01-12 15:18     ` Pali Rohár
2022-01-12 15:18   ` [PATCH v2 03/11] PCI: pci-bridge-emul: Add support for PCI Bridge Subsystem Vendor ID capability Pali Rohár
2022-01-12 15:18     ` Pali Rohár
2022-01-12 15:18   ` [PATCH v2 04/11] dt-bindings: PCI: mvebu: Add num-lanes property Pali Rohár
2022-01-12 15:18     ` Pali Rohár
2022-01-12 15:18   ` [PATCH v2 05/11] PCI: mvebu: Correctly configure x1/x4 mode Pali Rohár
2022-01-12 15:18     ` Pali Rohár
2022-01-20 17:09     ` Rob Herring
2022-01-20 17:09       ` Rob Herring
2022-01-20 17:19       ` Pali Rohár
2022-01-20 17:19         ` Pali Rohár
2022-01-12 15:18   ` [PATCH v2 06/11] PCI: mvebu: Add support for PCI Bridge Subsystem Vendor ID on emulated bridge Pali Rohár
2022-01-12 15:18     ` Pali Rohár
2022-01-12 15:18   ` [PATCH v2 07/11] PCI: mvebu: Add support for Advanced Error Reporting registers " Pali Rohár
2022-01-12 15:18     ` Pali Rohár
2022-01-12 15:18   ` [PATCH v2 08/11] PCI: mvebu: Use child_ops API Pali Rohár
2022-01-12 15:18     ` Pali Rohár
2022-01-20 16:49     ` Rob Herring
2022-01-20 16:49       ` Rob Herring
2022-01-20 16:55       ` Pali Rohár
2022-01-20 16:55         ` Pali Rohár
2022-01-20 18:40         ` Rob Herring
2022-01-20 18:40           ` Rob Herring
2022-01-12 15:18   ` [PATCH v2 09/11] dt-bindings: PCI: mvebu: Update information about intx interrupts Pali Rohár
2022-01-12 15:18     ` Pali Rohár
2022-01-12 15:36     ` Marek Behún
2022-01-12 15:36       ` Marek Behún
2022-01-12 15:18   ` [PATCH v2 10/11] PCI: mvebu: Implement support for legacy INTx interrupts Pali Rohár
2022-01-12 15:18     ` Pali Rohár
2022-02-11 17:19     ` Lorenzo Pieralisi
2022-02-11 17:19       ` Lorenzo Pieralisi
2022-02-11 17:52       ` Pali Rohár
2022-02-11 17:52         ` Pali Rohár
2022-02-11 18:21         ` Lorenzo Pieralisi
2022-02-11 18:21           ` Lorenzo Pieralisi
2022-02-12 10:59           ` Marc Zyngier
2022-02-12 10:59             ` Marc Zyngier
2022-02-16 23:40         ` Pali Rohár
2022-02-16 23:40           ` Pali Rohár
2022-02-22 10:21           ` Lorenzo Pieralisi
2022-02-22 10:21             ` Lorenzo Pieralisi
2022-02-22 10:51             ` Pali Rohár
2022-02-22 10:51               ` Pali Rohár
2022-02-22 15:24               ` Lorenzo Pieralisi
2022-02-22 15:24                 ` Lorenzo Pieralisi
2022-02-22 15:42                 ` Pali Rohár
2022-02-22 15:42                   ` Pali Rohár
2022-02-22 15:45                   ` Lorenzo Pieralisi
2022-02-22 15:45                     ` Lorenzo Pieralisi
2022-02-22 15:55                     ` Pali Rohár
2022-02-22 15:55                       ` Pali Rohár
2022-01-12 15:18   ` [PATCH v2 11/11] ARM: dts: armada-385.dtsi: Add definitions for PCIe " Pali Rohár
2022-01-12 15:18     ` Pali Rohár
2022-02-14 15:07     ` Gregory CLEMENT
2022-02-14 15:07       ` Gregory CLEMENT
2022-02-14 15:09       ` Pali Rohár
2022-02-14 15:09         ` Pali Rohár
2022-02-14 15:26         ` Gregory CLEMENT
2022-02-14 15:26           ` Gregory CLEMENT
     [not found]           ` <CAEzXK1qYKVk7QiSY_DwqkZ7WV6WU06WBtiqZx0JJCc+mOP-7Kg@mail.gmail.com>
2022-02-15 10:48             ` Luís Mendes
2022-02-15 10:48               ` Luís Mendes
2022-02-15 10:52               ` Pali Rohár
2022-02-15 10:52                 ` Pali Rohár
2022-02-18 21:53                 ` Luís Mendes
2022-02-18 21:53                   ` Luís Mendes
2022-02-19 13:36                   ` Pali Rohár
2022-02-19 13:36                     ` Pali Rohár
2022-02-11 17:50 ` [PATCH 00/11] PCI: mvebu: subsystem ids, AER and INTx Lorenzo Pieralisi
2022-02-11 17:50   ` Lorenzo Pieralisi
2022-02-11 18:01   ` Pali Rohár
2022-02-11 18:01     ` Pali Rohár

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='CAL_JsqLaB=WnZjD3UBSyLHdWC4N3V8rFS47u57NZmmB1o0j1XA@mail.gmail.com' \
    --to=robh+dt@kernel.org \
    --cc=bhelgaas@google.com \
    --cc=kabel@kernel.org \
    --cc=kw@linux.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=lorenzo.pieralisi@arm.com \
    --cc=pali@kernel.org \
    --cc=rmk+kernel@armlinux.org.uk \
    --cc=thomas.petazzoni@bootlin.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.