On Fri, 2005-02-11 at 21:58 -0800, Grant Grundler wrote:
> And sdp_main.h violates one of the kernel include file rules:
>       include asm/ headers *after* linux/ headers.

Include asm/ headers *after* linux/ headers.  This builds for me on x86,
x86_64, and sparc64.

Signed-off-by: Tom Duffy <[EMAIL PROTECTED]>

Index: drivers/infiniband/ulp/sdp/sdp_main.h
===================================================================
--- drivers/infiniband/ulp/sdp/sdp_main.h       (revision 1783)
+++ drivers/infiniband/ulp/sdp/sdp_main.h       (working copy)
@@ -57,11 +57,6 @@
 /*
  * kernel includes
  */
-#include <asm/atomic.h>
-#include <asm/byteorder.h>
-#include <asm/pgtable.h>
-#include <asm/io.h>
-
 #include <linux/version.h>
 #include <linux/slab.h>
 #include <linux/spinlock.h>
@@ -85,6 +80,12 @@
 #include <net/route.h>
 #include <net/dst.h>
 #include <net/ip.h>
+
+#include <asm/atomic.h>
+#include <asm/byteorder.h>
+#include <asm/pgtable.h>
+#include <asm/io.h>
+
 /*
  * IB includes
  */
@@ -107,7 +108,3 @@
 #include "sdp_buff_p.h"
 
 #endif /* _SDP_MAIN_H */
-
-
-
-

_______________________________________________
openib-general mailing list
[email protected]
http://openib.org/mailman/listinfo/openib-general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to