Skip to contentSkip to conversation

ide.h fix(es)

PATCH In review Brian Murphy 2 participants 2 messages Jun 20, 2003 – Jun 21, 2003 MIPS

lore
Share & tools
lore
#0 lore raw

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
diff -u -r1.21 ide.h
@@ -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__ */
 
#1 lore raw
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