ide.h fix(es)
PATCH 1/1 ide.h fix(es)
Hi Ralf,
this fixes some problems with undefined symbols in IDE and also seems to work (as far as my limited testing can go at the moment). Sorry if I was out of order in removing the big endian stuff, I just couldn't see the point.
/Brian
Index: include/asm-mips/ide.h
===================================================================
RCS file: /cvs/linux/include/asm-mips/ide.h,v
retrieving revision 1.21
asm-mips/ide.h +4 -22
@@ -64,28 +64,10 @@ #endif } -#ifdef __BIG_ENDIAN - -/* get rid of defs from io.h - ide has its private and conflicting versions */ -#ifdef insw -#undef insw -#endif -#ifdef outsw -#undef outsw -#endif -#ifdef insl -#undef insl -#endif -#ifdef outsl -#undef outsl -#endif - -#define insw (port, addr, count) ide_insw(port, addr, count) -#define insl (port, addr, count) ide_insl(port, addr, count) -#define outsw (port, addr, count) ide_outsw(port, addr, count) -#define outsl (port, addr, count) ide_outsl(port, addr, count) - -#endif /* __BIG_ENDIAN */ +#define __ide_mm_insw ide_insw +#define __ide_mm_insl ide_insl +#define __ide_mm_outsw ide_outsw +#define __ide_mm_outsl ide_outsl#endif /* __KERNEL__ */
Quoted 3 lines · Brian Murphy
this fixes some problems with undefined symbols in IDE and also seems to work (as far as my limited testing can go at the moment). Sorry if I was out of order in removing the big endian stuff,
The big endian stuff was just old code, so from the point you were right with removing it. It's not an entirely correct solution though, on some MIPS IDE systems the OS has to do endianess conversion, on some the hardware takes care of it. IDE as a big endian thing that was made popular through the little endian PC platform has funny endianess issues at times ...
Ralf