diff options
author | Sergei Shtylyov <sshtylyov@ru.mvista.com> | 2007-07-09 17:17:54 -0400 |
---|---|---|
committer | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2007-07-09 17:17:54 -0400 |
commit | c283f5dbe31920ca70b80a594a97bfaa2a28be13 (patch) | |
tree | 891268bfec27d88bb9c5773026dcf950afe063d9 /drivers/ide/pci | |
parent | 841d2a9bf16471716ba3a5172d24aa40a2ea9398 (diff) |
ide: make void and rename ide_dma_timeout() method
Since ide_dma_timeout() method's result is discarded, make it return 'void'.
While at it, drop 'ide_' from the method's name, drop the '__' prefix from
the default method's name, and do some cleanups in this method driver-wise:
- in ide-dma.c, au1xxx-ide.c, and pdc202xx_old.c, define/use 'hwif' variable;
- in au1xxx-ide.c, get rid of commented out printk();
- in sl82c105.c, get rid of unnecessary variables.
Signed-off-by: Sergei Shtylyov <sshtylyov@ru.mvista.com>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Diffstat (limited to 'drivers/ide/pci')
-rw-r--r-- | drivers/ide/pci/hpt366.c | 6 | ||||
-rw-r--r-- | drivers/ide/pci/pdc202xx_old.c | 13 | ||||
-rw-r--r-- | drivers/ide/pci/sgiioc4.c | 2 | ||||
-rw-r--r-- | drivers/ide/pci/sl82c105.c | 13 |
4 files changed, 17 insertions, 17 deletions
diff --git a/drivers/ide/pci/hpt366.c b/drivers/ide/pci/hpt366.c index 34d06dc7f2b2..65018814ed73 100644 --- a/drivers/ide/pci/hpt366.c +++ b/drivers/ide/pci/hpt366.c | |||
@@ -799,10 +799,10 @@ static int hpt370_ide_dma_end(ide_drive_t *drive) | |||
799 | return __ide_dma_end(drive); | 799 | return __ide_dma_end(drive); |
800 | } | 800 | } |
801 | 801 | ||
802 | static int hpt370_ide_dma_timeout(ide_drive_t *drive) | 802 | static void hpt370_dma_timeout(ide_drive_t *drive) |
803 | { | 803 | { |
804 | hpt370_irq_timeout(drive); | 804 | hpt370_irq_timeout(drive); |
805 | return __ide_dma_timeout(drive); | 805 | ide_dma_timeout(drive); |
806 | } | 806 | } |
807 | 807 | ||
808 | /* returns 1 if DMA IRQ issued, 0 otherwise */ | 808 | /* returns 1 if DMA IRQ issued, 0 otherwise */ |
@@ -1353,7 +1353,7 @@ static void __devinit init_hwif_hpt366(ide_hwif_t *hwif) | |||
1353 | } else if (chip_type >= HPT370) { | 1353 | } else if (chip_type >= HPT370) { |
1354 | hwif->dma_start = &hpt370_ide_dma_start; | 1354 | hwif->dma_start = &hpt370_ide_dma_start; |
1355 | hwif->ide_dma_end = &hpt370_ide_dma_end; | 1355 | hwif->ide_dma_end = &hpt370_ide_dma_end; |
1356 | hwif->ide_dma_timeout = &hpt370_ide_dma_timeout; | 1356 | hwif->dma_timeout = &hpt370_dma_timeout; |
1357 | } else | 1357 | } else |
1358 | hwif->dma_lost_irq = &hpt366_dma_lost_irq; | 1358 | hwif->dma_lost_irq = &hpt366_dma_lost_irq; |
1359 | 1359 | ||
diff --git a/drivers/ide/pci/pdc202xx_old.c b/drivers/ide/pci/pdc202xx_old.c index 5013df84e971..ae628b86e18a 100644 --- a/drivers/ide/pci/pdc202xx_old.c +++ b/drivers/ide/pci/pdc202xx_old.c | |||
@@ -277,11 +277,14 @@ static void pdc202xx_dma_lost_irq(ide_drive_t *drive) | |||
277 | ide_dma_lost_irq(drive); | 277 | ide_dma_lost_irq(drive); |
278 | } | 278 | } |
279 | 279 | ||
280 | static int pdc202xx_ide_dma_timeout(ide_drive_t *drive) | 280 | static void pdc202xx_dma_timeout(ide_drive_t *drive) |
281 | { | 281 | { |
282 | if (HWIF(drive)->resetproc != NULL) | 282 | ide_hwif_t *hwif = HWIF(drive); |
283 | HWIF(drive)->resetproc(drive); | 283 | |
284 | return __ide_dma_timeout(drive); | 284 | if (hwif->resetproc != NULL) |
285 | hwif->resetproc(drive); | ||
286 | |||
287 | ide_dma_timeout(drive); | ||
285 | } | 288 | } |
286 | 289 | ||
287 | static void pdc202xx_reset_host (ide_hwif_t *hwif) | 290 | static void pdc202xx_reset_host (ide_hwif_t *hwif) |
@@ -351,7 +354,7 @@ static void __devinit init_hwif_pdc202xx(ide_hwif_t *hwif) | |||
351 | 354 | ||
352 | hwif->ide_dma_check = &pdc202xx_config_drive_xfer_rate; | 355 | hwif->ide_dma_check = &pdc202xx_config_drive_xfer_rate; |
353 | hwif->dma_lost_irq = &pdc202xx_dma_lost_irq; | 356 | hwif->dma_lost_irq = &pdc202xx_dma_lost_irq; |
354 | hwif->ide_dma_timeout = &pdc202xx_ide_dma_timeout; | 357 | hwif->dma_timeout = &pdc202xx_dma_timeout; |
355 | 358 | ||
356 | if (hwif->pci_dev->device != PCI_DEVICE_ID_PROMISE_20246) { | 359 | if (hwif->pci_dev->device != PCI_DEVICE_ID_PROMISE_20246) { |
357 | if (!(hwif->udma_four)) | 360 | if (!(hwif->udma_four)) |
diff --git a/drivers/ide/pci/sgiioc4.c b/drivers/ide/pci/sgiioc4.c index 53bee59264ef..d396b2929ed8 100644 --- a/drivers/ide/pci/sgiioc4.c +++ b/drivers/ide/pci/sgiioc4.c | |||
@@ -608,7 +608,7 @@ ide_init_sgiioc4(ide_hwif_t * hwif) | |||
608 | hwif->dma_host_on = &sgiioc4_dma_host_on; | 608 | hwif->dma_host_on = &sgiioc4_dma_host_on; |
609 | hwif->dma_host_off = &sgiioc4_dma_host_off; | 609 | hwif->dma_host_off = &sgiioc4_dma_host_off; |
610 | hwif->dma_lost_irq = &sgiioc4_dma_lost_irq; | 610 | hwif->dma_lost_irq = &sgiioc4_dma_lost_irq; |
611 | hwif->ide_dma_timeout = &__ide_dma_timeout; | 611 | hwif->dma_timeout = &ide_dma_timeout; |
612 | 612 | ||
613 | hwif->INB = &sgiioc4_INB; | 613 | hwif->INB = &sgiioc4_INB; |
614 | } | 614 | } |
diff --git a/drivers/ide/pci/sl82c105.c b/drivers/ide/pci/sl82c105.c index 202ce4965b60..487879842af4 100644 --- a/drivers/ide/pci/sl82c105.c +++ b/drivers/ide/pci/sl82c105.c | |||
@@ -241,15 +241,12 @@ static void sl82c105_dma_start(ide_drive_t *drive) | |||
241 | ide_dma_start(drive); | 241 | ide_dma_start(drive); |
242 | } | 242 | } |
243 | 243 | ||
244 | static int sl82c105_ide_dma_timeout(ide_drive_t *drive) | 244 | static void sl82c105_dma_timeout(ide_drive_t *drive) |
245 | { | 245 | { |
246 | ide_hwif_t *hwif = HWIF(drive); | 246 | DBG(("sl82c105_dma_timeout(drive:%s)\n", drive->name)); |
247 | struct pci_dev *dev = hwif->pci_dev; | ||
248 | 247 | ||
249 | DBG(("sl82c105_ide_dma_timeout(drive:%s)\n", drive->name)); | 248 | sl82c105_reset_host(HWIF(drive)->pci_dev); |
250 | 249 | ide_dma_timeout(drive); | |
251 | sl82c105_reset_host(dev); | ||
252 | return __ide_dma_timeout(drive); | ||
253 | } | 250 | } |
254 | 251 | ||
255 | static int sl82c105_ide_dma_on(ide_drive_t *drive) | 252 | static int sl82c105_ide_dma_on(ide_drive_t *drive) |
@@ -440,7 +437,7 @@ static void __devinit init_hwif_sl82c105(ide_hwif_t *hwif) | |||
440 | hwif->dma_off_quietly = &sl82c105_dma_off_quietly; | 437 | hwif->dma_off_quietly = &sl82c105_dma_off_quietly; |
441 | hwif->dma_lost_irq = &sl82c105_dma_lost_irq; | 438 | hwif->dma_lost_irq = &sl82c105_dma_lost_irq; |
442 | hwif->dma_start = &sl82c105_dma_start; | 439 | hwif->dma_start = &sl82c105_dma_start; |
443 | hwif->ide_dma_timeout = &sl82c105_ide_dma_timeout; | 440 | hwif->dma_timeout = &sl82c105_dma_timeout; |
444 | 441 | ||
445 | if (!noautodma) | 442 | if (!noautodma) |
446 | hwif->autodma = 1; | 443 | hwif->autodma = 1; |