From: Rob Herring <robh+dt@kernel.org>
To: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp>
Cc: "Bjorn Helgaas" <bhelgaas@google.com>,
"Lorenzo Pieralisi" <lorenzo.pieralisi@arm.com>,
PCI <linux-pci@vger.kernel.org>,
"Krzysztof Wilczyński" <kw@linux.com>,
"Kishon Vijay Abraham I" <kishon@ti.com>,
devicetree@vger.kernel.org,
"Punit Agrawal" <punit1.agrawal@toshiba.co.jp>,
yuji2.ishikawa@toshiba.co.jp,
linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v6 2/3] PCI: visconti: Add Toshiba Visconti PCIe host controller driver
Date: Thu, 26 Aug 2021 07:11:21 -0500 [thread overview]
Message-ID: <CAL_Jsq+tGAOo-9oKGF=B=3hCjiECBwxDujQu9drwCXK6u8J0BA@mail.gmail.com> (raw)
In-Reply-To: <20210811083830.784065-3-nobuhiro1.iwamatsu@toshiba.co.jp>
On Wed, Aug 11, 2021 at 3:38 AM Nobuhiro Iwamatsu
<nobuhiro1.iwamatsu@toshiba.co.jp> wrote:
>
> Add support to PCIe RC controller on Toshiba Visconti ARM SoCs. PCIe
> controller is based of Synopsys DesignWare PCIe core.
>
> Signed-off-by: Yuji Ishikawa <yuji2.ishikawa@toshiba.co.jp>
> Signed-off-by: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp>
>
> v5 -> v6:
> - Remove unnecessary commit log.
> - Fix split line of visconti_add_pcie_port()
>
> v4 -> v5:
> - Remove PCIE_BUS_OFFSET
> - Change link_up confirmation function of visconti_pcie_link_up().
> - Move setting event mask before dw_pcie_link_up().
> - Move the contents of visconti_pcie_power_on() to visconti_pcie_host_init().
> - Remove code for link_gen.
>
> v3 -> v4:
> - Change variable from pci_addr to cpu_addr in visconti_pcie_cpu_addr_fixup().
> - Change the calculation method of CPU addres from subtraction to mask, and
> add comment.
> - Drop dma_set_mask_and_coherent().
> - Drop set MAX_MSI_IRQS.
> - Drop dev_dbg for Link speed.
> - Use use the dev_err_probe() to handle the devm_clk_get() failed.
> - Changed the redundant clock name.
>
> v2 -> v3:
> - Update subject.
> - Wrap description in 75 columns.
> - Change config name to PCIE_VISCONTI_HOST.
> - Update Kconfig text.
> - Drop blank lines.
> - Adjusted to 80 columns.
> - Drop inline from functions for register access.
> - Changed function name from visconti_pcie_check_link_status to
> visconti_pcie_link_up.
> - Update to using dw_pcie_host_init().
> - Reorder these in the order of use in visconti_pcie_establish_link.
> - Rewrite visconti_pcie_host_init() without dw_pcie_setup_rc().
> - Change function name from visconti_device_turnon() to
> visconti_pcie_power_on().
> - Unify formats such as dev_err().
> - Drop error label in visconti_add_pcie_port().
>
> v1 -> v2:
> - Fix typo in commit message.
> - Drop "depends on OF && HAS_IOMEM" from Kconfig.
> - Stop using the pointer of struct dw_pcie.
> - Use _relaxed variant.
> - Drop dw_pcie_wait_for_link.
> - Drop dbi resource processing.
> - Drop MSI IRQ initialization processing.
> ---
> drivers/pci/controller/dwc/Kconfig | 9 +
> drivers/pci/controller/dwc/Makefile | 1 +
> drivers/pci/controller/dwc/pcie-visconti.c | 333 +++++++++++++++++++++
> 3 files changed, 343 insertions(+)
> create mode 100644 drivers/pci/controller/dwc/pcie-visconti.c
Reviewed-by: Rob Herring <robh@kernel.org>
next prev parent reply other threads:[~2021-08-26 12:11 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-11 8:38 [PATCH v6 0/3] Visconti: Add Toshiba Visconti PCIe host controller driver Nobuhiro Iwamatsu
2021-08-11 8:38 ` [PATCH v6 1/3] dt-bindings: pci: Add DT binding for Toshiba Visconti PCIe controller Nobuhiro Iwamatsu
2021-08-11 17:50 ` Rob Herring
2021-08-11 8:38 ` [PATCH v6 2/3] PCI: visconti: Add Toshiba Visconti PCIe host controller driver Nobuhiro Iwamatsu
2021-08-26 12:11 ` Rob Herring [this message]
2021-08-11 8:38 ` [PATCH v6 3/3] MAINTAINERS: Add entries for Toshiba Visconti PCIe controller Nobuhiro Iwamatsu
2021-08-26 4:25 ` [PATCH v6 0/3] Visconti: Add Toshiba Visconti PCIe host controller driver nobuhiro1.iwamatsu
2021-08-26 13:01 ` Lorenzo Pieralisi
2021-08-26 23:49 ` nobuhiro1.iwamatsu
2021-08-27 9:48 ` Lorenzo Pieralisi
2021-08-28 1:14 ` Nobuhiro Iwamatsu
2021-08-30 16:00 ` Rob Herring
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_Jsq+tGAOo-9oKGF=B=3hCjiECBwxDujQu9drwCXK6u8J0BA@mail.gmail.com' \
--to=robh+dt@kernel.org \
--cc=bhelgaas@google.com \
--cc=devicetree@vger.kernel.org \
--cc=kishon@ti.com \
--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=nobuhiro1.iwamatsu@toshiba.co.jp \
--cc=punit1.agrawal@toshiba.co.jp \
--cc=yuji2.ishikawa@toshiba.co.jp \
/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).