From: "Thokala, Srikanth" <srikanth.thokala@intel.com>
To: "robh+dt@kernel.org" <robh+dt@kernel.org>,
"lorenzo.pieralisi@arm.com" <lorenzo.pieralisi@arm.com>,
"bhelgaas@google.com" <bhelgaas@google.com>
Cc: "linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"andriy.shevchenko@linux.intel.com"
<andriy.shevchenko@linux.intel.com>,
"mgross@linux.intel.com" <mgross@linux.intel.com>,
"Raja Subramanian,
Lakshmi Bai" <lakshmi.bai.raja.subramanian@intel.com>,
"Sangannavar,
Mallikarjunappa" <mallikarjunappa.sangannavar@intel.com>,
"kw@linux.com" <kw@linux.com>
Subject: RE: [PATCH v10 0/2] PCI: keembay: Add support for Intel Keem Bay
Date: Tue, 15 Jun 2021 14:38:37 +0000 [thread overview]
Message-ID: <PH0PR11MB55954001B54E69B49247E15785309@PH0PR11MB5595.namprd11.prod.outlook.com> (raw)
In-Reply-To: <20210607154044.26074-1-srikanth.thokala@intel.com>
Hi Lorenzo Pieralisi and Bjorn Helgaas,
> -----Original Message-----
> From: Thokala, Srikanth <srikanth.thokala@intel.com>
> Sent: Monday, June 7, 2021 9:11 PM
> To: 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; Raja
> Subramanian, Lakshmi Bai <lakshmi.bai.raja.subramanian@intel.com>;
> Sangannavar, Mallikarjunappa <mallikarjunappa.sangannavar@intel.com>;
> kw@linux.com; Thokala, Srikanth <srikanth.thokala@intel.com>
> Subject: [PATCH v10 0/2] PCI: keembay: Add support for Intel Keem Bay
>
> 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 B0
> stepping.
>
> Kindly review.
Kindly review the patch series.
I have got 'Reviewed-by' for both patches in the series and please let
me know if you have any comments that I need to address further.
Thanks!
Srikanth
>
> Thanks!
> Srikanth
>
> Changes since v9:
> - Add 'Reviewed-by: Krzysztof Wilczyński <kw@linux.com>', thanks to
> Krzysztof.
> - Rebased to v5.13-rc5.
>
> Changes since v8:
> - Use chained IRQ to handle MSIs, as suggested by Lorenzo Pieralisi.
> - Rename *_setup_irq() to *_setup_msi_irq() to be more meaningful.
> - Move *_setup_msi_irq() call to *_add_pcie_port() to make it invoke
> only when controller is in Root Complex mode. In Endpoint mode,
> IRQ will be setup by the respective driver which will be based on
> PCIe End Point Framework.
>
> Changes since v7:
> - Rename keembay_pcie_ltssm_enable() to align to its functionality.
> - Fix other minor comments from "Krzysztof Wilczyński <kw@linux.com>"
>
> Changes since v6:
> - Arrange SoB in chronological order.
> - Alphabetized and modified status of entry in MAINTAINERS.
> - Added a comment to specify the PCIe spec section about the delay.
>
> 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 | 69 +++
> .../bindings/pci/intel,keembay-pcie.yaml | 97 ++++
> MAINTAINERS | 7 +
> drivers/pci/controller/dwc/Kconfig | 28 ++
> drivers/pci/controller/dwc/Makefile | 1 +
> drivers/pci/controller/dwc/pcie-keembay.c | 451 ++++++++++++++++++
> 6 files changed, 653 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: 614124bea77e452aa6df7a8714e8bc820b489922
> --
> 2.17.1
prev parent reply other threads:[~2021-06-15 14:38 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-07 15:40 [PATCH v10 0/2] PCI: keembay: Add support for Intel Keem Bay srikanth.thokala
2021-06-07 15:40 ` [PATCH v10 1/2] dt-bindings: PCI: Add Intel Keem Bay PCIe controller srikanth.thokala
2021-06-07 15:40 ` [PATCH v10 2/2] PCI: keembay: Add support for Intel Keem Bay srikanth.thokala
2021-06-15 21:09 ` Rob Herring
2021-06-16 7:49 ` Thokala, Srikanth
2021-06-21 16:53 ` Lorenzo Pieralisi
2021-06-25 3:23 ` Thokala, Srikanth
2021-07-07 11:54 ` Thokala, Srikanth
2021-07-26 6:30 ` Thokala, Srikanth
2021-07-26 8:00 ` Marc Zyngier
2021-07-27 16:26 ` Thokala, Srikanth
2021-06-15 14:38 ` Thokala, Srikanth [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=PH0PR11MB55954001B54E69B49247E15785309@PH0PR11MB5595.namprd11.prod.outlook.com \
--to=srikanth.thokala@intel.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=bhelgaas@google.com \
--cc=devicetree@vger.kernel.org \
--cc=kw@linux.com \
--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).