Messages in this thread | | | From | (Nick Holloway) | Subject | Re: v2.3.17pre1 - Patches, Complaints, Questions and Jubilations | Date | 5 Sep 1999 20:51:35 +0100 |
| |
alan@lxorguk.ukuu.org.uk (Alan Cox) writes: > > > >+ /* Suppress compiler warning. See EI_SHIFT in 8390.h for details */ > > > >+ (void)ei_local; > > > Or use __attribute__ ((__unused__)) on the declaration. > > Even better, just wrap the definition with #ifndef __i386__ > > All of which just looks ugly. So I leave it as is
Merely pimples on the great pustulating boil.
It is far from obvious (proven empirically) that E8390_CMD (et al) use EI_SHIFT, which in turn _may_ require that "ei_local->reg_offset[]" to be valid in the current scope.
IMHO.
-- `O O' | Home: Nick.Holloway@alfie.demon.co.uk http://www.alfie.demon.co.uk/ // ^ \\ | Work: Nick.Holloway@parallax.co.uk
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu Please read the FAQ at http://www.tux.org/lkml/
| |