Martin Mares wrote:
>
> Hello!
>
> > * isa_xxx are only to ease the transition to newstyle ioremap. if you
> > are hacking code, go ahead and update it to use
> > ioremap/readX/writeX/iounmap.
>
> Not only for that -- they are IMHO just fine for things like probing
> for devices where perpetual ioremap'ing and iounmap'ing the regions you
> want to probe just complicates the code and brings nothing useful.
IMNSHO.... not really :) If you want that type of feature, you should
make yourself a helper function instead
extern inline mj_writeb (u8 val, unsigned long phys_addr) {
void *addr = ioremap (phys_addr, 1);
if (addr) {
writeb (val, addr);
iounmap (addr);
}
}
Using isa_xxx during init code is not a good idea because it encourages
assumptions which are not portable. isa_xxx just happens to work with
32-bit PCI memory, for example, but (IIRC) it will not work on anywhere
but x86.
Jeff
-- Jeff Garzik | "Vegetarian" is the Indian word Building 1024 | for 'lousy hunter.' MandrakeSoft, Inc. |- 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/
This archive was generated by hypermail 2b29 : Tue Feb 15 2000 - 21:00:21 EST