lkml.org 
[lkml]   [2007]   [Sep]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: Per option CFLAGS?
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 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/

\
 
 \ /
  Last update: 2007-09-15 17:21    [W:0.035 / U:0.080 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site