Re: kern/53741: /dev/apm is not MI

[Martin Husemann <martin%duskware.de@localhost>]

Hmm, maybe we should just document this better. The code in question does:

#if (defined(__FreeBSD__) || defined(__FreeBSD_kernel__)) && (defined(i386) || defined(__i386__))
#include <machine/apm_bios.h>
#elif (defined(__OpenBSD__) || defined(__NetBSD__))
#include <sys/param.h>
#include <sys/ioctl.h>
#if defined(__sparc64__)
#include <sparc/apmvar.h>
#elif defined(__x86_64__)
#include <dev/apm/apmbios.h>
#include <dev/apm/apmio.h>
#else
#include <machine/apmvar.h>
#endif
#define APMDEVICE "/dev/apm"


... and as you point out, we can just collapse all of NetBSD in a MI
way and be done:

#ifdef __NetBSD__
#include <sys/param.h>
#include <sys/ioctl.h>
#include <dev/apm/apmbios.h>
#include <dev/apm/apmio.h>
#define APMDEVICE "/dev/apm"
#define _ACPI_APM_BATT_UNKNOWN  0xffff /* from sys/dev/acpi/acpi_apm.c */
#elif (defined(__FreeBSD__) || defined(__FreeBSD_kernel__)) && (defined(i386) || defined(__i386__))


and this compiles - I'll deal at the pkgsrc level.

Martin