| Message ID | 1553613207-3988-11-git-send-email-vidyas@nvidia.com |
|---|---|
| State | Superseded |
| Delegated to: | Lorenzo Pieralisi |
| Headers | show |
| Series | Add Tegra194 PCIe support | expand |
On 26/03/2019 15:13, Vidya Sagar wrote: > Add PCIe host controller driver for DesignWare core based > PCIe controller IP present in Tegra194. > > Signed-off-by: Vidya Sagar <vidyas@nvidia.com> > --- > arch/arm64/configs/defconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig > index 2d9c39033c1a..2ddea5c4e87d 100644 > --- a/arch/arm64/configs/defconfig > +++ b/arch/arm64/configs/defconfig > @@ -87,6 +87,7 @@ CONFIG_PCIE_QCOM=y > CONFIG_PCIE_ARMADA_8K=y > CONFIG_PCIE_KIRIN=y > CONFIG_PCIE_HISI_STB=y > +CONFIG_PCIE_TEGRA194=y > CONFIG_ARM64_VA_BITS_48=y > CONFIG_SCHED_MC=y > CONFIG_NUMA=y Maybe building as a module is more appropriate here as I am not sure that anyone else will want this built-in and it is not critical to booting AFAIK. Cheers Jon
On 3/27/2019 3:38 PM, Jon Hunter wrote: > > On 26/03/2019 15:13, Vidya Sagar wrote: >> Add PCIe host controller driver for DesignWare core based >> PCIe controller IP present in Tegra194. >> >> Signed-off-by: Vidya Sagar <vidyas@nvidia.com> >> --- >> arch/arm64/configs/defconfig | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig >> index 2d9c39033c1a..2ddea5c4e87d 100644 >> --- a/arch/arm64/configs/defconfig >> +++ b/arch/arm64/configs/defconfig >> @@ -87,6 +87,7 @@ CONFIG_PCIE_QCOM=y >> CONFIG_PCIE_ARMADA_8K=y >> CONFIG_PCIE_KIRIN=y >> CONFIG_PCIE_HISI_STB=y >> +CONFIG_PCIE_TEGRA194=y >> CONFIG_ARM64_VA_BITS_48=y >> CONFIG_SCHED_MC=y >> CONFIG_NUMA=y > > Maybe building as a module is more appropriate here as I am not sure > that anyone else will want this built-in and it is not critical to > booting AFAIK. Since the DesignWare core framework doesn't yet have support for making this as a module, I added it as a built-in driver. I'll switch it to a module once support is available. > > Cheers > Jon >
On 27/03/2019 10:12, Vidya Sagar wrote: > On 3/27/2019 3:38 PM, Jon Hunter wrote: >> >> On 26/03/2019 15:13, Vidya Sagar wrote: >>> Add PCIe host controller driver for DesignWare core based >>> PCIe controller IP present in Tegra194. >>> >>> Signed-off-by: Vidya Sagar <vidyas@nvidia.com> >>> --- >>> arch/arm64/configs/defconfig | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig >>> index 2d9c39033c1a..2ddea5c4e87d 100644 >>> --- a/arch/arm64/configs/defconfig >>> +++ b/arch/arm64/configs/defconfig >>> @@ -87,6 +87,7 @@ CONFIG_PCIE_QCOM=y >>> CONFIG_PCIE_ARMADA_8K=y >>> CONFIG_PCIE_KIRIN=y >>> CONFIG_PCIE_HISI_STB=y >>> +CONFIG_PCIE_TEGRA194=y >>> CONFIG_ARM64_VA_BITS_48=y >>> CONFIG_SCHED_MC=y >>> CONFIG_NUMA=y >> >> Maybe building as a module is more appropriate here as I am not sure >> that anyone else will want this built-in and it is not critical to >> booting AFAIK. > Since the DesignWare core framework doesn't yet have support for making > this > as a module, I added it as a built-in driver. I'll switch it to a module > once support is available. Ah I see. We often get reports/patches if a driver has a removal function but is defined in the Kconfig to only support being built-in. Does the designware core just need to export some symbols to support it being a module? If so it maybe worth adding these as part of the series to see if it is acceptable, otherwise it might not get done and there is no point us supporting it as a module. Cheers Jon
diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig index 2d9c39033c1a..2ddea5c4e87d 100644 --- a/arch/arm64/configs/defconfig +++ b/arch/arm64/configs/defconfig @@ -87,6 +87,7 @@ CONFIG_PCIE_QCOM=y CONFIG_PCIE_ARMADA_8K=y CONFIG_PCIE_KIRIN=y CONFIG_PCIE_HISI_STB=y +CONFIG_PCIE_TEGRA194=y CONFIG_ARM64_VA_BITS_48=y CONFIG_SCHED_MC=y CONFIG_NUMA=y
Add PCIe host controller driver for DesignWare core based PCIe controller IP present in Tegra194. Signed-off-by: Vidya Sagar <vidyas@nvidia.com> --- arch/arm64/configs/defconfig | 1 + 1 file changed, 1 insertion(+)