Subject: Re: [PATCH v6 3/4] PCI: tegra: Add Tegra264 support

From: Jon Hunter <[email-masked]>
Date: Tue, 2 Jun 2026 17:49:58 +0100

On 02/06/2026 12:33, Thierry Reding wrote:
> From: Thierry Reding <[email-masked]>
> 
> Add a driver for the PCIe controller found on NVIDIA Tegra264 SoCs. The
> driver is very small, with its main purpose being to set up the address
> translation registers and then creating a standard PCI host using ECAM.
> 
> Signed-off-by: Manikanta Maddireddy <[email-masked]>
> Signed-off-by: Thierry Reding <[email-masked]>

...

> diff --git a/drivers/pci/controller/Kconfig b/drivers/pci/controller/Kconfig
> index 2247709ef6d6..dde2f59fef7a 100644
> --- a/drivers/pci/controller/Kconfig
> +++ b/drivers/pci/controller/Kconfig
> @@ -255,7 +255,14 @@ config PCI_TEGRA
>   	select IRQ_MSI_LIB
>   	help
>   	  Say Y here if you want support for the PCIe host controller found
> -	  on NVIDIA Tegra SoCs.
> +	  on NVIDIA Tegra SoCs (Tegra20 through Tegra186).
> +
> +config PCIE_TEGRA264
> +	tristate "NVIDIA Tegra264 PCIe controller"
> +	depends on ARCH_TEGRA || COMPILE_TEST
> +	help
> +	  Say Y here if you want support for the PCIe host controller found
> +	  on NVIDIA Tegra264 SoCs.

We are missing a 'depends on ECAM' here, because when I happened to 
enable for ARM builds I am getting errors like ...

  drivers/pci/controller/pcie-tegra264.c:485:(.text+0x320): undefined
   reference to `pci_ecam_free'
  drivers/pci/controller/pcie-tegra264.c:431:(.text+0x880): undefined
   reference to `pci_generic_ecam_ops'

Jon

-- 
nvpublic