From: srikanth.thokala@intel.com
To: bhelgaas@google.com, robh+dt@kernel.org, lorenzo.pieralisi@arm.com
Cc: linux-pci@vger.kernel.org, devicetree@vger.kernel.org,
andriy.shevchenko@linux.intel.com, mgross@linux.intel.com,
lakshmi.bai.raja.subramanian@intel.com,
mallikarjunappa.sangannavar@intel.com,
srikanth.thokala@intel.com
Subject: [PATCH v6 0/2] PCI: keembay: Add support for Intel Keem Bay
Date: Fri, 22 Jan 2021 08:56:08 +0530 [thread overview]
Message-ID: <20210122032610.4958-1-srikanth.thokala@intel.com> (raw)
From: Srikanth Thokala <srikanth.thokala@intel.com>
Hi,
The first patch is to document DT bindings for Keem Bay PCIe controller
for both Root Complex and Endpoint modes.
The second patch is the driver file, a glue driver. Keem Bay PCIe
controller is based on DesignWare PCIe IP.
The patch was tested with Keem Bay evaluation module board, with A0
stepping.
I would also like to inform that I will be taking ownership of this patch
series from this version as the previous author moved out of the organization.
Kindly review.
Thanks!
Srikanth
Changes since v5:
- Rebased to v5.11-rc4.
- Updated maintainers to add myself in DT binding documents.
- Fix checkpatch issues.
Changes since v4:
- Rebased to v5.11-rc1 and retest.
Changes since v3:
- Add Reviewed-by: Rob Herring <robh@kernel.org> tag in dt-bindings
patch.
- Remove the keembay_pcie_{readl,writel} wrappers. And replace them with
readl() and writel().
- Remove the dead code related to unused irqs.
- Remove unused definition for unused irqs.
- In keembay_pcie_ep_init(), initialize enabled interrupts to known state.
- Rebased to next-20201215.
Changes since v2:
- In keembay_pcie_probe(), use return keembay_pcie_add_pcie_port(pcie,
pdev); statement and remove return 0; at the end of the function.
Changes since v1:
- In dt-bindings patch.
- Fixed indent warning for compatible property.
- Rename interrupt-names to pcie, pcie_ev, pcie_err and
pcie_mem_access, similar to the name used in datasheet.
- Remove device_type, #address-cells and #size-cells property.
- Remove num-viewport, num-ib-windows and num-ob-windows property.
- Replace additionalProperties with unevaluatedProperties, for RC
only.
- Add dbi2 and atu property.
- Remove description for regs and interrupts property.
- Change enum value for num-lanes to 1 and 2 only.
- In driver patch.
- In Kconfig file, remove dependency on ARM64.
- Add new define, PCIE_REGS_PCIE_SII_LINK_UP.
- Remove PCIE_DBI2_MASK.
- In struct keembay_pcie, declare pci member as struct, not pointer.
And remove irq number members.
- Rename and rework keembay_pcie_establish_link(), to
keembay_pcie_start_link().
- Remove unneeded BAR disable steps.
- Remove unused interrupt handlers; keembay_pcie_ev_irq_handler(),
keembay_pcie_err_irq_handler().
- Remove keembay_pcie_enable_interrupts().
- Rework keembay_pcie_setup_irq() and call it from
keembay_pcie_probe().
- Remove keembay_pcie_host_init() and make keembay_pcie_host_ops
empty.
- Keep and rework keembay_pcie_add_pcie_port() a little.
- Remove keembay_pcie_add_pcie_ep() and call dw_pcie_ep_init() from
keembay_pcie_probe().
- In keembay_pcie_probe(), remove dbi setup as it will be handled in
dwc common code.
- In keembay_pcie_link_up(), use return (val &
PCIE_REGS_PCIE_SII_LINK_UP) == PCIE_REGS_PCIE_SII_LINK_UP.
- In keembay_pcie_ep_raise_irq(), rework error message for
PCI_EPC_IRQ_LEGACY and default cases.
- Rebased to next-20201124, that has dwc pci refactoring,
https://lore.kernel.org/linux-pci/20201105211159.1814485-1-robh@kernel.org/.
Srikanth Thokala (2):
dt-bindings: PCI: Add Intel Keem Bay PCIe controller
PCI: keembay: Add support for Intel Keem Bay
.../bindings/pci/intel,keembay-pcie-ep.yaml | 68 +++
.../bindings/pci/intel,keembay-pcie.yaml | 96 ++++
drivers/pci/controller/dwc/Kconfig | 24 +
drivers/pci/controller/dwc/Makefile | 1 +
drivers/pci/controller/dwc/pcie-keembay.c | 448 ++++++++++++++++++
5 files changed, 637 insertions(+)
create mode 100644 Documentation/devicetree/bindings/pci/intel,keembay-pcie-ep.yaml
create mode 100644 Documentation/devicetree/bindings/pci/intel,keembay-pcie.yaml
create mode 100644 drivers/pci/controller/dwc/pcie-keembay.c
base-commit: 45dfb8a5659ad286c28fa59008271dbc4e5e3f2d
--
2.17.1
next reply other threads:[~2021-01-21 19:37 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-22 3:26 srikanth.thokala [this message]
2021-01-22 3:26 ` [PATCH v6 1/2] dt-bindings: PCI: Add Intel Keem Bay PCIe controller srikanth.thokala
2021-01-22 3:26 ` [PATCH v6 2/2] PCI: keembay: Add support for Intel Keem Bay srikanth.thokala
2021-01-21 19:52 ` Bjorn Helgaas
2021-01-21 20:41 ` Andy Shevchenko
2021-01-21 21:11 ` Bjorn Helgaas
2021-01-22 20:08 ` Thokala, Srikanth
2021-01-22 19:20 ` Thokala, Srikanth
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=20210122032610.4958-1-srikanth.thokala@intel.com \
--to=srikanth.thokala@intel.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=bhelgaas@google.com \
--cc=devicetree@vger.kernel.org \
--cc=lakshmi.bai.raja.subramanian@intel.com \
--cc=linux-pci@vger.kernel.org \
--cc=lorenzo.pieralisi@arm.com \
--cc=mallikarjunappa.sangannavar@intel.com \
--cc=mgross@linux.intel.com \
--cc=robh+dt@kernel.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).