* Re: [PATCH V3] PCI: dwc: tegra194: Broaden architecture dependency
2025-04-17 7:46 [PATCH V3] PCI: dwc: tegra194: Broaden architecture dependency Vidya Sagar
@ 2025-04-17 7:51 ` Niklas Cassel
2025-04-18 11:18 ` Vinod Koul
2025-04-18 17:21 ` Manivannan Sadhasivam
` (2 subsequent siblings)
3 siblings, 1 reply; 19+ messages in thread
From: Niklas Cassel @ 2025-04-17 7:51 UTC (permalink / raw)
To: Vidya Sagar
Cc: lpieralisi, kw, manivannan.sadhasivam, robh, bhelgaas, linux-pci,
linux-kernel, treding, jonathanh, kthota, mmaddireddy, sagar.tv,
Vinod Koul
+CC PHY maintainer.
On Thu, Apr 17, 2025 at 01:16:07PM +0530, Vidya Sagar wrote:
> Replace ARCH_TEGRA_194_SOC dependency with a more generic ARCH_TEGRA
> check, allowing the PCIe controller to be built on Tegra platforms
> beyond Tegra194. Additionally, ensure compatibility by requiring
> ARM64 or COMPILE_TEST.
>
> Link: https://patchwork.kernel.org/project/linux-pci/patch/20250128044244.2766334-1-vidyas@nvidia.com/
> Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
> ---
> v3:
> * Addressed warning from kernel test robot
>
> v2:
> * Addressed review comments from Niklas Cassel and Manivannan Sadhasivam
>
> drivers/pci/controller/dwc/Kconfig | 4 ++--
> drivers/phy/tegra/Kconfig | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/pci/controller/dwc/Kconfig b/drivers/pci/controller/dwc/Kconfig
> index d9f0386396ed..815b6e0d6a0c 100644
> --- a/drivers/pci/controller/dwc/Kconfig
> +++ b/drivers/pci/controller/dwc/Kconfig
> @@ -226,7 +226,7 @@ config PCIE_TEGRA194
>
> config PCIE_TEGRA194_HOST
> tristate "NVIDIA Tegra194 (and later) PCIe controller (host mode)"
> - depends on ARCH_TEGRA_194_SOC || COMPILE_TEST
> + depends on ARCH_TEGRA && (ARM64 || COMPILE_TEST)
> depends on PCI_MSI
> select PCIE_DW_HOST
> select PHY_TEGRA194_P2U
> @@ -241,7 +241,7 @@ config PCIE_TEGRA194_HOST
>
> config PCIE_TEGRA194_EP
> tristate "NVIDIA Tegra194 (and later) PCIe controller (endpoint mode)"
> - depends on ARCH_TEGRA_194_SOC || COMPILE_TEST
> + depends on ARCH_TEGRA && (ARM64 || COMPILE_TEST)
> depends on PCI_ENDPOINT
> select PCIE_DW_EP
> select PHY_TEGRA194_P2U
> diff --git a/drivers/phy/tegra/Kconfig b/drivers/phy/tegra/Kconfig
> index f30cfb42b210..342fb736da4b 100644
> --- a/drivers/phy/tegra/Kconfig
> +++ b/drivers/phy/tegra/Kconfig
> @@ -13,7 +13,7 @@ config PHY_TEGRA_XUSB
>
> config PHY_TEGRA194_P2U
> tristate "NVIDIA Tegra194 PIPE2UPHY PHY driver"
> - depends on ARCH_TEGRA_194_SOC || ARCH_TEGRA_234_SOC || COMPILE_TEST
> + depends on ARCH_TEGRA || COMPILE_TEST
> select GENERIC_PHY
> help
> Enable this to support the P2U (PIPE to UPHY) that is part of Tegra 19x
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [PATCH V3] PCI: dwc: tegra194: Broaden architecture dependency
2025-04-17 7:51 ` Niklas Cassel
@ 2025-04-18 11:18 ` Vinod Koul
0 siblings, 0 replies; 19+ messages in thread
From: Vinod Koul @ 2025-04-18 11:18 UTC (permalink / raw)
To: Niklas Cassel
Cc: Vidya Sagar, lpieralisi, kw, manivannan.sadhasivam, robh,
bhelgaas, linux-pci, linux-kernel, treding, jonathanh, kthota,
mmaddireddy, sagar.tv
On 17-04-25, 09:51, Niklas Cassel wrote:
> +CC PHY maintainer.
>
> On Thu, Apr 17, 2025 at 01:16:07PM +0530, Vidya Sagar wrote:
> > Replace ARCH_TEGRA_194_SOC dependency with a more generic ARCH_TEGRA
> > check, allowing the PCIe controller to be built on Tegra platforms
> > beyond Tegra194. Additionally, ensure compatibility by requiring
> > ARM64 or COMPILE_TEST.
> >
> > Link: https://patchwork.kernel.org/project/linux-pci/patch/20250128044244.2766334-1-vidyas@nvidia.com/
> > Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
> > ---
> > v3:
> > * Addressed warning from kernel test robot
> >
> > v2:
> > * Addressed review comments from Niklas Cassel and Manivannan Sadhasivam
> >
> > drivers/pci/controller/dwc/Kconfig | 4 ++--
> > drivers/phy/tegra/Kconfig | 2 +-
can phy and pci be two different patches?
> > 2 files changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/pci/controller/dwc/Kconfig b/drivers/pci/controller/dwc/Kconfig
> > index d9f0386396ed..815b6e0d6a0c 100644
> > --- a/drivers/pci/controller/dwc/Kconfig
> > +++ b/drivers/pci/controller/dwc/Kconfig
> > @@ -226,7 +226,7 @@ config PCIE_TEGRA194
> >
> > config PCIE_TEGRA194_HOST
> > tristate "NVIDIA Tegra194 (and later) PCIe controller (host mode)"
> > - depends on ARCH_TEGRA_194_SOC || COMPILE_TEST
> > + depends on ARCH_TEGRA && (ARM64 || COMPILE_TEST)
> > depends on PCI_MSI
> > select PCIE_DW_HOST
> > select PHY_TEGRA194_P2U
> > @@ -241,7 +241,7 @@ config PCIE_TEGRA194_HOST
> >
> > config PCIE_TEGRA194_EP
> > tristate "NVIDIA Tegra194 (and later) PCIe controller (endpoint mode)"
> > - depends on ARCH_TEGRA_194_SOC || COMPILE_TEST
> > + depends on ARCH_TEGRA && (ARM64 || COMPILE_TEST)
> > depends on PCI_ENDPOINT
> > select PCIE_DW_EP
> > select PHY_TEGRA194_P2U
> > diff --git a/drivers/phy/tegra/Kconfig b/drivers/phy/tegra/Kconfig
> > index f30cfb42b210..342fb736da4b 100644
> > --- a/drivers/phy/tegra/Kconfig
> > +++ b/drivers/phy/tegra/Kconfig
> > @@ -13,7 +13,7 @@ config PHY_TEGRA_XUSB
> >
> > config PHY_TEGRA194_P2U
> > tristate "NVIDIA Tegra194 PIPE2UPHY PHY driver"
> > - depends on ARCH_TEGRA_194_SOC || ARCH_TEGRA_234_SOC || COMPILE_TEST
> > + depends on ARCH_TEGRA || COMPILE_TEST
> > select GENERIC_PHY
> > help
> > Enable this to support the P2U (PIPE to UPHY) that is part of Tegra 19x
> > --
> > 2.25.1
> >
--
~Vinod
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH V3] PCI: dwc: tegra194: Broaden architecture dependency
2025-04-17 7:46 [PATCH V3] PCI: dwc: tegra194: Broaden architecture dependency Vidya Sagar
2025-04-17 7:51 ` Niklas Cassel
@ 2025-04-18 17:21 ` Manivannan Sadhasivam
2025-05-05 6:47 ` Vidya Sagar
[not found] ` <SN7PR12MB8028F87148D1B635F832EEB9B88E2@SN7PR12MB8028.namprd12.prod.outlook.com>
2025-05-08 5:19 ` [PATCH V4] " Vidya Sagar
2025-05-08 5:20 ` [PATCH V4] phy: tegra: p2u: " Vidya Sagar
3 siblings, 2 replies; 19+ messages in thread
From: Manivannan Sadhasivam @ 2025-04-18 17:21 UTC (permalink / raw)
To: Vidya Sagar
Cc: lpieralisi, kw, robh, bhelgaas, cassel, linux-pci, linux-kernel,
treding, jonathanh, kthota, mmaddireddy, sagar.tv
On Thu, Apr 17, 2025 at 01:16:07PM +0530, Vidya Sagar wrote:
> Replace ARCH_TEGRA_194_SOC dependency with a more generic ARCH_TEGRA
> check, allowing the PCIe controller to be built on Tegra platforms
> beyond Tegra194. Additionally, ensure compatibility by requiring
> ARM64 or COMPILE_TEST.
>
> Link: https://patchwork.kernel.org/project/linux-pci/patch/20250128044244.2766334-1-vidyas@nvidia.com/
> Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
- Mani
> ---
> v3:
> * Addressed warning from kernel test robot
>
> v2:
> * Addressed review comments from Niklas Cassel and Manivannan Sadhasivam
>
> drivers/pci/controller/dwc/Kconfig | 4 ++--
> drivers/phy/tegra/Kconfig | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/pci/controller/dwc/Kconfig b/drivers/pci/controller/dwc/Kconfig
> index d9f0386396ed..815b6e0d6a0c 100644
> --- a/drivers/pci/controller/dwc/Kconfig
> +++ b/drivers/pci/controller/dwc/Kconfig
> @@ -226,7 +226,7 @@ config PCIE_TEGRA194
>
> config PCIE_TEGRA194_HOST
> tristate "NVIDIA Tegra194 (and later) PCIe controller (host mode)"
> - depends on ARCH_TEGRA_194_SOC || COMPILE_TEST
> + depends on ARCH_TEGRA && (ARM64 || COMPILE_TEST)
> depends on PCI_MSI
> select PCIE_DW_HOST
> select PHY_TEGRA194_P2U
> @@ -241,7 +241,7 @@ config PCIE_TEGRA194_HOST
>
> config PCIE_TEGRA194_EP
> tristate "NVIDIA Tegra194 (and later) PCIe controller (endpoint mode)"
> - depends on ARCH_TEGRA_194_SOC || COMPILE_TEST
> + depends on ARCH_TEGRA && (ARM64 || COMPILE_TEST)
> depends on PCI_ENDPOINT
> select PCIE_DW_EP
> select PHY_TEGRA194_P2U
> diff --git a/drivers/phy/tegra/Kconfig b/drivers/phy/tegra/Kconfig
> index f30cfb42b210..342fb736da4b 100644
> --- a/drivers/phy/tegra/Kconfig
> +++ b/drivers/phy/tegra/Kconfig
> @@ -13,7 +13,7 @@ config PHY_TEGRA_XUSB
>
> config PHY_TEGRA194_P2U
> tristate "NVIDIA Tegra194 PIPE2UPHY PHY driver"
> - depends on ARCH_TEGRA_194_SOC || ARCH_TEGRA_234_SOC || COMPILE_TEST
> + depends on ARCH_TEGRA || COMPILE_TEST
> select GENERIC_PHY
> help
> Enable this to support the P2U (PIPE to UPHY) that is part of Tegra 19x
> --
> 2.25.1
>
--
மணிவண்ணன் சதாசிவம்
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [PATCH V3] PCI: dwc: tegra194: Broaden architecture dependency
2025-04-18 17:21 ` Manivannan Sadhasivam
@ 2025-05-05 6:47 ` Vidya Sagar
[not found] ` <SN7PR12MB8028F87148D1B635F832EEB9B88E2@SN7PR12MB8028.namprd12.prod.outlook.com>
1 sibling, 0 replies; 19+ messages in thread
From: Vidya Sagar @ 2025-05-05 6:47 UTC (permalink / raw)
To: Manivannan Sadhasivam
Cc: lpieralisi@kernel.org, kw@linux.com, robh@kernel.org,
bhelgaas@google.com, cassel@kernel.org, linux-pci@vger.kernel.org,
linux-kernel@vger.kernel.org, Thierry Reding, Jon Hunter,
Krishna Thota, Manikanta Maddireddy, sagar.tv@gmail.com
Hi Manivannan,
Please let me know if you are expecting any further changes to this patch.
Thanks,
Vidya Sagar
________________________________________
From: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Sent: Friday, April 18, 2025 22:51
To: Vidya Sagar <vidyas@nvidia.com>
Cc: lpieralisi@kernel.org <lpieralisi@kernel.org>; kw@linux.com <kw@linux.com>; robh@kernel.org <robh@kernel.org>; bhelgaas@google.com <bhelgaas@google.com>; cassel@kernel.org <cassel@kernel.org>; linux-pci@vger.kernel.org <linux-pci@vger.kernel.org>; linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org>; Thierry Reding <treding@nvidia.com>; Jon Hunter <jonathanh@nvidia.com>; Krishna Thota <kthota@nvidia.com>; Manikanta Maddireddy <mmaddireddy@nvidia.com>; sagar.tv@gmail.com <sagar.tv@gmail.com>
Subject: Re: [PATCH V3] PCI: dwc: tegra194: Broaden architecture dependency
External email: Use caution opening links or attachments
On Thu, Apr 17, 2025 at 01:16:07PM +0530, Vidya Sagar wrote:
> Replace ARCH_TEGRA_194_SOC dependency with a more generic ARCH_TEGRA
> check, allowing the PCIe controller to be built on Tegra platforms
> beyond Tegra194. Additionally, ensure compatibility by requiring
> ARM64 or COMPILE_TEST.
>
> Link: https://patchwork.kernel.org/project/linux-pci/patch/20250128044244.2766334-1-vidyas@nvidia.com/
> Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
- Mani
> ---
> v3:
> * Addressed warning from kernel test robot
>
> v2:
> * Addressed review comments from Niklas Cassel and Manivannan Sadhasivam
>
> drivers/pci/controller/dwc/Kconfig | 4 ++--
> drivers/phy/tegra/Kconfig | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/pci/controller/dwc/Kconfig b/drivers/pci/controller/dwc/Kconfig
> index d9f0386396ed..815b6e0d6a0c 100644
> --- a/drivers/pci/controller/dwc/Kconfig
> +++ b/drivers/pci/controller/dwc/Kconfig
> @@ -226,7 +226,7 @@ config PCIE_TEGRA194
>
> config PCIE_TEGRA194_HOST
> tristate "NVIDIA Tegra194 (and later) PCIe controller (host mode)"
> - depends on ARCH_TEGRA_194_SOC || COMPILE_TEST
> + depends on ARCH_TEGRA && (ARM64 || COMPILE_TEST)
> depends on PCI_MSI
> select PCIE_DW_HOST
> select PHY_TEGRA194_P2U
> @@ -241,7 +241,7 @@ config PCIE_TEGRA194_HOST
>
> config PCIE_TEGRA194_EP
> tristate "NVIDIA Tegra194 (and later) PCIe controller (endpoint mode)"
> - depends on ARCH_TEGRA_194_SOC || COMPILE_TEST
> + depends on ARCH_TEGRA && (ARM64 || COMPILE_TEST)
> depends on PCI_ENDPOINT
> select PCIE_DW_EP
> select PHY_TEGRA194_P2U
> diff --git a/drivers/phy/tegra/Kconfig b/drivers/phy/tegra/Kconfig
> index f30cfb42b210..342fb736da4b 100644
> --- a/drivers/phy/tegra/Kconfig
> +++ b/drivers/phy/tegra/Kconfig
> @@ -13,7 +13,7 @@ config PHY_TEGRA_XUSB
>
> config PHY_TEGRA194_P2U
> tristate "NVIDIA Tegra194 PIPE2UPHY PHY driver"
> - depends on ARCH_TEGRA_194_SOC || ARCH_TEGRA_234_SOC || COMPILE_TEST
> + depends on ARCH_TEGRA || COMPILE_TEST
> select GENERIC_PHY
> help
> Enable this to support the P2U (PIPE to UPHY) that is part of Tegra 19x
> --
> 2.25.1
>
--
மணிவண்ணன் சதாசிவம்
^ permalink raw reply [flat|nested] 19+ messages in thread[parent not found: <SN7PR12MB8028F87148D1B635F832EEB9B88E2@SN7PR12MB8028.namprd12.prod.outlook.com>]
* Re: [PATCH V3] PCI: dwc: tegra194: Broaden architecture dependency
[not found] ` <SN7PR12MB8028F87148D1B635F832EEB9B88E2@SN7PR12MB8028.namprd12.prod.outlook.com>
@ 2025-05-05 8:24 ` Manivannan Sadhasivam
0 siblings, 0 replies; 19+ messages in thread
From: Manivannan Sadhasivam @ 2025-05-05 8:24 UTC (permalink / raw)
To: Vidya Sagar
Cc: lpieralisi@kernel.org, kw@linux.com, robh@kernel.org,
bhelgaas@google.com, cassel@kernel.org, linux-pci@vger.kernel.org,
linux-kernel@vger.kernel.org, Thierry Reding, Jon Hunter,
Krishna Thota, Manikanta Maddireddy, sagar.tv@gmail.com
On Mon, May 05, 2025 at 06:21:41AM +0000, Vidya Sagar wrote:
> Hi Manivannan,
> Please let me know if you are expecting any further changes to this patch.
>
As per Vinod's comment, you either need to split PHY changes separate or get an
Ack from Vinod to merge both changes through PCI tree.
- Mani
> Thanks,
> Vidya Sagar
> ________________________________
> From: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> Sent: Friday, April 18, 2025 22:51
> To: Vidya Sagar <vidyas@nvidia.com>
> Cc: lpieralisi@kernel.org <lpieralisi@kernel.org>; kw@linux.com <kw@linux.com>; robh@kernel.org <robh@kernel.org>; bhelgaas@google.com <bhelgaas@google.com>; cassel@kernel.org <cassel@kernel.org>; linux-pci@vger.kernel.org <linux-pci@vger.kernel.org>; linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org>; Thierry Reding <treding@nvidia.com>; Jon Hunter <jonathanh@nvidia.com>; Krishna Thota <kthota@nvidia.com>; Manikanta Maddireddy <mmaddireddy@nvidia.com>; sagar.tv@gmail.com <sagar.tv@gmail.com>
> Subject: Re: [PATCH V3] PCI: dwc: tegra194: Broaden architecture dependency
>
> External email: Use caution opening links or attachments
>
>
> On Thu, Apr 17, 2025 at 01:16:07PM +0530, Vidya Sagar wrote:
> > Replace ARCH_TEGRA_194_SOC dependency with a more generic ARCH_TEGRA
> > check, allowing the PCIe controller to be built on Tegra platforms
> > beyond Tegra194. Additionally, ensure compatibility by requiring
> > ARM64 or COMPILE_TEST.
> >
> > Link: https://patchwork.kernel.org/project/linux-pci/patch/20250128044244.2766334-1-vidyas@nvidia.com/
> > Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
>
> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
>
> - Mani
>
> > ---
> > v3:
> > * Addressed warning from kernel test robot
> >
> > v2:
> > * Addressed review comments from Niklas Cassel and Manivannan Sadhasivam
> >
> > drivers/pci/controller/dwc/Kconfig | 4 ++--
> > drivers/phy/tegra/Kconfig | 2 +-
> > 2 files changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/pci/controller/dwc/Kconfig b/drivers/pci/controller/dwc/Kconfig
> > index d9f0386396ed..815b6e0d6a0c 100644
> > --- a/drivers/pci/controller/dwc/Kconfig
> > +++ b/drivers/pci/controller/dwc/Kconfig
> > @@ -226,7 +226,7 @@ config PCIE_TEGRA194
> >
> > config PCIE_TEGRA194_HOST
> > tristate "NVIDIA Tegra194 (and later) PCIe controller (host mode)"
> > - depends on ARCH_TEGRA_194_SOC || COMPILE_TEST
> > + depends on ARCH_TEGRA && (ARM64 || COMPILE_TEST)
> > depends on PCI_MSI
> > select PCIE_DW_HOST
> > select PHY_TEGRA194_P2U
> > @@ -241,7 +241,7 @@ config PCIE_TEGRA194_HOST
> >
> > config PCIE_TEGRA194_EP
> > tristate "NVIDIA Tegra194 (and later) PCIe controller (endpoint mode)"
> > - depends on ARCH_TEGRA_194_SOC || COMPILE_TEST
> > + depends on ARCH_TEGRA && (ARM64 || COMPILE_TEST)
> > depends on PCI_ENDPOINT
> > select PCIE_DW_EP
> > select PHY_TEGRA194_P2U
> > diff --git a/drivers/phy/tegra/Kconfig b/drivers/phy/tegra/Kconfig
> > index f30cfb42b210..342fb736da4b 100644
> > --- a/drivers/phy/tegra/Kconfig
> > +++ b/drivers/phy/tegra/Kconfig
> > @@ -13,7 +13,7 @@ config PHY_TEGRA_XUSB
> >
> > config PHY_TEGRA194_P2U
> > tristate "NVIDIA Tegra194 PIPE2UPHY PHY driver"
> > - depends on ARCH_TEGRA_194_SOC || ARCH_TEGRA_234_SOC || COMPILE_TEST
> > + depends on ARCH_TEGRA || COMPILE_TEST
> > select GENERIC_PHY
> > help
> > Enable this to support the P2U (PIPE to UPHY) that is part of Tegra 19x
> > --
> > 2.25.1
> >
>
> --
> மணிவண்ணன் சதாசிவம்
--
மணிவண்ணன் சதாசிவம்
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH V4] PCI: dwc: tegra194: Broaden architecture dependency
2025-04-17 7:46 [PATCH V3] PCI: dwc: tegra194: Broaden architecture dependency Vidya Sagar
2025-04-17 7:51 ` Niklas Cassel
2025-04-18 17:21 ` Manivannan Sadhasivam
@ 2025-05-08 5:19 ` Vidya Sagar
2025-05-08 12:17 ` Niklas Cassel
` (3 more replies)
2025-05-08 5:20 ` [PATCH V4] phy: tegra: p2u: " Vidya Sagar
3 siblings, 4 replies; 19+ messages in thread
From: Vidya Sagar @ 2025-05-08 5:19 UTC (permalink / raw)
To: lpieralisi, kw, manivannan.sadhasivam, robh, bhelgaas, cassel
Cc: linux-pci, linux-kernel, treding, jonathanh, kthota, mmaddireddy,
vidyas, sagar.tv
Replace ARCH_TEGRA_194_SOC dependency with a more generic ARCH_TEGRA
check for the Tegra194 PCIe controller, allowing it to be built on
Tegra platforms beyond Tegra194. Additionally, ensure compatibility
by requiring ARM64 or COMPILE_TEST.
Link: https://patchwork.kernel.org/project/linux-pci/patch/20250128044244.2766334-1-vidyas@nvidia.com/
Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
---
v4:
* Split the Tegra194 driver and phy driver changes
v3:
* Addressed warning from kernel test robot
v2:
* Addressed review comments from Niklas Cassel and Manivannan Sadhasivam
drivers/pci/controller/dwc/Kconfig | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/pci/controller/dwc/Kconfig b/drivers/pci/controller/dwc/Kconfig
index d9f0386396ed..815b6e0d6a0c 100644
--- a/drivers/pci/controller/dwc/Kconfig
+++ b/drivers/pci/controller/dwc/Kconfig
@@ -226,7 +226,7 @@ config PCIE_TEGRA194
config PCIE_TEGRA194_HOST
tristate "NVIDIA Tegra194 (and later) PCIe controller (host mode)"
- depends on ARCH_TEGRA_194_SOC || COMPILE_TEST
+ depends on ARCH_TEGRA && (ARM64 || COMPILE_TEST)
depends on PCI_MSI
select PCIE_DW_HOST
select PHY_TEGRA194_P2U
@@ -241,7 +241,7 @@ config PCIE_TEGRA194_HOST
config PCIE_TEGRA194_EP
tristate "NVIDIA Tegra194 (and later) PCIe controller (endpoint mode)"
- depends on ARCH_TEGRA_194_SOC || COMPILE_TEST
+ depends on ARCH_TEGRA && (ARM64 || COMPILE_TEST)
depends on PCI_ENDPOINT
select PCIE_DW_EP
select PHY_TEGRA194_P2U
--
2.25.1
^ permalink raw reply related [flat|nested] 19+ messages in thread* Re: [PATCH V4] PCI: dwc: tegra194: Broaden architecture dependency
2025-05-08 5:19 ` [PATCH V4] " Vidya Sagar
@ 2025-05-08 12:17 ` Niklas Cassel
2025-05-08 12:51 ` Thierry Reding
2025-05-08 13:20 ` Krzysztof Wilczyński
` (2 subsequent siblings)
3 siblings, 1 reply; 19+ messages in thread
From: Niklas Cassel @ 2025-05-08 12:17 UTC (permalink / raw)
To: Vidya Sagar
Cc: lpieralisi, kw, manivannan.sadhasivam, robh, bhelgaas, linux-pci,
linux-kernel, treding, jonathanh, kthota, mmaddireddy, sagar.tv
On Thu, May 08, 2025 at 10:49:22AM +0530, Vidya Sagar wrote:
> Replace ARCH_TEGRA_194_SOC dependency with a more generic ARCH_TEGRA
> check for the Tegra194 PCIe controller, allowing it to be built on
> Tegra platforms beyond Tegra194. Additionally, ensure compatibility
> by requiring ARM64 or COMPILE_TEST.
>
> Link: https://patchwork.kernel.org/project/linux-pci/patch/20250128044244.2766334-1-vidyas@nvidia.com/
> Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
> ---
Looks good to me, but there will need coordination between the
PHY and PCI maintainers for this to not cause a kernel test bot
build failure, if the PCI patch is merged before the PHY patch.
Reviewed-by: Niklas Cassel <cassel@kernel.org>
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [PATCH V4] PCI: dwc: tegra194: Broaden architecture dependency
2025-05-08 12:17 ` Niklas Cassel
@ 2025-05-08 12:51 ` Thierry Reding
2025-05-08 13:22 ` Krzysztof Wilczyński
0 siblings, 1 reply; 19+ messages in thread
From: Thierry Reding @ 2025-05-08 12:51 UTC (permalink / raw)
To: Niklas Cassel
Cc: Vidya Sagar, lpieralisi, kw, manivannan.sadhasivam, robh,
bhelgaas, linux-pci, linux-kernel, treding, jonathanh, kthota,
mmaddireddy, sagar.tv
[-- Attachment #1: Type: text/plain, Size: 1183 bytes --]
On Thu, May 08, 2025 at 02:17:24PM +0200, Niklas Cassel wrote:
> On Thu, May 08, 2025 at 10:49:22AM +0530, Vidya Sagar wrote:
> > Replace ARCH_TEGRA_194_SOC dependency with a more generic ARCH_TEGRA
> > check for the Tegra194 PCIe controller, allowing it to be built on
> > Tegra platforms beyond Tegra194. Additionally, ensure compatibility
> > by requiring ARM64 or COMPILE_TEST.
> >
> > Link: https://patchwork.kernel.org/project/linux-pci/patch/20250128044244.2766334-1-vidyas@nvidia.com/
> > Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
> > ---
>
> Looks good to me, but there will need coordination between the
> PHY and PCI maintainers for this to not cause a kernel test bot
> build failure, if the PCI patch is merged before the PHY patch.
>
> Reviewed-by: Niklas Cassel <cassel@kernel.org>
Either the PCI or PHY maintainers would need to provide an Acked-by so
these can go through the same tree.
Alternatively, since these are only platform-related Kconfig changes, I
could pick this up into the Tegra tree if I get Acked-bys from both
subsystems.
Either way is fine, and in case it helps:
Acked-by: Thierry Reding <treding@nvidia.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH V4] PCI: dwc: tegra194: Broaden architecture dependency
2025-05-08 12:51 ` Thierry Reding
@ 2025-05-08 13:22 ` Krzysztof Wilczyński
0 siblings, 0 replies; 19+ messages in thread
From: Krzysztof Wilczyński @ 2025-05-08 13:22 UTC (permalink / raw)
To: Thierry Reding
Cc: Niklas Cassel, Vidya Sagar, lpieralisi, manivannan.sadhasivam,
robh, bhelgaas, linux-pci, linux-kernel, treding, jonathanh,
kthota, mmaddireddy, sagar.tv
Hello,
[...]
> Alternatively, since these are only platform-related Kconfig changes, I
> could pick this up into the Tegra tree if I get Acked-bys from both
> subsystems.
No objections! Go ahead, and thank you! :)
Krzysztof
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH V4] PCI: dwc: tegra194: Broaden architecture dependency
2025-05-08 5:19 ` [PATCH V4] " Vidya Sagar
2025-05-08 12:17 ` Niklas Cassel
@ 2025-05-08 13:20 ` Krzysztof Wilczyński
2025-05-14 11:38 ` Vinod Koul
2026-01-05 12:20 ` Manivannan Sadhasivam
3 siblings, 0 replies; 19+ messages in thread
From: Krzysztof Wilczyński @ 2025-05-08 13:20 UTC (permalink / raw)
To: Vidya Sagar
Cc: lpieralisi, manivannan.sadhasivam, robh, bhelgaas, cassel,
linux-pci, linux-kernel, treding, jonathanh, kthota, mmaddireddy,
sagar.tv
Hello,
> Replace ARCH_TEGRA_194_SOC dependency with a more generic ARCH_TEGRA
> check for the Tegra194 PCIe controller, allowing it to be built on
> Tegra platforms beyond Tegra194. Additionally, ensure compatibility
> by requiring ARM64 or COMPILE_TEST.
Looks good! With that,
Acked-by: Krzysztof Wilczyński <kwilczynski@kernel.org>
Thank you!
Krzysztof
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH V4] PCI: dwc: tegra194: Broaden architecture dependency
2025-05-08 5:19 ` [PATCH V4] " Vidya Sagar
2025-05-08 12:17 ` Niklas Cassel
2025-05-08 13:20 ` Krzysztof Wilczyński
@ 2025-05-14 11:38 ` Vinod Koul
2025-05-14 13:02 ` Niklas Cassel
2026-01-05 12:20 ` Manivannan Sadhasivam
3 siblings, 1 reply; 19+ messages in thread
From: Vinod Koul @ 2025-05-14 11:38 UTC (permalink / raw)
To: lpieralisi, kw, manivannan.sadhasivam, robh, bhelgaas, cassel,
Vidya Sagar
Cc: linux-pci, linux-kernel, treding, jonathanh, kthota, mmaddireddy,
sagar.tv
On Thu, 08 May 2025 10:49:22 +0530, Vidya Sagar wrote:
> Replace ARCH_TEGRA_194_SOC dependency with a more generic ARCH_TEGRA
> check for the Tegra194 PCIe controller, allowing it to be built on
> Tegra platforms beyond Tegra194. Additionally, ensure compatibility
> by requiring ARM64 or COMPILE_TEST.
>
>
Applied, thanks!
[1/1] PCI: dwc: tegra194: Broaden architecture dependency
(no commit info)
[1/1] phy: tegra: p2u: Broaden architecture dependency
commit: 0c22287319741b4e7c7beaedac1f14fbe01a03b9
Best regards,
--
~Vinod
^ permalink raw reply [flat|nested] 19+ messages in thread* Re: [PATCH V4] PCI: dwc: tegra194: Broaden architecture dependency
2025-05-14 11:38 ` Vinod Koul
@ 2025-05-14 13:02 ` Niklas Cassel
2025-05-14 13:17 ` Vinod Koul
0 siblings, 1 reply; 19+ messages in thread
From: Niklas Cassel @ 2025-05-14 13:02 UTC (permalink / raw)
To: Vinod Koul
Cc: lpieralisi, kw, manivannan.sadhasivam, robh, bhelgaas,
Vidya Sagar, linux-pci, linux-kernel, treding, jonathanh, kthota,
mmaddireddy, sagar.tv
Hello Vinod, Krzysztof,
On Wed, May 14, 2025 at 12:38:01PM +0100, Vinod Koul wrote:
>
> On Thu, 08 May 2025 10:49:22 +0530, Vidya Sagar wrote:
> > Replace ARCH_TEGRA_194_SOC dependency with a more generic ARCH_TEGRA
> > check for the Tegra194 PCIe controller, allowing it to be built on
> > Tegra platforms beyond Tegra194. Additionally, ensure compatibility
> > by requiring ARM64 or COMPILE_TEST.
> >
> >
>
> Applied, thanks!
>
> [1/1] PCI: dwc: tegra194: Broaden architecture dependency
> (no commit info)
> [1/1] phy: tegra: p2u: Broaden architecture dependency
> commit: 0c22287319741b4e7c7beaedac1f14fbe01a03b9
>
> Best regards,
> --
I see that Vinod has queued patch 1/2.
Please don't forget that this series requires coordination.
There are many ways to solve it.
1) One tree takes both patches.
2) PHY tree puts the PHY patch on an immutable branch with just that
commit, and PCI merges that branch, so the same SHA1 of the PHY patch
is in both trees.
3) Send PHY patch for the upcoming merge window. Send PCI patch for
merge window + 1.
Kind regards,
Niklas
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH V4] PCI: dwc: tegra194: Broaden architecture dependency
2025-05-14 13:02 ` Niklas Cassel
@ 2025-05-14 13:17 ` Vinod Koul
2025-05-15 11:33 ` Manivannan Sadhasivam
0 siblings, 1 reply; 19+ messages in thread
From: Vinod Koul @ 2025-05-14 13:17 UTC (permalink / raw)
To: Niklas Cassel
Cc: lpieralisi, kw, manivannan.sadhasivam, robh, bhelgaas,
Vidya Sagar, linux-pci, linux-kernel, treding, jonathanh, kthota,
mmaddireddy, sagar.tv
On 14-05-25, 15:02, Niklas Cassel wrote:
> Hello Vinod, Krzysztof,
>
> On Wed, May 14, 2025 at 12:38:01PM +0100, Vinod Koul wrote:
> >
> > On Thu, 08 May 2025 10:49:22 +0530, Vidya Sagar wrote:
> > > Replace ARCH_TEGRA_194_SOC dependency with a more generic ARCH_TEGRA
> > > check for the Tegra194 PCIe controller, allowing it to be built on
> > > Tegra platforms beyond Tegra194. Additionally, ensure compatibility
> > > by requiring ARM64 or COMPILE_TEST.
> > >
> > >
> >
> > Applied, thanks!
> >
> > [1/1] PCI: dwc: tegra194: Broaden architecture dependency
> > (no commit info)
> > [1/1] phy: tegra: p2u: Broaden architecture dependency
> > commit: 0c22287319741b4e7c7beaedac1f14fbe01a03b9
> >
> > Best regards,
> > --
>
> I see that Vinod has queued patch 1/2.
>
> Please don't forget that this series requires coordination.
>
> There are many ways to solve it.
>
> 1) One tree takes both patches.
>
> 2) PHY tree puts the PHY patch on an immutable branch with just that
> commit, and PCI merges that branch, so the same SHA1 of the PHY patch
> is in both trees.
>
> 3) Send PHY patch for the upcoming merge window. Send PCI patch for
> merge window + 1.
1, 3 works for me, for 2 pls let me know, I need to prep a branch with
this patch and tag on it...
BR
--
~Vinod
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH V4] PCI: dwc: tegra194: Broaden architecture dependency
2025-05-14 13:17 ` Vinod Koul
@ 2025-05-15 11:33 ` Manivannan Sadhasivam
0 siblings, 0 replies; 19+ messages in thread
From: Manivannan Sadhasivam @ 2025-05-15 11:33 UTC (permalink / raw)
To: Vinod Koul
Cc: Niklas Cassel, lpieralisi, kw, robh, bhelgaas, Vidya Sagar,
linux-pci, linux-kernel, treding, jonathanh, kthota, mmaddireddy,
sagar.tv
On Wed, May 14, 2025 at 02:17:26PM +0100, Vinod Koul wrote:
> On 14-05-25, 15:02, Niklas Cassel wrote:
> > Hello Vinod, Krzysztof,
> >
> > On Wed, May 14, 2025 at 12:38:01PM +0100, Vinod Koul wrote:
> > >
> > > On Thu, 08 May 2025 10:49:22 +0530, Vidya Sagar wrote:
> > > > Replace ARCH_TEGRA_194_SOC dependency with a more generic ARCH_TEGRA
> > > > check for the Tegra194 PCIe controller, allowing it to be built on
> > > > Tegra platforms beyond Tegra194. Additionally, ensure compatibility
> > > > by requiring ARM64 or COMPILE_TEST.
> > > >
> > > >
> > >
> > > Applied, thanks!
> > >
> > > [1/1] PCI: dwc: tegra194: Broaden architecture dependency
> > > (no commit info)
> > > [1/1] phy: tegra: p2u: Broaden architecture dependency
> > > commit: 0c22287319741b4e7c7beaedac1f14fbe01a03b9
> > >
> > > Best regards,
> > > --
> >
> > I see that Vinod has queued patch 1/2.
> >
> > Please don't forget that this series requires coordination.
> >
> > There are many ways to solve it.
> >
> > 1) One tree takes both patches.
> >
> > 2) PHY tree puts the PHY patch on an immutable branch with just that
> > commit, and PCI merges that branch, so the same SHA1 of the PHY patch
> > is in both trees.
> >
> > 3) Send PHY patch for the upcoming merge window. Send PCI patch for
> > merge window + 1.
>
> 1, 3 works for me, for 2 pls let me know, I need to prep a branch with
> this patch and tag on it...
>
Feel free to take the patch 1 through PHY tree with:
Acked-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
I believe there should be no conflict with the PCI tree.
- Mani
--
மணிவண்ணன் சதாசிவம்
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH V4] PCI: dwc: tegra194: Broaden architecture dependency
2025-05-08 5:19 ` [PATCH V4] " Vidya Sagar
` (2 preceding siblings ...)
2025-05-14 11:38 ` Vinod Koul
@ 2026-01-05 12:20 ` Manivannan Sadhasivam
3 siblings, 0 replies; 19+ messages in thread
From: Manivannan Sadhasivam @ 2026-01-05 12:20 UTC (permalink / raw)
To: lpieralisi, robh, bhelgaas, cassel, Krzysztof Wilczyński,
Manivannan Sadhasivam, Vidya Sagar
Cc: linux-pci, linux-kernel, treding, jonathanh, kthota, mmaddireddy,
sagar.tv
On Thu, 08 May 2025 10:49:22 +0530, Vidya Sagar wrote:
> Replace ARCH_TEGRA_194_SOC dependency with a more generic ARCH_TEGRA
> check for the Tegra194 PCIe controller, allowing it to be built on
> Tegra platforms beyond Tegra194. Additionally, ensure compatibility
> by requiring ARM64 or COMPILE_TEST.
>
>
Applied, thanks!
[1/1] PCI: dwc: tegra194: Broaden architecture dependency
commit: bd9277c26522eebc6539edd0b0c99b6abdcb14ed
Best regards,
--
Manivannan Sadhasivam <mani@kernel.org>
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH V4] phy: tegra: p2u: Broaden architecture dependency
2025-04-17 7:46 [PATCH V3] PCI: dwc: tegra194: Broaden architecture dependency Vidya Sagar
` (2 preceding siblings ...)
2025-05-08 5:19 ` [PATCH V4] " Vidya Sagar
@ 2025-05-08 5:20 ` Vidya Sagar
2025-05-08 12:17 ` Niklas Cassel
2025-05-08 12:51 ` Thierry Reding
3 siblings, 2 replies; 19+ messages in thread
From: Vidya Sagar @ 2025-05-08 5:20 UTC (permalink / raw)
To: vkoul, kishon, manivannan.sadhasivam, bhelgaas, cassel
Cc: linux-phy, linux-tegra, linux-kernel, treding, jonathanh, kthota,
mmaddireddy, vidyas, sagar.tv
Replace the ARCH_TEGRA_194_SOC || ARCH_TEGRA_234_SOC dependency with a
more generic ARCH_TEGRA check for the Tegra194 PIPE2UPHY PHY driver.
This allows the PHY driver to be built on all Tegra platforms instead
of being limited to specific SoCs.
Link: https://patchwork.kernel.org/project/linux-pci/patch/20250128044244.2766334-1-vidyas@nvidia.com/
Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
---
v4:
* Split the Tegra194 driver and phy driver changes
v3:
* Addressed warning from kernel test robot
v2:
* Addressed review comments from Niklas Cassel and Manivannan Sadhasivam
drivers/phy/tegra/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/phy/tegra/Kconfig b/drivers/phy/tegra/Kconfig
index f30cfb42b210..342fb736da4b 100644
--- a/drivers/phy/tegra/Kconfig
+++ b/drivers/phy/tegra/Kconfig
@@ -13,7 +13,7 @@ config PHY_TEGRA_XUSB
config PHY_TEGRA194_P2U
tristate "NVIDIA Tegra194 PIPE2UPHY PHY driver"
- depends on ARCH_TEGRA_194_SOC || ARCH_TEGRA_234_SOC || COMPILE_TEST
+ depends on ARCH_TEGRA || COMPILE_TEST
select GENERIC_PHY
help
Enable this to support the P2U (PIPE to UPHY) that is part of Tegra 19x
--
2.25.1
^ permalink raw reply related [flat|nested] 19+ messages in thread* Re: [PATCH V4] phy: tegra: p2u: Broaden architecture dependency
2025-05-08 5:20 ` [PATCH V4] phy: tegra: p2u: " Vidya Sagar
@ 2025-05-08 12:17 ` Niklas Cassel
2025-05-08 12:51 ` Thierry Reding
1 sibling, 0 replies; 19+ messages in thread
From: Niklas Cassel @ 2025-05-08 12:17 UTC (permalink / raw)
To: Vidya Sagar
Cc: vkoul, kishon, manivannan.sadhasivam, bhelgaas, linux-phy,
linux-tegra, linux-kernel, treding, jonathanh, kthota,
mmaddireddy, sagar.tv
On Thu, May 08, 2025 at 10:50:21AM +0530, Vidya Sagar wrote:
> Replace the ARCH_TEGRA_194_SOC || ARCH_TEGRA_234_SOC dependency with a
> more generic ARCH_TEGRA check for the Tegra194 PIPE2UPHY PHY driver.
> This allows the PHY driver to be built on all Tegra platforms instead
> of being limited to specific SoCs.
>
> Link: https://patchwork.kernel.org/project/linux-pci/patch/20250128044244.2766334-1-vidyas@nvidia.com/
> Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
Looks good to me, but there will need coordination between the
PHY and PCI maintainers for this to not cause a kernel test bot
build failure, if the PCI patch is merged before the PHY patch.
Reviewed-by: Niklas Cassel <cassel@kernel.org>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH V4] phy: tegra: p2u: Broaden architecture dependency
2025-05-08 5:20 ` [PATCH V4] phy: tegra: p2u: " Vidya Sagar
2025-05-08 12:17 ` Niklas Cassel
@ 2025-05-08 12:51 ` Thierry Reding
1 sibling, 0 replies; 19+ messages in thread
From: Thierry Reding @ 2025-05-08 12:51 UTC (permalink / raw)
To: Vidya Sagar
Cc: vkoul, kishon, manivannan.sadhasivam, bhelgaas, cassel, linux-phy,
linux-tegra, linux-kernel, treding, jonathanh, kthota,
mmaddireddy, sagar.tv
[-- Attachment #1: Type: text/plain, Size: 832 bytes --]
On Thu, May 08, 2025 at 10:50:21AM +0530, Vidya Sagar wrote:
> Replace the ARCH_TEGRA_194_SOC || ARCH_TEGRA_234_SOC dependency with a
> more generic ARCH_TEGRA check for the Tegra194 PIPE2UPHY PHY driver.
> This allows the PHY driver to be built on all Tegra platforms instead
> of being limited to specific SoCs.
>
> Link: https://patchwork.kernel.org/project/linux-pci/patch/20250128044244.2766334-1-vidyas@nvidia.com/
> Signed-off-by: Vidya Sagar <vidyas@nvidia.com>
> ---
> v4:
> * Split the Tegra194 driver and phy driver changes
>
> v3:
> * Addressed warning from kernel test robot
>
> v2:
> * Addressed review comments from Niklas Cassel and Manivannan Sadhasivam
>
> drivers/phy/tegra/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Acked-by: Thierry Reding <treding@nvidia.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 19+ messages in thread