[Buildroot] [PATCH 3/3] packages: ensure linux supports modules even when not using kernel-module

[Peter Korsgaard]

>>>>> "Yann" == Yann E MORIN <yann.morin.1998 at free.fr> writes:

 > Some packages build kernel modules without using the kernel-module infra
 > (because they use custom build systems); they do not automatically get
 > the kernel to support modules which is ensured when using the infra.

 > It must be done manually for all those packages.

 > Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
 > Cc: Peter Korsgaard <jacmet at uclibc.org>
 > Cc: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
 > Cc: Noé Rubinstein <nrubinstein at aldebaran.com>
 > Cc: Jan Viktorin <viktorin at rehivetech.com>
 > Cc: Gustavo Zacarias <gustavo at zacarias.com.ar>
 > ---
 >  package/linux-fusion/linux-fusion.mk     | 9 ++++++---
 >  package/nvidia-driver/nvidia-driver.mk   | 4 ++++
 >  package/racehound/racehound.mk           | 4 ++++
 >  package/rtai/rtai.mk                     | 4 ++++
 >  package/ti-gfx/ti-gfx.mk                 | 5 +++++
 >  package/xtables-addons/xtables-addons.mk | 4 ++++
 >  6 files changed, 27 insertions(+), 3 deletions(-)

 > +++ b/package/nvidia-driver/nvidia-driver.mk
 > @@ -91,6 +91,10 @@ ifeq ($(BR2_PACKAGE_NVIDIA_DRIVER_MODULE),y)
 
 >  NVIDIA_DRIVER_DEPENDENCIES += linux
 
 > +# We're building a kernel module without using the kernel-module infra,
 > +# so we need to tell we want module support in the kernel:
 > +LINUX_NEEDS_MODULES = y
 > +

Ehh, this should only be done if the package is enabled, E.G.

ifeq ($(BR2_PACKAGE_NVIDIA_DRIVER),y)
LINUX_NEEDS_MODULES = y
endif

-- 
Bye, Peter Korsgaard