Re: Per option CFLAGS?

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

 



On 09/15/2007 10:47 AM, Adrian Bunk wrote:

On Sat, Sep 15, 2007 at 01:30:21AM +0200, Rene Herman wrote:
On 09/15/2007 01:13 AM, H. Peter Anvin wrote:

Rene Herman wrote:
I have a single file foo.c that I want to generate two (ALSA) modules
from, snd-foo2000.ko and snd-foo2001.ko, by compiling with either
FOO2000 or FOO2001 defined.

I can do this, and ALSA does this a few times, by providing dummy
foo2000.c and foo2001.c files, like:

=== foo2000.c
#define FOO2000
#include "foo.c"
===

[ ... ]

The stub source file is usually considered a good way to do this.
Mmm. If I'll have to live with it, I can, but thought I'd ask if there was some nice build trickery available instead.

The usual trick is to create _three_ modules:

Two with the foo2000 and foo2001 specific parts, and a third one with all code used by both.

Or if foo2000 and foo2001 differ only in small details, create one snd-foo200x module supporting both at the same time.

Thanks for the comment. Yes, first would be massive overkill in this case and second somewhat annoying as one of the differences is support for different resources (IRQs) among the two versions, whereas I'm checking the validity of the passed in values at a time I do not know which version I'm looking at yet -- knowing that requires having talked to the hardware.

Can do, but for now it seems like the two seperate modules might be cleaner. Can keep things much more straighforward that way by just redefining a bunch of #defines.

I'll just do the split version first and if someone really wants me to, I'll merge them after all...

Rene

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

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux