From: Marek Vasut <marex@denx.de>
To: linux-arm-kernel@lists.infradead.org
Cc: Bjorn Helgaas <bhelgaas@google.com>,
Shawn Guo <shawn.guo@linaro.org>,
"Zhu Richard-R65037" <r65037@freescale.com>,
"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
tharvey@gateworks.com, Frank Li <lznuaa@gmail.com>,
Sean Cross <xobs@kosagi.com>,
Sascha Hauer <s.hauer@pengutronix.de>
Subject: Re: [PATCH v7 0/2] Add PCIe support for i.MX6q
Date: Tue, 8 Oct 2013 22:56:28 +0200 [thread overview]
Message-ID: <201310082256.28474.marex@denx.de> (raw)
In-Reply-To: <CAErSpo4JhjW5a-74eJp5bajmAMKSn4Vxknk8NoVdD462N=Ncug@mail.gmail.com>
Hi,
> On Wed, Sep 25, 2013 at 9:24 PM, Shawn Guo <shawn.guo@linaro.org> wrote:
> > Hi Bjorn,
> >
> > This is basically a resend of imx6 pcie driver patch for inclusion into
> > PCI tree. Since we expect you apply the patch with your SoB, I dropped
> > your ACK tag there.
> >
> > I have to ask you to carry the first patch which adds some macros to be
> > used by pcie driver patch. Since imx6q-iomuxc-gpr.h gets rare updates
> > recently, it should be less likely we will run into merge conflict on it.
> >
> > Changes since v6:
> >
> > * Drop arch bits from driver patch, and will handle arch bits through
> >
> > arm-soc separately.
> >
> > * Use devm_clk_get() instead of clk_get().
> > * Drop unnecessary (val != exp_val) check in pcie_phy_poll_ack() loop.
> > * Rename function imx6_init() to imx6_pcie_init().
> >
> > Shawn
> >
> > Sean Cross (2):
> > ARM: imx6q: Add PCIe bits to GPR syscon definition
> > PCI: imx6: Add support for i.MX6 PCIe controller
> >
> > .../devicetree/bindings/pci/designware-pcie.txt | 7 +-
> > drivers/pci/host/Kconfig | 6 +
> > drivers/pci/host/Makefile | 1 +
> > drivers/pci/host/pci-imx6.c | 576
> > ++++++++++++++++++++ include/linux/mfd/syscon/imx6q-iomuxc-gpr.h
> > | 8 +
> > 5 files changed, 597 insertions(+), 1 deletion(-)
> > create mode 100644 drivers/pci/host/pci-imx6.c
>
> I merged both of these to my pci/host-imx6 branch for v3.13.
>
> I made a couple minor spelling and whitespace corrections.
I have MX6 hardware with a PCIe switch, so I can test that stuff. Thus far, of
course, the switch doesn't work. Is getting the switch to work only a matter of
programming the RdWr0 vs. RdWr1 mode into the iATU or is there something more ?
Thanks!
Best regards,
Marek Vasut
WARNING: multiple messages have this Message-ID (diff)
From: marex@denx.de (Marek Vasut)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v7 0/2] Add PCIe support for i.MX6q
Date: Tue, 8 Oct 2013 22:56:28 +0200 [thread overview]
Message-ID: <201310082256.28474.marex@denx.de> (raw)
In-Reply-To: <CAErSpo4JhjW5a-74eJp5bajmAMKSn4Vxknk8NoVdD462N=Ncug@mail.gmail.com>
Hi,
> On Wed, Sep 25, 2013 at 9:24 PM, Shawn Guo <shawn.guo@linaro.org> wrote:
> > Hi Bjorn,
> >
> > This is basically a resend of imx6 pcie driver patch for inclusion into
> > PCI tree. Since we expect you apply the patch with your SoB, I dropped
> > your ACK tag there.
> >
> > I have to ask you to carry the first patch which adds some macros to be
> > used by pcie driver patch. Since imx6q-iomuxc-gpr.h gets rare updates
> > recently, it should be less likely we will run into merge conflict on it.
> >
> > Changes since v6:
> >
> > * Drop arch bits from driver patch, and will handle arch bits through
> >
> > arm-soc separately.
> >
> > * Use devm_clk_get() instead of clk_get().
> > * Drop unnecessary (val != exp_val) check in pcie_phy_poll_ack() loop.
> > * Rename function imx6_init() to imx6_pcie_init().
> >
> > Shawn
> >
> > Sean Cross (2):
> > ARM: imx6q: Add PCIe bits to GPR syscon definition
> > PCI: imx6: Add support for i.MX6 PCIe controller
> >
> > .../devicetree/bindings/pci/designware-pcie.txt | 7 +-
> > drivers/pci/host/Kconfig | 6 +
> > drivers/pci/host/Makefile | 1 +
> > drivers/pci/host/pci-imx6.c | 576
> > ++++++++++++++++++++ include/linux/mfd/syscon/imx6q-iomuxc-gpr.h
> > | 8 +
> > 5 files changed, 597 insertions(+), 1 deletion(-)
> > create mode 100644 drivers/pci/host/pci-imx6.c
>
> I merged both of these to my pci/host-imx6 branch for v3.13.
>
> I made a couple minor spelling and whitespace corrections.
I have MX6 hardware with a PCIe switch, so I can test that stuff. Thus far, of
course, the switch doesn't work. Is getting the switch to work only a matter of
programming the RdWr0 vs. RdWr1 mode into the iATU or is there something more ?
Thanks!
Best regards,
Marek Vasut
next prev parent reply other threads:[~2013-10-08 20:56 UTC|newest]
Thread overview: 80+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-26 3:24 [PATCH v7 0/2] Add PCIe support for i.MX6q Shawn Guo
2013-09-26 3:24 ` Shawn Guo
2013-09-26 3:24 ` [PATCH v7 1/2] ARM: imx6q: Add PCIe bits to GPR syscon definition Shawn Guo
2013-09-26 3:24 ` Shawn Guo
2013-09-26 3:24 ` [PATCH v7 2/2] PCI: imx6: Add support for i.MX6 PCIe controller Shawn Guo
2013-09-26 3:24 ` Shawn Guo
2013-09-27 19:24 ` [PATCH v7 0/2] Add PCIe support for i.MX6q Bjorn Helgaas
2013-09-27 19:24 ` Bjorn Helgaas
2013-09-28 6:56 ` Shawn Guo
2013-09-28 6:56 ` Shawn Guo
2013-10-08 20:56 ` Marek Vasut [this message]
2013-10-08 20:56 ` Marek Vasut
2013-10-09 5:23 ` Zhu Richard-R65037
2013-10-09 5:23 ` Zhu Richard-R65037
2013-10-10 10:25 ` Marek Vasut
2013-10-10 10:25 ` Marek Vasut
2013-10-10 10:40 ` Zhu Richard-R65037
2013-10-10 10:40 ` Zhu Richard-R65037
2013-10-10 12:59 ` Marek Vasut
2013-10-10 12:59 ` Marek Vasut
2013-10-10 20:33 ` Tim Harvey
2013-10-10 20:33 ` Tim Harvey
2013-10-10 20:40 ` Marek Vasut
2013-10-10 20:40 ` Marek Vasut
2013-10-10 13:27 ` Bjorn Helgaas
2013-10-10 13:27 ` Bjorn Helgaas
2013-10-10 13:43 ` Marek Vasut
2013-10-10 13:43 ` Marek Vasut
2013-10-10 15:58 ` Marek Vasut
2013-10-10 15:58 ` Marek Vasut
2013-10-10 17:17 ` Bjorn Helgaas
2013-10-10 17:17 ` Bjorn Helgaas
2013-10-10 17:39 ` Marek Vasut
2013-10-10 17:39 ` Marek Vasut
2013-10-10 17:56 ` Bjorn Helgaas
2013-10-10 17:56 ` Bjorn Helgaas
2013-10-11 2:12 ` [PATCH 1/2] PCI: imx6: Make reset-gpio optional Marek Vasut
2013-10-11 2:12 ` Marek Vasut
2013-10-11 2:12 ` [PATCH 2/2] PCI: imx6: Fix the clock for PCIe Marek Vasut
2013-10-11 2:12 ` Marek Vasut
2013-10-11 7:20 ` Jingoo Han
2013-10-11 7:20 ` Jingoo Han
2013-10-11 11:55 ` Marek Vasut
2013-10-11 11:55 ` Marek Vasut
2013-10-12 7:13 ` Shawn Guo
2013-10-12 7:13 ` Shawn Guo
2013-10-11 7:09 ` [PATCH 1/2] PCI: imx6: Make reset-gpio optional Jingoo Han
2013-10-11 7:09 ` Jingoo Han
2013-10-12 7:20 ` Shawn Guo
2013-10-12 7:20 ` Shawn Guo
2013-10-12 9:28 ` Marek Vasut
2013-10-12 9:28 ` Marek Vasut
2013-10-14 0:02 ` Jingoo Han
2013-10-14 0:02 ` Jingoo Han
2013-10-14 0:44 ` Marek Vasut
2013-10-14 0:44 ` Marek Vasut
2013-10-14 1:17 ` Marek Vasut
2013-10-14 1:17 ` Marek Vasut
2013-10-14 2:33 ` Jingoo Han
2013-10-14 2:33 ` Jingoo Han
2013-10-14 3:23 ` Marek Vasut
2013-10-14 3:23 ` Marek Vasut
2013-10-11 2:13 ` [PATCH v7 0/2] Add PCIe support for i.MX6q Marek Vasut
2013-10-11 2:13 ` Marek Vasut
2013-10-11 2:18 ` Marek Vasut
2013-10-11 2:18 ` Marek Vasut
2013-10-11 2:29 ` Zhu Richard-R65037
2013-10-11 2:29 ` Zhu Richard-R65037
2013-10-11 4:44 ` Yinghai Lu
2013-10-11 4:44 ` Yinghai Lu
2013-10-11 14:44 ` Marek Vasut
2013-10-11 14:44 ` Marek Vasut
2013-10-11 15:24 ` Tim Harvey
2013-10-11 15:24 ` Tim Harvey
2013-10-11 20:13 ` Marek Vasut
2013-10-11 20:13 ` Marek Vasut
2013-10-12 2:16 ` Zhu Richard-R65037
2013-10-12 2:16 ` Zhu Richard-R65037
2013-10-12 2:30 ` Marek Vasut
2013-10-12 2:30 ` Marek Vasut
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=201310082256.28474.marex@denx.de \
--to=marex@denx.de \
--cc=bhelgaas@google.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-pci@vger.kernel.org \
--cc=lznuaa@gmail.com \
--cc=r65037@freescale.com \
--cc=s.hauer@pengutronix.de \
--cc=shawn.guo@linaro.org \
--cc=tharvey@gateworks.com \
--cc=xobs@kosagi.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.