[lkml]   [2001]   [May]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] net #3
David Woodhouse wrote:
> said:
> > -#ifdef CONFIG_ISAPNP
> > +#if defined(CONFIG_ISAPNP) || (defined(CONFIG_ISAPNP_MODULE) && defined(MODULE))
> The result here would be a 3c509 module which differs depending on whether
> the ISAPNP module happened to be compiled at the same time or not.

In and of itself, that is an acceptable result. We simply cannot take
into kernel configuration variations between two kernel builds. If you
build your isapnp module with one configuration, and your 3c509 module
with another config, consider yourself lucky that it works at all.

> The ISAPNP-specific parts of the code aren't large. Please consider
> including them unconditionally instead.

I probably agree with you and Paul, but I'll have to give it a bit more
thought. Past a certain point, bus support needs to be in its own
driver, similar to drivers/sound/{sb_card,sb_common}.c. But in this
particular case, that would be overkill.

Jeff Garzik | Disbelief, that's why you fail.
Building 1024 |
MandrakeSoft |
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2005-03-22 12:54    [W:0.074 / U:0.520 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site