Re: [PATCH V4 0/5] Add support for PCIe endpoint mode in Tegra194

From: Vidya Sagar <vidyas@nvidia.com>
To: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Cc: <bhelgaas@google.com>, <robh+dt@kernel.org>,
	<thierry.reding@gmail.com>, <jonathanh@nvidia.com>,
	<andrew.murray@arm.com>, <kishon@ti.com>,
	<gustavo.pimentel@synopsys.com>, <linux-pci@vger.kernel.org>,
	<devicetree@vger.kernel.org>, <linux-tegra@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>, <kthota@nvidia.com>,
	<mmaddireddy@nvidia.com>, <sagar.tv@gmail.com>
Subject: Re: [PATCH V4 0/5] Add support for PCIe endpoint mode in Tegra194
Date: Tue, 10 Mar 2020 23:12:35 +0530	[thread overview]
Message-ID: <b47de9cd-be18-1672-d72c-de4169fb90e5@nvidia.com> (raw)
In-Reply-To: <d1e3a42f-c179-fd03-fac0-7cd449580e5b@nvidia.com>


On 3/3/2020 11:43 PM, Vidya Sagar wrote:
> 
> 
> On 3/3/2020 10:31 PM, Lorenzo Pieralisi wrote:
>> External email: Use caution opening links or attachments
>>
>>
>> On Tue, Mar 03, 2020 at 04:24:13PM +0530, Vidya Sagar wrote:
>>> Tegra194 has three (C0, C4 & C5) dual mode PCIe controllers that can 
>>> operate
>>> either in root port mode or in end point mode but only in one mode at 
>>> a time.
>>> Platform P2972-0000 supports enabling endpoint mode for C5 
>>> controller. This
>>> patch series adds support for PCIe endpoint mode in both the driver 
>>> as well as
>>> in DT.
>>> This patch series depends on the changes made for Synopsys DesignWare 
>>> endpoint
>>> mode subsystem that are recently accepted.
>>> @ https://patchwork.kernel.org/project/linux-pci/list/?series=202211
>>> which in turn depends on the patch made by Kishon
>>> @ https://patchwork.kernel.org/patch/10975123/
>>> which is also under review.
>>>
>>> V4:
>>> * Started using threaded irqs instead of kthreads
>>
>> Hi Vidya,
>>
>> sorry for the bother, may I ask you to rebase the series (after
>> answering Thierry's query) on top of my pci/endpoint branch please ?
>>
>> Please resend it and I will merge patches {1,2,5} then.
> Sure.
> I just sent V5 series rebasing patches on top of your pci/endpoint branch.
Hi,
Sorry to bother you again.
Could you please take a look at V5 series?

Thanks,
Vidya Sagar
> 
> Thanks,
> Vidya Sagar
>>
>> Thanks,
>> Lorenzo
>>
>>> V3:
>>> * Re-ordered patches in the series to make the driver change as the 
>>> last patch
>>> * Took care of Thierry's review comments
>>>
>>> V2:
>>> * Addressed Thierry & Bjorn's review comments
>>> * Added EP mode specific binding documentation to already existing 
>>> binding documentation file
>>> * Removed patch that enables GPIO controller nodes explicitly as they 
>>> are enabled already
>>>
>>> Vidya Sagar (5):
>>>    soc/tegra: bpmp: Update ABI header
>>>    dt-bindings: PCI: tegra: Add DT support for PCIe EP nodes in Tegra194
>>>    arm64: tegra: Add PCIe endpoint controllers nodes for Tegra194
>>>    arm64: tegra: Add support for PCIe endpoint mode in P2972-0000
>>>      platform
>>>    PCI: tegra: Add support for PCIe endpoint mode in Tegra194
>>>
>>>   .../bindings/pci/nvidia,tegra194-pcie.txt     | 125 +++-
>>>   .../boot/dts/nvidia/tegra194-p2972-0000.dts   |  18 +
>>>   arch/arm64/boot/dts/nvidia/tegra194.dtsi      |  99 +++
>>>   drivers/pci/controller/dwc/Kconfig            |  30 +-
>>>   drivers/pci/controller/dwc/pcie-tegra194.c    | 681 +++++++++++++++++-
>>>   include/soc/tegra/bpmp-abi.h                  |  10 +-
>>>   6 files changed, 918 insertions(+), 45 deletions(-)
>>>
>>> -- 
>>> 2.17.1
>>>