diff options
author | Sergei Shtylyov <sshtylyov@ru.mvista.com> | 2008-02-01 17:09:30 -0500 |
---|---|---|
committer | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2008-02-01 17:09:30 -0500 |
commit | ecf32796395ed0e27667e7f735946d6dc60e1765 (patch) | |
tree | 0cafd743bd39148b85470c8fb6398b30dbce7ffd /drivers | |
parent | 4e5a68aeb6b513ee0adda11642e10e528b7721c0 (diff) |
ide: ide_setup_dma() assumes 8 ports
According to http://marc.info/?l=linux-ide&m=114346138611631, the drivers must
always register 8 DMA ports with ide_setup_dma(), so its last argument is not
needed. While at it, kill some useless parens in that function...
Signed-off-by: Sergei Shtylyov <sshtylyov@ru.mvista.com>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/ide/ide-dma.c | 40 | ||||
-rw-r--r-- | drivers/ide/pci/alim15x3.c | 2 | ||||
-rw-r--r-- | drivers/ide/pci/hpt366.c | 2 | ||||
-rw-r--r-- | drivers/ide/pci/pdc202xx_old.c | 4 | ||||
-rw-r--r-- | drivers/ide/setup-pci.c | 2 |
5 files changed, 25 insertions, 25 deletions
diff --git a/drivers/ide/ide-dma.c b/drivers/ide/ide-dma.c index 8d668a80adb3..51c86bc241ac 100644 --- a/drivers/ide/ide-dma.c +++ b/drivers/ide/ide-dma.c | |||
@@ -889,19 +889,19 @@ static int ide_allocate_dma_engine(ide_hwif_t *hwif) | |||
889 | return 1; | 889 | return 1; |
890 | } | 890 | } |
891 | 891 | ||
892 | static int ide_mapped_mmio_dma(ide_hwif_t *hwif, unsigned long base, unsigned int ports) | 892 | static int ide_mapped_mmio_dma(ide_hwif_t *hwif, unsigned long base) |
893 | { | 893 | { |
894 | printk(KERN_INFO " %s: MMIO-DMA ", hwif->name); | 894 | printk(KERN_INFO " %s: MMIO-DMA ", hwif->name); |
895 | 895 | ||
896 | return 0; | 896 | return 0; |
897 | } | 897 | } |
898 | 898 | ||
899 | static int ide_iomio_dma(ide_hwif_t *hwif, unsigned long base, unsigned int ports) | 899 | static int ide_iomio_dma(ide_hwif_t *hwif, unsigned long base) |
900 | { | 900 | { |
901 | printk(KERN_INFO " %s: BM-DMA at 0x%04lx-0x%04lx", | 901 | printk(KERN_INFO " %s: BM-DMA at 0x%04lx-0x%04lx", |
902 | hwif->name, base, base + ports - 1); | 902 | hwif->name, base, base + 7); |
903 | 903 | ||
904 | if (!request_region(base, ports, hwif->name)) { | 904 | if (!request_region(base, 8, hwif->name)) { |
905 | printk(" -- Error, ports in use.\n"); | 905 | printk(" -- Error, ports in use.\n"); |
906 | return 1; | 906 | return 1; |
907 | } | 907 | } |
@@ -913,7 +913,7 @@ static int ide_iomio_dma(ide_hwif_t *hwif, unsigned long base, unsigned int port | |||
913 | if (!request_region(hwif->extra_base, | 913 | if (!request_region(hwif->extra_base, |
914 | hwif->cds->extra, hwif->cds->name)) { | 914 | hwif->cds->extra, hwif->cds->name)) { |
915 | printk(" -- Error, extra ports in use.\n"); | 915 | printk(" -- Error, extra ports in use.\n"); |
916 | release_region(base, ports); | 916 | release_region(base, 8); |
917 | return 1; | 917 | return 1; |
918 | } | 918 | } |
919 | hwif->extra_ports = hwif->cds->extra; | 919 | hwif->extra_ports = hwif->cds->extra; |
@@ -923,19 +923,19 @@ static int ide_iomio_dma(ide_hwif_t *hwif, unsigned long base, unsigned int port | |||
923 | return 0; | 923 | return 0; |
924 | } | 924 | } |
925 | 925 | ||
926 | static int ide_dma_iobase(ide_hwif_t *hwif, unsigned long base, unsigned int ports) | 926 | static int ide_dma_iobase(ide_hwif_t *hwif, unsigned long base) |
927 | { | 927 | { |
928 | if (hwif->mmio) | 928 | if (hwif->mmio) |
929 | return ide_mapped_mmio_dma(hwif, base,ports); | 929 | return ide_mapped_mmio_dma(hwif, base); |
930 | 930 | ||
931 | return ide_iomio_dma(hwif, base, ports); | 931 | return ide_iomio_dma(hwif, base); |
932 | } | 932 | } |
933 | 933 | ||
934 | void ide_setup_dma(ide_hwif_t *hwif, unsigned long base, unsigned num_ports) | 934 | void ide_setup_dma(ide_hwif_t *hwif, unsigned long base) |
935 | { | 935 | { |
936 | u8 dma_stat; | 936 | u8 dma_stat; |
937 | 937 | ||
938 | if (ide_dma_iobase(hwif, base, num_ports)) | 938 | if (ide_dma_iobase(hwif, base)) |
939 | return; | 939 | return; |
940 | 940 | ||
941 | if (ide_allocate_dma_engine(hwif)) { | 941 | if (ide_allocate_dma_engine(hwif)) { |
@@ -945,16 +945,16 @@ void ide_setup_dma(ide_hwif_t *hwif, unsigned long base, unsigned num_ports) | |||
945 | 945 | ||
946 | hwif->dma_base = base; | 946 | hwif->dma_base = base; |
947 | 947 | ||
948 | if (!(hwif->dma_command)) | 948 | if (!hwif->dma_command) |
949 | hwif->dma_command = hwif->dma_base; | 949 | hwif->dma_command = hwif->dma_base + 0; |
950 | if (!(hwif->dma_vendor1)) | 950 | if (!hwif->dma_vendor1) |
951 | hwif->dma_vendor1 = (hwif->dma_base + 1); | 951 | hwif->dma_vendor1 = hwif->dma_base + 1; |
952 | if (!(hwif->dma_status)) | 952 | if (!hwif->dma_status) |
953 | hwif->dma_status = (hwif->dma_base + 2); | 953 | hwif->dma_status = hwif->dma_base + 2; |
954 | if (!(hwif->dma_vendor3)) | 954 | if (!hwif->dma_vendor3) |
955 | hwif->dma_vendor3 = (hwif->dma_base + 3); | 955 | hwif->dma_vendor3 = hwif->dma_base + 3; |
956 | if (!(hwif->dma_prdtable)) | 956 | if (!hwif->dma_prdtable) |
957 | hwif->dma_prdtable = (hwif->dma_base + 4); | 957 | hwif->dma_prdtable = hwif->dma_base + 4; |
958 | 958 | ||
959 | if (!hwif->dma_host_set) | 959 | if (!hwif->dma_host_set) |
960 | hwif->dma_host_set = &ide_dma_host_set; | 960 | hwif->dma_host_set = &ide_dma_host_set; |
diff --git a/drivers/ide/pci/alim15x3.c b/drivers/ide/pci/alim15x3.c index 49aa82e412b6..9dd7cb4d07d4 100644 --- a/drivers/ide/pci/alim15x3.c +++ b/drivers/ide/pci/alim15x3.c | |||
@@ -745,7 +745,7 @@ static void __devinit init_dma_ali15x3 (ide_hwif_t *hwif, unsigned long dmabase) | |||
745 | return; | 745 | return; |
746 | if (!hwif->channel) | 746 | if (!hwif->channel) |
747 | outb(inb(dmabase + 2) & 0x60, dmabase + 2); | 747 | outb(inb(dmabase + 2) & 0x60, dmabase + 2); |
748 | ide_setup_dma(hwif, dmabase, 8); | 748 | ide_setup_dma(hwif, dmabase); |
749 | } | 749 | } |
750 | 750 | ||
751 | static const struct ide_port_info ali15x3_chipset __devinitdata = { | 751 | static const struct ide_port_info ali15x3_chipset __devinitdata = { |
diff --git a/drivers/ide/pci/hpt366.c b/drivers/ide/pci/hpt366.c index 12685939a813..d33a3f9c12b7 100644 --- a/drivers/ide/pci/hpt366.c +++ b/drivers/ide/pci/hpt366.c | |||
@@ -1413,7 +1413,7 @@ static void __devinit init_dma_hpt366(ide_hwif_t *hwif, unsigned long dmabase) | |||
1413 | 1413 | ||
1414 | local_irq_restore(flags); | 1414 | local_irq_restore(flags); |
1415 | 1415 | ||
1416 | ide_setup_dma(hwif, dmabase, 8); | 1416 | ide_setup_dma(hwif, dmabase); |
1417 | } | 1417 | } |
1418 | 1418 | ||
1419 | static void __devinit hpt374_init(struct pci_dev *dev, struct pci_dev *dev2) | 1419 | static void __devinit hpt374_init(struct pci_dev *dev, struct pci_dev *dev2) |
diff --git a/drivers/ide/pci/pdc202xx_old.c b/drivers/ide/pci/pdc202xx_old.c index 3a1e081fe390..ea66383c5579 100644 --- a/drivers/ide/pci/pdc202xx_old.c +++ b/drivers/ide/pci/pdc202xx_old.c | |||
@@ -334,7 +334,7 @@ static void __devinit init_dma_pdc202xx(ide_hwif_t *hwif, unsigned long dmabase) | |||
334 | u8 udma_speed_flag = 0, primary_mode = 0, secondary_mode = 0; | 334 | u8 udma_speed_flag = 0, primary_mode = 0, secondary_mode = 0; |
335 | 335 | ||
336 | if (hwif->channel) { | 336 | if (hwif->channel) { |
337 | ide_setup_dma(hwif, dmabase, 8); | 337 | ide_setup_dma(hwif, dmabase); |
338 | return; | 338 | return; |
339 | } | 339 | } |
340 | 340 | ||
@@ -358,7 +358,7 @@ static void __devinit init_dma_pdc202xx(ide_hwif_t *hwif, unsigned long dmabase) | |||
358 | } | 358 | } |
359 | #endif /* CONFIG_PDC202XX_BURST */ | 359 | #endif /* CONFIG_PDC202XX_BURST */ |
360 | 360 | ||
361 | ide_setup_dma(hwif, dmabase, 8); | 361 | ide_setup_dma(hwif, dmabase); |
362 | } | 362 | } |
363 | 363 | ||
364 | static void __devinit pdc202ata4_fixup_irq(struct pci_dev *dev, | 364 | static void __devinit pdc202ata4_fixup_irq(struct pci_dev *dev, |
diff --git a/drivers/ide/setup-pci.c b/drivers/ide/setup-pci.c index 676c66e72881..c473f45abd33 100644 --- a/drivers/ide/setup-pci.c +++ b/drivers/ide/setup-pci.c | |||
@@ -451,7 +451,7 @@ static void ide_hwif_setup_dma(struct pci_dev *dev, const struct ide_port_info * | |||
451 | if (d->init_dma) { | 451 | if (d->init_dma) { |
452 | d->init_dma(hwif, dma_base); | 452 | d->init_dma(hwif, dma_base); |
453 | } else { | 453 | } else { |
454 | ide_setup_dma(hwif, dma_base, 8); | 454 | ide_setup_dma(hwif, dma_base); |
455 | } | 455 | } |
456 | } else { | 456 | } else { |
457 | printk(KERN_INFO "%s: %s Bus-Master DMA disabled " | 457 | printk(KERN_INFO "%s: %s Bus-Master DMA disabled " |