>> 	struct foo { ... } __attribute__((__packed__));
>> to imply also __aligned__(1)
> A good reason to use bus_space_*() functions to access the device
> registers.

Except those are kernel-only.  The code in question at the moment is
entirely in userland (the kernel is doing nothing but mmap() support,
until I understand things enough to do more there), and much of it will
remain there for all expected further work....

> Though in this case just dropping __packed__ should be ok, since all
> fields are 32 bit ints anyway.

Not quite, but experience indicates that they can all be accessed as if
they were.  (Well, except for fields which are array-of-32bit, but
that's a quibble.)

/~\ The ASCII				der Mouse
\ / Ribbon Campaign
 X  Against HTML	       mouse@rodents.montreal.qc.ca
/ \ Email!	     7D C8 61 52 5D E7 2D 39  4E F1 31 3E E8 B3 27 4B