devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Shawn Lin <shawn.lin-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
To: Lorenzo Pieralisi <lorenzo.pieralisi-5wv7dgnIgG8@public.gmane.org>
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Heiko Stuebner <heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org>,
	linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	shawn.lin-TNX95d0MmH7DzftRWevZcw@public.gmane.org,
	Jeffy Chen <jeffy.chen-TNX95d0MmH7DzftRWevZcw@public.gmane.org>,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Bjorn Helgaas <bhelgaas-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>,
	Simon Xue <xxm-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
Subject: Re: [PATCH v5 0/10] Add endpoint driver for Rockchip PCIe controller
Date: Mon, 23 Apr 2018 09:07:46 +0800	[thread overview]
Message-ID: <834c0cd3-f036-53f6-8447-774e7ee1ea36@rock-chips.com> (raw)
In-Reply-To: <1522630865-151344-1-git-send-email-shawn.lin-TNX95d0MmH7DzftRWevZcw@public.gmane.org>

Hi Lorenzo,

Gentle ping :)

On 2018/4/2 9:01, Shawn Lin wrote:
> Hi Lorenzo,
> 
>      This's the 5th try to upstream endpoint driver for Rockchip PCIe
> controller. In previous discussion, we both agree to do some directory
> reconstruction as soon as possible in 4.17 cycle. So the first 3 patches
> attempt to move forward with that.
> 
> Then patch 4-9 are mostly the same as previous version, but just
> fold in the fixes found by Kbuild Robot.
> 
> And patch 10 will update the defconfig for arm64 to avoid the warning,
> which may go via linux-rockchip tree.
> 
> Hope it looks good to you. :)
> 
> Changes in v5:
> - fix all issues found by Kbuild Robot
> - rename directory from host to controller, and move back dwc and cadence
> 
> Changes in v4:
> - remove empty line
> - keep rockchip's order next to cadence and dwc, but we will
>    try to resort it later when reconstructing the directories
> 
> Changes in v2:
> - remove rockchip,rk3399-pcie-host compatible(Suggested by Rob)
> - fix some error handling
> 
> 
> 
> Shawn Lin (10):
>    PCI: Rename directory from host to controller
>    PCI: cadence: Move cadence drivers back to controller directory
>    PCI: dwc: Move dwc drivers back to controller directory
>    PCI: rockchip: Factor out common code and host code
>    PCI: rockchip: Split out common function to parse DT
>    PCI: rockchip: Split out common function to init controller
>    dt-bindings: PCI: rockchip: Rename rockchip-pcie.txt to
>      rockchip-pcie-host.txt
>    PCI: rockchip: Add Endpoint controller driver for Rockchip PCIe
>      controller
>    dt-bindings: PCI: rockchip: Add DT bindings for Rockchip PCIe endpoint
>      controller
>    arm64: defconfig: update config for Rockchip PCIe
> 
>   .../devicetree/bindings/pci/rockchip-pcie-ep.txt   |  62 ++
>   .../{rockchip-pcie.txt => rockchip-pcie-host.txt}  |   0
>   MAINTAINERS                                        |  65 ++-
>   arch/arm64/configs/defconfig                       |   3 +-
>   drivers/pci/Kconfig                                |   4 +-
>   drivers/pci/Makefile                               |   6 +-
>   drivers/pci/cadence/Kconfig                        |  27 -
>   drivers/pci/cadence/Makefile                       |   4 -
>   drivers/pci/{host => controller}/Kconfig           |  50 +-
>   drivers/pci/{host => controller}/Makefile          |   7 +
>   drivers/pci/{ => controller}/dwc/Kconfig           |   0
>   drivers/pci/{ => controller}/dwc/Makefile          |   0
>   drivers/pci/{ => controller}/dwc/pci-dra7xx.c      |   0
>   drivers/pci/{ => controller}/dwc/pci-exynos.c      |   0
>   drivers/pci/{ => controller}/dwc/pci-imx6.c        |   0
>   drivers/pci/{ => controller}/dwc/pci-keystone-dw.c |   0
>   drivers/pci/{ => controller}/dwc/pci-keystone.c    |   0
>   drivers/pci/{ => controller}/dwc/pci-keystone.h    |   0
>   drivers/pci/{ => controller}/dwc/pci-layerscape.c  |   0
>   drivers/pci/{ => controller}/dwc/pcie-armada8k.c   |   0
>   drivers/pci/{ => controller}/dwc/pcie-artpec6.c    |   0
>   .../pci/{ => controller}/dwc/pcie-designware-ep.c  |   0
>   .../{ => controller}/dwc/pcie-designware-host.c    |   0
>   .../{ => controller}/dwc/pcie-designware-plat.c    |   0
>   drivers/pci/{ => controller}/dwc/pcie-designware.c |   0
>   drivers/pci/{ => controller}/dwc/pcie-designware.h |   0
>   drivers/pci/{ => controller}/dwc/pcie-hisi.c       |   2 +-
>   drivers/pci/{ => controller}/dwc/pcie-histb.c      |   0
>   drivers/pci/{ => controller}/dwc/pcie-kirin.c      |   0
>   drivers/pci/{ => controller}/dwc/pcie-qcom.c       |   0
>   drivers/pci/{ => controller}/dwc/pcie-spear13xx.c  |   0
>   drivers/pci/{host => controller}/pci-aardvark.c    |   0
>   drivers/pci/{host => controller}/pci-ftpci100.c    |   0
>   drivers/pci/{host => controller}/pci-host-common.c |   0
>   .../pci/{host => controller}/pci-host-generic.c    |   0
>   drivers/pci/{host => controller}/pci-hyperv.c      |   0
>   drivers/pci/{host => controller}/pci-mvebu.c       |   0
>   drivers/pci/{host => controller}/pci-rcar-gen2.c   |   0
>   drivers/pci/{host => controller}/pci-tegra.c       |   0
>   .../pci/{host => controller}/pci-thunder-ecam.c    |   0
>   drivers/pci/{host => controller}/pci-thunder-pem.c |   0
>   drivers/pci/{host => controller}/pci-v3-semi.c     |   0
>   drivers/pci/{host => controller}/pci-versatile.c   |   0
>   drivers/pci/{host => controller}/pci-xgene-msi.c   |   0
>   drivers/pci/{host => controller}/pci-xgene.c       |   0
>   drivers/pci/{host => controller}/pcie-altera-msi.c |   0
>   drivers/pci/{host => controller}/pcie-altera.c     |   0
>   .../pci/{cadence => controller}/pcie-cadence-ep.c  |   0
>   .../{cadence => controller}/pcie-cadence-host.c    |   0
>   drivers/pci/{cadence => controller}/pcie-cadence.c |   0
>   drivers/pci/{cadence => controller}/pcie-cadence.h |   0
>   drivers/pci/{host => controller}/pcie-iproc-bcma.c |   0
>   drivers/pci/{host => controller}/pcie-iproc-msi.c  |   0
>   .../pci/{host => controller}/pcie-iproc-platform.c |   0
>   drivers/pci/{host => controller}/pcie-iproc.c      |   0
>   drivers/pci/{host => controller}/pcie-iproc.h      |   0
>   drivers/pci/{host => controller}/pcie-mediatek.c   |   0
>   drivers/pci/{host => controller}/pcie-rcar.c       |   0
>   drivers/pci/controller/pcie-rockchip-ep.c          | 639 +++++++++++++++++++++
>   .../pcie-rockchip-host.c}                          | 600 +------------------
>   drivers/pci/controller/pcie-rockchip.c             | 423 ++++++++++++++
>   drivers/pci/controller/pcie-rockchip.h             | 338 +++++++++++
>   drivers/pci/{host => controller}/pcie-tango.c      |   0
>   drivers/pci/{host => controller}/pcie-xilinx-nwl.c |   0
>   drivers/pci/{host => controller}/pcie-xilinx.c     |   0
>   drivers/pci/{host => controller}/vmd.c             |   0
>   66 files changed, 1571 insertions(+), 659 deletions(-)
>   create mode 100644 Documentation/devicetree/bindings/pci/rockchip-pcie-ep.txt
>   rename Documentation/devicetree/bindings/pci/{rockchip-pcie.txt => rockchip-pcie-host.txt} (100%)
>   delete mode 100644 drivers/pci/cadence/Kconfig
>   delete mode 100644 drivers/pci/cadence/Makefile
>   rename drivers/pci/{host => controller}/Kconfig (84%)
>   rename drivers/pci/{host => controller}/Makefile (84%)
>   rename drivers/pci/{ => controller}/dwc/Kconfig (100%)
>   rename drivers/pci/{ => controller}/dwc/Makefile (100%)
>   rename drivers/pci/{ => controller}/dwc/pci-dra7xx.c (100%)
>   rename drivers/pci/{ => controller}/dwc/pci-exynos.c (100%)
>   rename drivers/pci/{ => controller}/dwc/pci-imx6.c (100%)
>   rename drivers/pci/{ => controller}/dwc/pci-keystone-dw.c (100%)
>   rename drivers/pci/{ => controller}/dwc/pci-keystone.c (100%)
>   rename drivers/pci/{ => controller}/dwc/pci-keystone.h (100%)
>   rename drivers/pci/{ => controller}/dwc/pci-layerscape.c (100%)
>   rename drivers/pci/{ => controller}/dwc/pcie-armada8k.c (100%)
>   rename drivers/pci/{ => controller}/dwc/pcie-artpec6.c (100%)
>   rename drivers/pci/{ => controller}/dwc/pcie-designware-ep.c (100%)
>   rename drivers/pci/{ => controller}/dwc/pcie-designware-host.c (100%)
>   rename drivers/pci/{ => controller}/dwc/pcie-designware-plat.c (100%)
>   rename drivers/pci/{ => controller}/dwc/pcie-designware.c (100%)
>   rename drivers/pci/{ => controller}/dwc/pcie-designware.h (100%)
>   rename drivers/pci/{ => controller}/dwc/pcie-hisi.c (99%)
>   rename drivers/pci/{ => controller}/dwc/pcie-histb.c (100%)
>   rename drivers/pci/{ => controller}/dwc/pcie-kirin.c (100%)
>   rename drivers/pci/{ => controller}/dwc/pcie-qcom.c (100%)
>   rename drivers/pci/{ => controller}/dwc/pcie-spear13xx.c (100%)
>   rename drivers/pci/{host => controller}/pci-aardvark.c (100%)
>   rename drivers/pci/{host => controller}/pci-ftpci100.c (100%)
>   rename drivers/pci/{host => controller}/pci-host-common.c (100%)
>   rename drivers/pci/{host => controller}/pci-host-generic.c (100%)
>   rename drivers/pci/{host => controller}/pci-hyperv.c (100%)
>   rename drivers/pci/{host => controller}/pci-mvebu.c (100%)
>   rename drivers/pci/{host => controller}/pci-rcar-gen2.c (100%)
>   rename drivers/pci/{host => controller}/pci-tegra.c (100%)
>   rename drivers/pci/{host => controller}/pci-thunder-ecam.c (100%)
>   rename drivers/pci/{host => controller}/pci-thunder-pem.c (100%)
>   rename drivers/pci/{host => controller}/pci-v3-semi.c (100%)
>   rename drivers/pci/{host => controller}/pci-versatile.c (100%)
>   rename drivers/pci/{host => controller}/pci-xgene-msi.c (100%)
>   rename drivers/pci/{host => controller}/pci-xgene.c (100%)
>   rename drivers/pci/{host => controller}/pcie-altera-msi.c (100%)
>   rename drivers/pci/{host => controller}/pcie-altera.c (100%)
>   rename drivers/pci/{cadence => controller}/pcie-cadence-ep.c (100%)
>   rename drivers/pci/{cadence => controller}/pcie-cadence-host.c (100%)
>   rename drivers/pci/{cadence => controller}/pcie-cadence.c (100%)
>   rename drivers/pci/{cadence => controller}/pcie-cadence.h (100%)
>   rename drivers/pci/{host => controller}/pcie-iproc-bcma.c (100%)
>   rename drivers/pci/{host => controller}/pcie-iproc-msi.c (100%)
>   rename drivers/pci/{host => controller}/pcie-iproc-platform.c (100%)
>   rename drivers/pci/{host => controller}/pcie-iproc.c (100%)
>   rename drivers/pci/{host => controller}/pcie-iproc.h (100%)
>   rename drivers/pci/{host => controller}/pcie-mediatek.c (100%)
>   rename drivers/pci/{host => controller}/pcie-rcar.c (100%)
>   create mode 100644 drivers/pci/controller/pcie-rockchip-ep.c
>   rename drivers/pci/{host/pcie-rockchip.c => controller/pcie-rockchip-host.c} (61%)
>   create mode 100644 drivers/pci/controller/pcie-rockchip.c
>   create mode 100644 drivers/pci/controller/pcie-rockchip.h
>   rename drivers/pci/{host => controller}/pcie-tango.c (100%)
>   rename drivers/pci/{host => controller}/pcie-xilinx-nwl.c (100%)
>   rename drivers/pci/{host => controller}/pcie-xilinx.c (100%)
>   rename drivers/pci/{host => controller}/vmd.c (100%)
> 

      parent reply	other threads:[~2018-04-23  1:07 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-02  1:01 [PATCH v5 0/10] Add endpoint driver for Rockchip PCIe controller Shawn Lin
2018-04-02  1:04 ` [PATCH v5 10/10] arm64: defconfig: update config for Rockchip PCIe Shawn Lin
     [not found] ` <1522630865-151344-1-git-send-email-shawn.lin-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
2018-04-02  1:01   ` [PATCH v5 01/10] PCI: Rename directory from host to controller Shawn Lin
     [not found]     ` <1522630910-151391-1-git-send-email-shawn.lin-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
2018-04-23 14:37       ` Lorenzo Pieralisi
     [not found]         ` <20180423143727.GA17979-4tUPXFaYRHv6sAKXYmQ0tx/iLCjYCKR+VpNB7YpNyf8@public.gmane.org>
2018-04-25 10:49           ` Lorenzo Pieralisi
     [not found]             ` <20180425104917.GB15875-4tUPXFaYRHv6sAKXYmQ0tx/iLCjYCKR+VpNB7YpNyf8@public.gmane.org>
2018-04-26  0:48               ` Shawn Lin
     [not found]                 ` <ac1b92e0-7cfc-6cad-4113-cfd863ede572-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
2018-04-26 17:05                   ` Lorenzo Pieralisi
     [not found]                     ` <20180426170515.GA8928-4tUPXFaYRHv6sAKXYmQ0tx/iLCjYCKR+VpNB7YpNyf8@public.gmane.org>
2018-04-27  2:50                       ` Shawn Lin
2018-04-02  1:02   ` [PATCH v5 02/10] PCI: cadence: Move cadence drivers back to controller directory Shawn Lin
2018-04-02  1:02   ` [PATCH v5 03/10] PCI: dwc: Move dwc " Shawn Lin
2018-04-02  1:02   ` [PATCH v5 04/10] PCI: rockchip: Factor out common code and host code Shawn Lin
2018-04-02  1:03   ` [PATCH v5 05/10] PCI: rockchip: Split out common function to parse DT Shawn Lin
2018-04-02  1:03   ` [PATCH v5 06/10] PCI: rockchip: Split out common function to init controller Shawn Lin
2018-04-02  1:03   ` [PATCH v5 07/10] dt-bindings: PCI: rockchip: Rename rockchip-pcie.txt to rockchip-pcie-host.txt Shawn Lin
2018-04-02  1:04   ` [PATCH v5 08/10] PCI: rockchip: Add Endpoint controller driver for Rockchip PCIe controller Shawn Lin
2018-04-02  1:04   ` [PATCH v5 09/10] dt-bindings: PCI: rockchip: Add DT bindings for Rockchip PCIe endpoint controller Shawn Lin
2018-04-23  1:07   ` Shawn Lin [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=834c0cd3-f036-53f6-8447-774e7ee1ea36@rock-chips.com \
    --to=shawn.lin-tnx95d0mmh7dzftrwevzcw@public.gmane.org \
    --cc=bhelgaas-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org \
    --cc=jeffy.chen-TNX95d0MmH7DzftRWevZcw@public.gmane.org \
    --cc=linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=lorenzo.pieralisi-5wv7dgnIgG8@public.gmane.org \
    --cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=xxm-TNX95d0MmH7DzftRWevZcw@public.gmane.org \
    /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).