On Mon, Jul 30, 2018 at 06:50:34PM -0400, Alexei Colin wrote: > Platforms with a PCI bus will be offered the RapidIO menu since they may > be want support for a RapidIO PCI device. Platforms without a PCI bus > that might include a RapidIO IP block will need to "select HAS_RAPIDIO" > in the platform-/machine-specific "config ARCH_*" Kconfig entry. > > Tested that kernel builds for arm64 with RapidIO subsystem and > switch drivers enabled, also that the modules load successfully > on a custom Aarch64 Qemu model. > > Cc: Andrew Morton <a...@linux-foundation.org> > Cc: Russell King <li...@armlinux.org.uk> > Cc: John Paul Walters <jwalt...@isi.edu> > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-kernel@vger.kernel.org, > Signed-off-by: Alexei Colin <aco...@isi.edu> > --- > arch/arm64/Kconfig | 2 ++ > 1 file changed, 2 insertions(+)
Thanks, this looks much cleaner than before: Acked-by: Will Deacon <will.dea...@arm.com> The only thing I'm not sure about is why we don't just select HAS_RAPIDIO unconditionally in the arm64 Kconfig. Does selecting only that option actually pull in new code to the build? Will > diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig > index a8f0c74e6f7f..5e8cf90505ec 100644 > --- a/arch/arm64/Kconfig > +++ b/arch/arm64/Kconfig > @@ -308,6 +308,8 @@ config PCI_SYSCALL > > source "drivers/pci/Kconfig" > > +source "drivers/rapidio/Kconfig" > + > endmenu > > menu "Kernel Features" > -- > 2.18.0 >
