Steve Price writes:
> }
> +#ifdef __i386__
> sc->wb_btag = I386_BUS_SPACE_IO;
> +#endif
> +#ifdef __alpha__
> + sc->wb_btag = ALPHA_BUS_SPACE_IO;
> +#endif
> #else
> if (!(command & PCIM_CMD_MEMEN)) {
Just a minor comment.. anytime you have something like this, it's
always nice to do it this way instead:
#if defined(__i386__)
sc->wb_btag = I386_BUS_SPACE_IO;
#elif defined(__alpha__)
sc->wb_btag = ALPHA_BUS_SPACE_IO;
#else
#error Machine architecture unsupported
#endif
That way when somebody wants to add M68K support or whatever they
are alerted that they need to implement the new flag at compile
time instead of at panic time :-)
-Archie
___________________________________________________________________________
Archie Cobbs * Whistle Communications, Inc. * http://www.whistle.com
To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message
