Re: compiling cmipci in 2.5.10 on Alpha doesn't work

Jaroslav Kysela (perex@suse.cz)
Fri, 26 Apr 2002 12:26:10 +0200 (CEST)


On Fri, 26 Apr 2002, Ivan Kokshaysky wrote:

> On Fri, Apr 26, 2002 at 11:17:23AM +0200, Jaroslav Kysela wrote:
> > The real fix is to add '#include <linux/pci.h>' line to all necessary
> > source files (sound/pci/cmipci.c in this example). Not all source files
> > need pci.h for compilation.
>
> Yes, but quite a few of them. Almost all files under sound/pci,
> plus isapnp stuff which needs struct pci_dev and struct pci_bus.
> Including <linux/pci.h> in single place would be much simpler
> and shouldn't break anything, no?

<linux/isapnp.h> already includes <linux/pci.h> and there are many files
in sound/core, sound/drivers, sound/i2c which really have not anything
related to PCI. I think that it's better to include only related header
files to optimize compilation (although current CPUs are fast enough).

Jaroslav

-----
Jaroslav Kysela <perex@suse.cz>
Linux Kernel Sound Maintainer
ALSA Project http://www.alsa-project.org
SuSE Linux http://www.suse.com

-
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/