From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: [PATCH] Do not include linux/irq.h from linux/netpoll.h Date: Mon, 15 Mar 2004 16:52:50 +0000 Sender: netdev-bounce@oss.sgi.com Message-ID: <1079369568.19012.100.camel@icampbell-debian> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Cc: Linux Kernel Mailing List Return-path: To: netdev@oss.sgi.com Errors-to: netdev-bounce@oss.sgi.com List-Id: netdev.vger.kernel.org Hi, (I hope netdev is the right place for netpoll stuff) It seems that the changeset at http://www.kernel.org/pub/linux/kernel/v2.6/testing/cset/cset-jgarzik@redhat.com|ChangeSet|20040302073919|27676.txt breaks the current BK tree build for ARM. The culprit would appear to be the addition of a #include to net/core/dev.c which in turn pulls in which (as Russell King notes in a comment therein) should not be included from generic code. >>From what I can tell from the netpoll code used to call the drivers irq handler to simulate a poll but now it uses the poll_controller function, therefore I don't think the linux/irq.h needs to be included any longer. The patch below removes the include. I successfully built an ARM kernel with NETCONSOLE and NETPOLL enabled, although I was not able to test it since my network driver has no poll method. Cheers, Ian. Index: linux-2.6-bkpxa/include/linux/netpoll.h =================================================================== --- linux-2.6-bkpxa.orig/include/linux/netpoll.h 2004-03-15 15:03:30.000000000 +0000 +++ linux-2.6-bkpxa/include/linux/netpoll.h 2004-03-15 16:24:25.000000000 +0000 @@ -9,7 +9,6 @@ #include #include -#include #include struct netpoll; -- Ian Campbell, Senior Design Engineer Web: http://www.arcom.com Arcom, Clifton Road, Direct: +44 (0)1223 403 465 Cambridge CB1 7EA, United Kingdom Phone: +44 (0)1223 411 200