diff options
author | Jeff Garzik <jeff@garzik.org> | 2007-08-15 05:38:46 -0400 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-10-12 14:55:34 -0400 |
commit | 6d32d30f55020d766388df7515f771f68c973033 (patch) | |
tree | ec0efe48460b5b3f177fed2698ccd04b7964359b /drivers/ata/pata_amd.c | |
parent | cb94c1cf5a6beffbd8935eb91227df0dd1987644 (diff) |
[libata] Remove ->irq_ack() hook, and ata_dummy_irq_on()
* ->irq_ack() is redundant to what the irq handler already
performs... chk-status + irq-clear. Furthermore, it is only
called in one place, when screaming-irq-debugging is enabled,
so we don't want to bother with a hook just for that.
* ata_dummy_irq_on() is only ever used in drivers that have
no callpath reaching ->irq_on(). Remove .irq_on hook from
those drivers, and the now-unused ata_dummy_irq_on()
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/ata/pata_amd.c')
-rw-r--r-- | drivers/ata/pata_amd.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/drivers/ata/pata_amd.c b/drivers/ata/pata_amd.c index d0cebe16a8ee..46f829c838b3 100644 --- a/drivers/ata/pata_amd.c +++ b/drivers/ata/pata_amd.c | |||
@@ -358,7 +358,6 @@ static struct ata_port_operations amd33_port_ops = { | |||
358 | .irq_handler = ata_interrupt, | 358 | .irq_handler = ata_interrupt, |
359 | .irq_clear = ata_bmdma_irq_clear, | 359 | .irq_clear = ata_bmdma_irq_clear, |
360 | .irq_on = ata_irq_on, | 360 | .irq_on = ata_irq_on, |
361 | .irq_ack = ata_irq_ack, | ||
362 | 361 | ||
363 | .port_start = ata_port_start, | 362 | .port_start = ata_port_start, |
364 | }; | 363 | }; |
@@ -393,7 +392,6 @@ static struct ata_port_operations amd66_port_ops = { | |||
393 | .irq_handler = ata_interrupt, | 392 | .irq_handler = ata_interrupt, |
394 | .irq_clear = ata_bmdma_irq_clear, | 393 | .irq_clear = ata_bmdma_irq_clear, |
395 | .irq_on = ata_irq_on, | 394 | .irq_on = ata_irq_on, |
396 | .irq_ack = ata_irq_ack, | ||
397 | 395 | ||
398 | .port_start = ata_port_start, | 396 | .port_start = ata_port_start, |
399 | }; | 397 | }; |
@@ -428,7 +426,6 @@ static struct ata_port_operations amd100_port_ops = { | |||
428 | .irq_handler = ata_interrupt, | 426 | .irq_handler = ata_interrupt, |
429 | .irq_clear = ata_bmdma_irq_clear, | 427 | .irq_clear = ata_bmdma_irq_clear, |
430 | .irq_on = ata_irq_on, | 428 | .irq_on = ata_irq_on, |
431 | .irq_ack = ata_irq_ack, | ||
432 | 429 | ||
433 | .port_start = ata_port_start, | 430 | .port_start = ata_port_start, |
434 | }; | 431 | }; |
@@ -463,7 +460,6 @@ static struct ata_port_operations amd133_port_ops = { | |||
463 | .irq_handler = ata_interrupt, | 460 | .irq_handler = ata_interrupt, |
464 | .irq_clear = ata_bmdma_irq_clear, | 461 | .irq_clear = ata_bmdma_irq_clear, |
465 | .irq_on = ata_irq_on, | 462 | .irq_on = ata_irq_on, |
466 | .irq_ack = ata_irq_ack, | ||
467 | 463 | ||
468 | .port_start = ata_port_start, | 464 | .port_start = ata_port_start, |
469 | }; | 465 | }; |
@@ -498,7 +494,6 @@ static struct ata_port_operations nv100_port_ops = { | |||
498 | .irq_handler = ata_interrupt, | 494 | .irq_handler = ata_interrupt, |
499 | .irq_clear = ata_bmdma_irq_clear, | 495 | .irq_clear = ata_bmdma_irq_clear, |
500 | .irq_on = ata_irq_on, | 496 | .irq_on = ata_irq_on, |
501 | .irq_ack = ata_irq_ack, | ||
502 | 497 | ||
503 | .port_start = ata_port_start, | 498 | .port_start = ata_port_start, |
504 | }; | 499 | }; |
@@ -533,7 +528,6 @@ static struct ata_port_operations nv133_port_ops = { | |||
533 | .irq_handler = ata_interrupt, | 528 | .irq_handler = ata_interrupt, |
534 | .irq_clear = ata_bmdma_irq_clear, | 529 | .irq_clear = ata_bmdma_irq_clear, |
535 | .irq_on = ata_irq_on, | 530 | .irq_on = ata_irq_on, |
536 | .irq_ack = ata_irq_ack, | ||
537 | 531 | ||
538 | .port_start = ata_port_start, | 532 | .port_start = ata_port_start, |
539 | }; | 533 | }; |