On Tue, Mar 01, 2011 at 06:39:53PM +0100, Nicolas Kaiser wrote: > find /usr/src/linux/ -name "*.[ch]"| \ > xargs egrep "^[[:space:]]*struct [^[:space:])]*;"| \ > sort|uniq -d > > > /usr/src/linux/arch/arm/include/asm/elf.h:struct task_struct; > /usr/src/linux/arch/microblaze/include/asm/prom.h:struct pci_dev; > /usr/src/linux/arch/parisc/include/asm/dma-mapping.h:struct parisc_device; > /usr/src/linux/arch/powerpc/include/asm/prom.h:struct pci_dev; > /usr/src/linux/arch/s390/include/asm/ccwdev.h:struct ccw1; > /usr/src/linux/arch/s390/kernel/compat_linux.h:struct old_sigaction32; > /usr/src/linux/arch/x86/include/asm/io_apic.h:struct io_apic_irq_attr; I just looked through some of these at random. For this one removing either struct io_apic_irq_attr will break the compile. It depends on how CONFIG_X86_IO_APIC is configured if it will break *your* compile or not. Sending cleanup patches that break the compile makes people really angry, so be careful. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
