Add comments after endifs

* device/cons.c [MACH_KMSG]: Likewise.
[CONSBUFSIZE > 0]: Likewise.
* i386/i386/trap.c [MACH_KDB]: Likewise.
[MACH_PV_PAGETABLES]: Likewise.
* i386/i386at/kd.c [ENABLE_IMMEDIATE_CONSOLE]: Likewise.
* ipc/ipc_kmsg_queue.h [_IPC_KMSG_QUEUE_H_]: Likewise.
* kern/act.c [ACTWATCH]: Likewise.
* kern/refcount.h [MACHINE_REFCOUNT]: Likewise.
* kern/task.c [FAST_TAS]: Likewise.
Add ifndefs

* i386/i386at/cram.h [_CRAM_H_]: Add ifndef.
* i386/i386at/disk.h [_DISK_H_]: Likewise.
* i386/i386at/i8250.h [_I8250_H_]: Likewise.
* i386/include/mach/i386/asm.h [_MACH_I386_ASM_H_]: Likewise.
* i386/include/mach/i386/disk.h [_MACH_I386_DISK_H_]: Likewise.
device/device_types_kernel.h: remove redeclaration of dev_port_lookup()

* device/device_types_kernel.h (dev_port_lookup): Remove prototype.
Fix implicit declaration of function

Thanks Marin Ramesa for the report.

* device/ds_routines.h (device_deallocate): Move declaration to...
* include/device/device_types.h (device_deallocate): ... here.
Always make Assert trigger debugger

* kern/debug.c (Assert): Always call Debugger, even if db_breakpoints_insert
is not set.
Make unsigned character constants unsigned

Thanks Marin Ramesa for the report.

* i386/i386at/kd.h (K_DONE, NC, K_SCAN): Make constants unsigned.
ipc/ipc_port.c: trivial stylistic fix for consistency

* ipc/ipc_port.c (indent): Trivial stylistic fix for consistency.
kern/machine.c: remove __volatile__

Shutdown can be optimised. Remove __volatile__.

* kern/machine.c [__GNUC__] (processor_doshutdown): Remove volatile function qualifier.
i386/i386/setjmp.h: remove ifdef

Function _longjmp() is used even if __GNUC__ is not defined. Avoid
implicit declaration in that case by removing the ifdef.

* i386/i386/setjmp.h [__GNUC__] (_longjmp): Remove ifdef.
ddb/db_command.c: remove forward declaration

* ddb/db_command.c (db_skip_to_eol): Remove forward declaration.
ddb/db_command.h: remove duplicate variable declaration

* ddb/db_command.h (db_recover): Remove duplicate variable declaration.
ddb/db_output.c: remove call to nonexistent db_printf_enter()

* ddb/db_output.c [db_printf_enter] (db_printf_enter): Remove call to nonexistent function.
ddb/db_output.c: remove duplicate function

Function kdbprintf() and db_printf() are the same function. Remove
kdbprintf() and define kdbprintf to db_printf.

* ddb/db_output.c (kdbprintf): Remove function.
* ddb/db_output.h: Define kdbprintf to db_printf.
(kdbprintf): Remove prototype.
device/chario.c: use boolean instead of an int

* device/chario.c (pdma_default): Use boolean instead of an int.
device/cirbuf.c: use boolean instead of an int

* device/cirbuf.c (cb_check_enable): Use boolean instead of an int.
device/dev_lookup.c: remove unused lock

* device/dev_lookup.c (dev_port_lock): Remove unused lock.
device/if_ether.h: remove unused variable

* device/if_ether.h (etherbroadcastaddr): Remove unused variable.
* device/subrs.c (etherbroadcastaddr): Remove initialization.
device/kmsg.c: use boolean instead of an int

* device/kmsg.c (kmsg_in_use): Use boolean instead of an int.
device/kmsg.c: use boolean instead of an int

* device/kmsg.c (kmsg_init_done): Use boolean instead of an int.
device/tty.h, chario.h: move prototype

Now that we have device/chario.h move the chario_init() prototype
from device/tty.h to device/chario.h.

* device/chario.h (chario_init): Add prototype.
* device/device_init.c: Include device/chario.h.
* device/tty.h (chario_init): Remove prototype.