Re: Linux 2.6.15-rc2

[Adrian Bunk <bunk@stusta.de>]

On Wed, Nov 23, 2005 at 07:26:09PM +0100, Sam Ravnborg wrote:
> On Wed, Nov 23, 2005 at 06:42:37PM +0100, Adrian Bunk wrote:
> >  EXTRA_CFLAGS += -I$(src)/..
> Wonder if this compiles with O=...
> 
> > -ifneq ($(CONFIG_VIDEO_BUF_DVB),n)
> > +ifneq ($(CONFIG_VIDEO_BUF_DVB),)
> >   EXTRA_CFLAGS += -DHAVE_VIDEO_BUF_DVB=1
> >  endif
> > -ifneq ($(CONFIG_DVB_CX22702),n)
> > +ifneq ($(CONFIG_DVB_CX22702),)
> >   EXTRA_CFLAGS += -DHAVE_CX22702=1
> >  endif
> > -ifneq ($(CONFIG_DVB_OR51132),n)
> > +ifneq ($(CONFIG_DVB_OR51132),)
> >   EXTRA_CFLAGS += -DHAVE_OR51132=1
> >  endif
> > -ifneq ($(CONFIG_DVB_LGDT330X),n)
> > +ifneq ($(CONFIG_DVB_LGDT330X),)
> >   EXTRA_CFLAGS += -DHAVE_LGDT330X=1
> >  endif
> > -ifneq ($(CONFIG_DVB_MT352),n)
> > +ifneq ($(CONFIG_DVB_MT352),)
> >   EXTRA_CFLAGS += -DHAVE_MT352=1
> >  endif
> > -ifneq ($(CONFIG_DVB_NXT200X),n)
> > +ifneq ($(CONFIG_DVB_NXT200X),)
> >   EXTRA_CFLAGS += -DHAVE_NXT200X=1
> >  endif
> > -
> 
> If we stick with HAVE_XXX then please use following style:
> 
> extra-cflags-$(CONFIG_VIDEO_BUF_DVB) += -DHAVE_VIDEO_BUF_DVB=1
> extra-cflags-$(CONFIG_DVB_CX22702)   += -DHAVE_CX22702=1
> extra-cflags-$(CONFIG_DVB_OR51132)   += -DHAVE_OR51132=1
> extra-cflags-$(CONFIG_DVB_LGDT330X)  += -DHAVE_LGDT330X=1
> extra-cflags-$(CONFIG_DVB_MT352)     += -DHAVE_MT352=1
> extra-cflags-$(CONFIG_DVB_NXT200X)   += -DHAVE_NXT200X=1
> 
> EXTRA_CFLAGS += $(extra-cflags-y) $(extra-cflags-m)

And this does still not solve all problems, the whole approach is a 
mess.

E.g. CONFIG_VIDEO_CX88_DVB=y, CONFIG_DVB_CX22702=m is one of the many 
cases that still won't compile.

I'll try to find a way for doing this all properly.

> 	Sam

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/