[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAK8P3a2wNNSQkN_m3DzEF5RLmq1aED1JpOkXW9Yq13+ypiAaGQ@mail.gmail.com>
Date: Tue, 21 Dec 2021 16:55:48 +0100
From: Arnd Bergmann <arnd@...db.de>
To: Tom Rini <trini@...sulko.com>
Cc: Vignesh Raghavendra <vigneshr@...com>,
Arnd Bergmann <arnd@...db.de>, Nishanth Menon <nm@...com>,
Olof Johansson <olof@...om.net>, SoC <soc@...nel.org>,
arm-soc <arm@...nel.org>, Tero Kristo <kristo@...nel.org>,
Linux ARM <linux-arm-kernel@...ts.infradead.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [GIT PULL 1/2] arm64: TI K3 SoC configs changes for v5.17
On Tue, Dec 21, 2021 at 4:32 PM Tom Rini <trini@...sulko.com> wrote:
> On Mon, Dec 20, 2021 at 11:10:25PM +0530, Vignesh Raghavendra wrote:
> >
> > Currently its not possible to build PCIE_CADENCE_PLAT_HOST/EP drivers as
> > modules (symbols are bool only).
> > PCIe is not necessary for basic boot either. So, I can drop these
> > configs until these drivers are build able as modules, if you prefer.
>
> Is PCIe required for basic boot for the other platforms in the defconfig
> which do enable it in the defconfig today? It is required for non-basic
> boot (whatever storage one puts in a PCIe slot). If someone is going to
> be fixing the PCIe driver to be able to be modular, that's fine too but
> I ran in to this trying to see what works out of the box in the
> defconfig, on this platform and hit both of these rather large
> omissions.
If PCI is often used for storage, then that's ok. There are a number of
other platforms where PCIe is only used for wireless networking or
other secondary devices, but they are still built-in because they got
added before it became possible for PCIe host drivers to be loadable
modules. I would like to eventually go through and turn those into
loadable modules, but for the moment it would be good to only add
built-in drivers where this is actually useful.
Arnd
Powered by blists - more mailing lists