Re: Per option CFLAGS?

[Denys Vlasenko]

On Saturday 15 September 2007 16:17, Rene Herman wrote:
> On 09/15/2007 10:47 AM, Adrian Bunk wrote:
> >>> 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.

If I understood you right, this is PCI, USB or other kind of devices
where module is selected according to PCI/USB/whatever IDs.

You thinking is - if module foo2000 is loaded, this is definitely a
2000 device and you can "check the validity of the passed in values"
for the device 2000 now.

Correct me if I'm wrong.

But you can do absolutely the same at runtime in unified driver,
since PCI/USB/whatever IDs are accessible to the driver.

Two drivers will most likely have code duplication.

Unified driver will be harder to read and maintain because of all
ifs/switch's/function pointers you need.

The choice depends on *how much* maintainability pain you can
tolerate by taking "unified code" path.

> Can do, but for now it seems like the two seperate modules might be cleaner. 

Can you give a bit more info what the dirrefences between devices are
in this particular cases?
--
vda
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/