Re: Problem with changeset 10837: causes "make all" not to build many modules

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Op zondag 08-03-2009 om 13:54 uur [tijdzone -0300], schreef Mauro
Carvalho Chehab:
> Hi Alain,
> 
> On Sat, 7 Mar 2009, Alain Kalker wrote:
> 
> > Mauro,
> >
> > Your latest changeset causes many modules (100 in total!) not to be
> > built anymore when doing "make all", i.e. without doing any "make
> > xconfig"/"make gconfig".
> >
> > I think this is related to the config variables for the frontend drivers
> > no longer being defined when DVB_FE_CUSTOMISE=n , so the card drivers
> > cannot depend on them anymore.
> 
> Thanks to warning me about that!
> 
> This seems to be yet another difference between the in-kernel and the 
> out-of-tree building environment.

If the problem doesn't manifest itself during in-kernel build, I believe
it must be with either v4l/Makefile or one of the scripts in scripts/*

As a matter of fact, I found out that commenting out
"disable_config('DVB_FE_CUSTOMISE');" in scripts/make_kconfig.pl line
588 and doing a "make distclean; make all" will cause all the undefined
config variables to be set to 'm' and the missing modules to be built
again.

Why is this disable_config() in there anyway? There is no corresponding
disable_config("MEDIA_TUNER_CUSTOMIZE"), which is used in the same way
in linux/drivers/media/common/tuners/Kconfig to hide a menu.

The only (aesthetic?) difference is that DVB_FE_CUSTOMISE ends up set to
'y' in the generated config (as has always been the case with
MEDIA_TUNER_CUSTOMIZE by the way), but that doesn't matter much at
module build time. A user should not configure _after_ building modules
anyway, so the menu showing up doesn't really matter.

Also note yet another -IZE / -ISE spelling issue :-)

Kind regards,

Alain

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux