Skip to contentSkip to conversation

ide: include <linux/hdreg.h> only when needed

PATCH New Bartlomiej Zolnierkiewicz 1 participant 1 message Jul 27, 2008

Subsystems (2)
SCSI libata
lore
Share & tools
lore

Bartlomiej Zolnierkiewicz

author
#0 lore raw

PATCH 1/3 ide: include <linux/hdreg.h> only when needed

* Include <linux/ata.h> directly in <linux/ide.h>
instead of through <linux/hdreg.h>.

* Include <linux/hdreg.h> only when needed.

Cc Christoph Hellwig <[email protected]>
Signed-off-by Bartlomiej Zolnierkiewicz <[email protected]>

---
reworked patchset, no longer removes <linux/hdreg.h> or moves the code around needlessly

 drivers/ide/arm/icside.c       |    1 -
 drivers/ide/arm/palm_bk3710.c  |    1 -
 drivers/ide/ide-disk.c         |    1 +
 drivers/ide/ide-floppy.c       |    1 +
 drivers/ide/ide-io.c           |    1 +
 drivers/ide/ide-iops.c         |    1 -
 drivers/ide/ide-lib.c          |    1 -
 drivers/ide/ide-proc.c         |    1 -
 drivers/ide/ide-timings.c      |    1 -
 drivers/ide/ide.c              |    1 +
 drivers/ide/legacy/ali14xx.c   |    1 -
 drivers/ide/legacy/buddha.c    |    1 -
 drivers/ide/legacy/dtc2278.c   |    1 -
 drivers/ide/legacy/falconide.c |    1 -
 drivers/ide/legacy/gayle.c     |    1 -
 drivers/ide/legacy/ht6560b.c   |    1 -
 drivers/ide/legacy/ide-cs.c    |    1 -
 drivers/ide/legacy/macide.c    |    1 -
 drivers/ide/legacy/q40ide.c    |    2 --
 drivers/ide/legacy/qd65xx.c    |    1 -
 drivers/ide/legacy/umc8672.c   |    1 -
 drivers/ide/pci/aec62xx.c      |    1 -
 drivers/ide/pci/alim15x3.c     |    1 -
 drivers/ide/pci/atiixp.c       |    1 -
 drivers/ide/pci/cmd640.c       |    1 -
 drivers/ide/pci/cmd64x.c       |    1 -
 drivers/ide/pci/cs5520.c       |    1 -
 drivers/ide/pci/cs5530.c       |    1 -
 drivers/ide/pci/delkin_cb.c    |    1 -
 drivers/ide/pci/generic.c      |    1 -
 drivers/ide/pci/hpt34x.c       |    1 -
 drivers/ide/pci/hpt366.c       |    1 -
 drivers/ide/pci/it8213.c       |    1 -
 drivers/ide/pci/it821x.c       |    1 -
 drivers/ide/pci/jmicron.c      |    1 -
 drivers/ide/pci/ns87415.c      |    1 -
 drivers/ide/pci/opti621.c      |    1 -
 drivers/ide/pci/pdc202xx_new.c |    1 -
 drivers/ide/pci/pdc202xx_old.c |    1 -
 drivers/ide/pci/piix.c         |    1 -
 drivers/ide/pci/rz1000.c       |    1 -
 drivers/ide/pci/sc1200.c       |    1 -
 drivers/ide/pci/scc_pata.c     |    1 -
 drivers/ide/pci/serverworks.c  |    1 -
 drivers/ide/pci/sgiioc4.c      |    1 -
 drivers/ide/pci/siimage.c      |    1 -
 drivers/ide/pci/sis5513.c      |    1 -
 drivers/ide/pci/sl82c105.c     |    1 -
 drivers/ide/pci/slc90e66.c     |    1 -
 drivers/ide/pci/triflex.c      |    1 -
 drivers/ide/pci/trm290.c       |    1 -
 drivers/scsi/ide-scsi.c        |    1 -
 include/linux/ide.h            |    2 +-
 53 files changed, 5 insertions(+), 50 deletions(-)
drivers/ide/arm/icside.c +0 -1
Index: b/drivers/ide/arm/icside.c
===================================================================
@@ -10,7 +10,6 @@
 #include <linux/slab.h>
 #include <linux/blkdev.h>
 #include <linux/errno.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/dma-mapping.h>
 #include <linux/device.h>
drivers/ide/arm/palm_bk3710.c +0 -1
Index: b/drivers/ide/arm/palm_bk3710.c
===================================================================
@@ -27,7 +27,6 @@
 #include <linux/module.h>
 #include <linux/kernel.h>
 #include <linux/ioport.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/delay.h>
 #include <linux/init.h>
drivers/ide/ide-disk.c +1 -0
Index: b/drivers/ide/ide-disk.c
===================================================================
@@ -31,6 +31,7 @@
 #include <linux/mutex.h>
 #include <linux/leds.h>
 #include <linux/ide.h>
+#include <linux/hdreg.h>
 
 #include <asm/byteorder.h>
 #include <asm/irq.h>
drivers/ide/ide-floppy.c +1 -0
Index: b/drivers/ide/ide-floppy.c
===================================================================
@@ -31,6 +31,7 @@
 #include <linux/slab.h>
 #include <linux/cdrom.h>
 #include <linux/ide.h>
+#include <linux/hdreg.h>
 #include <linux/bitops.h>
 #include <linux/mutex.h>
drivers/ide/ide-io.c +1 -0
Index: b/drivers/ide/ide-io.c
===================================================================
@@ -40,6 +40,7 @@
 #include <linux/pci.h>
 #include <linux/delay.h>
 #include <linux/ide.h>
+#include <linux/hdreg.h>
 #include <linux/completion.h>
 #include <linux/reboot.h>
 #include <linux/cdrom.h>
drivers/ide/ide-iops.c +0 -1
Index: b/drivers/ide/ide-iops.c
===================================================================
@@ -18,7 +18,6 @@
 #include <linux/slab.h>
 #include <linux/pci.h>
 #include <linux/delay.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/bitops.h>
 #include <linux/nmi.h>
drivers/ide/ide-lib.c +0 -1
Index: b/drivers/ide/ide-lib.c
===================================================================
@@ -2,7 +2,6 @@
 #include <linux/string.h>
 #include <linux/kernel.h>
 #include <linux/interrupt.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/bitops.h>
drivers/ide/ide-proc.c +0 -1
Index: b/drivers/ide/ide-proc.c
===================================================================
@@ -23,7 +23,6 @@
 #include <linux/mm.h>
 #include <linux/pci.h>
 #include <linux/ctype.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/seq_file.h>
drivers/ide/ide-timings.c +0 -1
Index: b/drivers/ide/ide-timings.c
===================================================================
@@ -22,7 +22,6 @@
  */
 
 #include <linux/kernel.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/module.h>
drivers/ide/ide.c +1 -0
Index: b/drivers/ide/ide.c
===================================================================
@@ -56,6 +56,7 @@
 #include <linux/init.h>
 #include <linux/pci.h>
 #include <linux/ide.h>
+#include <linux/hdreg.h>
 #include <linux/completion.h>
 #include <linux/device.h>
drivers/ide/legacy/ali14xx.c +0 -1
Index: b/drivers/ide/legacy/ali14xx.c
===================================================================
@@ -43,7 +43,6 @@
 #include <linux/mm.h>
 #include <linux/ioport.h>
 #include <linux/blkdev.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/legacy/buddha.c +0 -1
Index: b/drivers/ide/legacy/buddha.c
===================================================================
@@ -20,7 +20,6 @@
 #include <linux/mm.h>
 #include <linux/interrupt.h>
 #include <linux/blkdev.h>
-#include <linux/hdreg.h>
 #include <linux/zorro.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/legacy/dtc2278.c +0 -1
Index: b/drivers/ide/legacy/dtc2278.c
===================================================================
@@ -10,7 +10,6 @@
 #include <linux/mm.h>
 #include <linux/ioport.h>
 #include <linux/blkdev.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/legacy/falconide.c +0 -1
Index: b/drivers/ide/legacy/falconide.c
===================================================================
@@ -13,7 +13,6 @@
 #include <linux/mm.h>
 #include <linux/interrupt.h>
 #include <linux/blkdev.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/legacy/gayle.c +0 -1
Index: b/drivers/ide/legacy/gayle.c
===================================================================
@@ -12,7 +12,6 @@
 #include <linux/mm.h>
 #include <linux/interrupt.h>
 #include <linux/blkdev.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
 #include <linux/zorro.h>
drivers/ide/legacy/ht6560b.c +0 -1
Index: b/drivers/ide/legacy/ht6560b.c
===================================================================
@@ -24,7 +24,6 @@
 #include <linux/mm.h>
 #include <linux/ioport.h>
 #include <linux/blkdev.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/legacy/ide-cs.c +0 -1
Index: b/drivers/ide/legacy/ide-cs.c
===================================================================
@@ -38,7 +38,6 @@
 #include <linux/timer.h>
 #include <linux/ioport.h>
 #include <linux/ide.h>
-#include <linux/hdreg.h>
 #include <linux/major.h>
 #include <linux/delay.h>
 #include <asm/io.h>
drivers/ide/legacy/macide.c +0 -1
Index: b/drivers/ide/legacy/macide.c
===================================================================
@@ -15,7 +15,6 @@
 #include <linux/mm.h>
 #include <linux/interrupt.h>
 #include <linux/blkdev.h>
-#include <linux/hdreg.h>
 #include <linux/delay.h>
 #include <linux/ide.h>
drivers/ide/legacy/q40ide.c +0 -2
Index: b/drivers/ide/legacy/q40ide.c
===================================================================
@@ -14,8 +14,6 @@
 #include <linux/mm.h>
 #include <linux/interrupt.h>
 #include <linux/blkdev.h>
-#include <linux/hdreg.h>
-
 #include <linux/ide.h>
 
     /*
drivers/ide/legacy/qd65xx.c +0 -1
Index: b/drivers/ide/legacy/qd65xx.c
===================================================================
@@ -27,7 +27,6 @@
 #include <linux/mm.h>
 #include <linux/ioport.h>
 #include <linux/blkdev.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
 #include <asm/system.h>
drivers/ide/legacy/umc8672.c +0 -1
Index: b/drivers/ide/legacy/umc8672.c
===================================================================
@@ -45,7 +45,6 @@
 #include <linux/mm.h>
 #include <linux/ioport.h>
 #include <linux/blkdev.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/pci/aec62xx.c +0 -1
Index: b/drivers/ide/pci/aec62xx.c
===================================================================
@@ -7,7 +7,6 @@
 #include <linux/module.h>
 #include <linux/types.h>
 #include <linux/pci.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/pci/alim15x3.c +0 -1
Index: b/drivers/ide/pci/alim15x3.c
===================================================================
@@ -31,7 +31,6 @@
 #include <linux/types.h>
 #include <linux/kernel.h>
 #include <linux/pci.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
 #include <linux/dmi.h>
drivers/ide/pci/atiixp.c +0 -1
Index: b/drivers/ide/pci/atiixp.c
===================================================================
@@ -7,7 +7,6 @@
 #include <linux/module.h>
 #include <linux/kernel.h>
 #include <linux/pci.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/pci/cmd640.c +0 -1
Index: b/drivers/ide/pci/cmd640.c
===================================================================
@@ -103,7 +103,6 @@
 #include <linux/types.h>
 #include <linux/kernel.h>
 #include <linux/delay.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/pci/cmd64x.c +0 -1
Index: b/drivers/ide/pci/cmd64x.c
===================================================================
@@ -13,7 +13,6 @@
 #include <linux/module.h>
 #include <linux/types.h>
 #include <linux/pci.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/pci/cs5520.c +0 -1
Index: b/drivers/ide/pci/cs5520.c
===================================================================
@@ -35,7 +35,6 @@
 #include <linux/module.h>
 #include <linux/types.h>
 #include <linux/kernel.h>
-#include <linux/hdreg.h>
 #include <linux/init.h>
 #include <linux/pci.h>
 #include <linux/ide.h>
drivers/ide/pci/cs5530.c +0 -1
Index: b/drivers/ide/pci/cs5530.c
===================================================================
@@ -15,7 +15,6 @@
 #include <linux/module.h>
 #include <linux/types.h>
 #include <linux/kernel.h>
-#include <linux/hdreg.h>
 #include <linux/pci.h>
 #include <linux/init.h>
 #include <linux/ide.h>
drivers/ide/pci/delkin_cb.c +0 -1
Index: b/drivers/ide/pci/delkin_cb.c
===================================================================
@@ -19,7 +19,6 @@
 
 #include <linux/types.h>
 #include <linux/module.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
 #include <linux/pci.h>
drivers/ide/pci/generic.c +0 -1
Index: b/drivers/ide/pci/generic.c
===================================================================
@@ -22,7 +22,6 @@
 #include <linux/types.h>
 #include <linux/module.h>
 #include <linux/kernel.h>
-#include <linux/hdreg.h>
 #include <linux/pci.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/pci/hpt34x.c +0 -1
Index: b/drivers/ide/pci/hpt34x.c
===================================================================
@@ -27,7 +27,6 @@
 #include <linux/types.h>
 #include <linux/kernel.h>
 #include <linux/ioport.h>
-#include <linux/hdreg.h>
 #include <linux/interrupt.h>
 #include <linux/pci.h>
 #include <linux/init.h>
drivers/ide/pci/hpt366.c +0 -1
Index: b/drivers/ide/pci/hpt366.c
===================================================================
@@ -122,7 +122,6 @@
 #include <linux/kernel.h>
 #include <linux/delay.h>
 #include <linux/blkdev.h>
-#include <linux/hdreg.h>
 #include <linux/interrupt.h>
 #include <linux/pci.h>
 #include <linux/init.h>
drivers/ide/pci/it8213.c +0 -1
Index: b/drivers/ide/pci/it8213.c
===================================================================
@@ -10,7 +10,6 @@
 #include <linux/types.h>
 #include <linux/module.h>
 #include <linux/pci.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/pci/it821x.c +0 -1
Index: b/drivers/ide/pci/it821x.c
===================================================================
@@ -63,7 +63,6 @@
 #include <linux/types.h>
 #include <linux/module.h>
 #include <linux/pci.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/pci/jmicron.c +0 -1
Index: b/drivers/ide/pci/jmicron.c
===================================================================
@@ -8,7 +8,6 @@
 #include <linux/types.h>
 #include <linux/module.h>
 #include <linux/pci.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/pci/ns87415.c +0 -1
Index: b/drivers/ide/pci/ns87415.c
===================================================================
@@ -11,7 +11,6 @@
 #include <linux/types.h>
 #include <linux/kernel.h>
 #include <linux/interrupt.h>
-#include <linux/hdreg.h>
 #include <linux/pci.h>
 #include <linux/delay.h>
 #include <linux/ide.h>
drivers/ide/pci/opti621.c +0 -1
Index: b/drivers/ide/pci/opti621.c
===================================================================
@@ -85,7 +85,6 @@
 #include <linux/module.h>
 #include <linux/kernel.h>
 #include <linux/pci.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 
 #include <asm/io.h>
drivers/ide/pci/pdc202xx_new.c +0 -1
Index: b/drivers/ide/pci/pdc202xx_new.c
===================================================================
@@ -19,7 +19,6 @@
 #include <linux/types.h>
 #include <linux/kernel.h>
 #include <linux/delay.h>
-#include <linux/hdreg.h>
 #include <linux/pci.h>
 #include <linux/init.h>
 #include <linux/ide.h>
drivers/ide/pci/pdc202xx_old.c +0 -1
Index: b/drivers/ide/pci/pdc202xx_old.c
===================================================================
@@ -13,7 +13,6 @@
 #include <linux/kernel.h>
 #include <linux/delay.h>
 #include <linux/blkdev.h>
-#include <linux/hdreg.h>
 #include <linux/pci.h>
 #include <linux/init.h>
 #include <linux/ide.h>
drivers/ide/pci/piix.c +0 -1
Index: b/drivers/ide/pci/piix.c
===================================================================
@@ -48,7 +48,6 @@
 #include <linux/module.h>
 #include <linux/kernel.h>
 #include <linux/pci.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/pci/rz1000.c +0 -1
Index: b/drivers/ide/pci/rz1000.c
===================================================================
@@ -16,7 +16,6 @@
 #include <linux/types.h>
 #include <linux/module.h>
 #include <linux/kernel.h>
-#include <linux/hdreg.h>
 #include <linux/pci.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/pci/sc1200.c +0 -1
Index: b/drivers/ide/pci/sc1200.c
===================================================================
@@ -14,7 +14,6 @@
 #include <linux/module.h>
 #include <linux/types.h>
 #include <linux/kernel.h>
-#include <linux/hdreg.h>
 #include <linux/pci.h>
 #include <linux/init.h>
 #include <linux/ide.h>
drivers/ide/pci/scc_pata.c +0 -1
Index: b/drivers/ide/pci/scc_pata.c
===================================================================
@@ -26,7 +26,6 @@
 #include <linux/module.h>
 #include <linux/pci.h>
 #include <linux/delay.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/pci/serverworks.c +0 -1
Index: b/drivers/ide/pci/serverworks.c
===================================================================
@@ -32,7 +32,6 @@
 #include <linux/module.h>
 #include <linux/kernel.h>
 #include <linux/pci.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/pci/sgiioc4.c +0 -1
Index: b/drivers/ide/pci/sgiioc4.c
===================================================================
@@ -22,7 +22,6 @@
 #include <linux/types.h>
 #include <linux/pci.h>
 #include <linux/delay.h>
-#include <linux/hdreg.h>
 #include <linux/init.h>
 #include <linux/kernel.h>
 #include <linux/ioport.h>
drivers/ide/pci/siimage.c +0 -1
Index: b/drivers/ide/pci/siimage.c
===================================================================
@@ -39,7 +39,6 @@
 #include <linux/types.h>
 #include <linux/module.h>
 #include <linux/pci.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
 #include <linux/io.h>
drivers/ide/pci/sis5513.c +0 -1
Index: b/drivers/ide/pci/sis5513.c
===================================================================
@@ -47,7 +47,6 @@
 #include <linux/types.h>
 #include <linux/module.h>
 #include <linux/kernel.h>
-#include <linux/hdreg.h>
 #include <linux/pci.h>
 #include <linux/init.h>
 #include <linux/ide.h>
drivers/ide/pci/sl82c105.c +0 -1
Index: b/drivers/ide/pci/sl82c105.c
===================================================================
@@ -17,7 +17,6 @@
 #include <linux/types.h>
 #include <linux/module.h>
 #include <linux/kernel.h>
-#include <linux/hdreg.h>
 #include <linux/pci.h>
 #include <linux/ide.h>
drivers/ide/pci/slc90e66.c +0 -1
Index: b/drivers/ide/pci/slc90e66.c
===================================================================
@@ -11,7 +11,6 @@
 #include <linux/module.h>
 #include <linux/kernel.h>
 #include <linux/pci.h>
-#include <linux/hdreg.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/pci/triflex.c +0 -1
Index: b/drivers/ide/pci/triflex.c
===================================================================
@@ -28,7 +28,6 @@
 #include <linux/types.h>
 #include <linux/module.h>
 #include <linux/kernel.h>
-#include <linux/hdreg.h>
 #include <linux/pci.h>
 #include <linux/ide.h>
 #include <linux/init.h>
drivers/ide/pci/trm290.c +0 -1
Index: b/drivers/ide/pci/trm290.c
===================================================================
@@ -135,7 +135,6 @@
 #include <linux/interrupt.h>
 #include <linux/blkdev.h>
 #include <linux/init.h>
-#include <linux/hdreg.h>
 #include <linux/pci.h>
 #include <linux/ide.h>
drivers/scsi/ide-scsi.c +0 -1
Index: b/drivers/scsi/ide-scsi.c
===================================================================
@@ -40,7 +40,6 @@
 #include <linux/ioport.h>
 #include <linux/blkdev.h>
 #include <linux/errno.h>
-#include <linux/hdreg.h>
 #include <linux/slab.h>
 #include <linux/ide.h>
 #include <linux/scatterlist.h>
include/linux/ide.h +1 -1
Index: b/include/linux/ide.h
===================================================================
@@ -8,7 +8,7 @@
 
 #include <linux/init.h>
 #include <linux/ioport.h>
-#include <linux/hdreg.h>
+#include <linux/ata.h>
 #include <linux/blkdev.h>
 #include <linux/proc_fs.h>
 #include <linux/interrupt.h>