acpi(4): Always compile the files dealing with ACPI_DEBUG into the module.
authorSascha Wildner <saw@online.de>
Wed, 2 Feb 2011 18:04:16 +0000 (19:04 +0100)
committerSascha Wildner <saw@online.de>
Wed, 2 Feb 2011 18:04:59 +0000 (19:04 +0100)
commit7b19b1789a96e40c92e42cf7291ad1819d6d043a
tree02b5bd3eb7b0ea1c4c5466d7f9e7548cdc555a0d
parent7bf5fa56b57c6864fef14bc71b211a265244d886
acpi(4): Always compile the files dealing with ACPI_DEBUG into the module.

Before this commit, one had to define ACPI_DEBUG as a make variable to
enable debugging support in the module, such as in:

$ make -DACPI_DEBUG buildkernel

Specifying ACPI_DEBUG in the kernel config alone did not enable it, but
our modules are supposed to honor kernel options. Also this was contrary
to what the manual page says.

So to make this work for ACPI_DEBUG too, we just put all the affected
source files into SRCS and always compile them. #ifdef's in these
source files will take care of enabling/disabling debugging support
so a module compiled without ACPI_DEBUG defined in the kernel or on the
command line will still not have support after this commit (I've checked
with nm(1)).

The only change for someone not using ACPI_DEBUG is a little bit of
additional buildkernel time.

FWIW, it is the same way in FreeBSD, too.

Reported-by: Andrea Magliano <masterblaster@tiscali.it>
sys/dev/acpica5/Makefile