On Mon, 21 May 2007, Sam Ravnborg wrote: > rday wrote:
> > under the circumstances, is there *any* cleanup worth doing WRT to > > this issue? because of the fact that module.h currently includes > > moduleparam.h, developers have been able to get away with being > > incredibly sloppy in their includes. > > It is wortwhile to make module.h independent of moduleparam.h. > The MODULE_INFO stuff has nothing to do with module parameters. > > But keep the include so you do not break the > 500 drivers. ok, i can do that. but, at the very least, it might be worth it for some people to stop including moduleparam.h in their code when that code does nothing with module parameters. my scanning script picked out the following files under drivers/ that *seem* to be doing just that, so respective maintainers might want to toss those useless includes. (obviously, at the moment, *all* includes of moduleparam.h are useless, but the ones in the files below are *doubly* useless. :-) drivers/pcmcia/socket_sysfs.c drivers/pcmcia/au1000_generic.c drivers/net/macb.c drivers/net/ixp2000/ixpdev.c drivers/net/ixp2000/enp2611.c drivers/net/arm/ep93xx_eth.c drivers/net/irda/kingsun-sir.c drivers/scsi/aacraid/linit.c drivers/scsi/3w-9xxx.c drivers/scsi/arcmsr/arcmsr_hba.c drivers/scsi/megaraid/megaraid_sas.c drivers/scsi/3w-xxxx.c drivers/bluetooth/bt3c_cs.c drivers/bluetooth/btuart_cs.c drivers/bluetooth/bluecard_cs.c drivers/bluetooth/dtl1_cs.c drivers/acpi/processor_core.c drivers/serial/mpsc.c drivers/mmc/card/block.c drivers/mmc/host/at91_mci.c drivers/mmc/host/omap.c drivers/char/hvc_rtas.c drivers/char/hw_random/ixp4xx-rng.c drivers/char/mbcs.c drivers/char/watchdog/mtx-1_wdt.c drivers/usb/gadget/fsl_usb2_udc.c drivers/usb/atm/cxacru.c drivers/usb/host/sl811-hcd.c drivers/media/dvb/frontends/isl6421.c drivers/media/dvb/frontends/lnbp21.c drivers/media/dvb/frontends/dvb_dummy_fe.c drivers/media/common/ir-keymaps.c drivers/media/video/ov7670.c drivers/media/video/cx88/cx88-vp3054-i2c.c drivers/infiniband/hw/mthca/mthca_profile.c drivers/infiniband/hw/amso1100/c2_rnic.c drivers/infiniband/hw/amso1100/c2_provider.c drivers/infiniband/hw/cxgb3/iwch.c drivers/infiniband/hw/cxgb3/iwch_provider.c drivers/input/mouse/trackpoint.c drivers/input/serio/libps2.c drivers/w1/slaves/w1_ds2433.c drivers/w1/slaves/w1_therm.c drivers/w1/slaves/w1_smem.c drivers/s390/net/qeth_main.c the same could be said for these files under net/: net/netfilter/nf_conntrack_proto.c net/ipv6/netfilter/ip6t_LOG.c net/ipv4/netfilter/nf_nat_h323.c net/ipv4/netfilter/nf_nat_tftp.c net/iucv/iucv.c rday -- ======================================================================== Robert P. J. Day Linux Consulting, Training and Annoying Kernel Pedantry Waterloo, Ontario, CANADA http://fsdev.net/wiki/index.php?title=Main_Page ======================================================================== - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/