diff options
author | Jeff Garzik <jeff@garzik.org> | 2007-10-31 07:27:57 -0400 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-11-03 08:46:28 -0400 |
commit | 9465d5324834f1e99c1343b7bbdc5e6ac8c83f87 (patch) | |
tree | 25f74397534df87680ec730d9ccf0c24865efc6f | |
parent | a2962dd0967d7a16a907f1c63dcb7f83e3bb1795 (diff) |
ata/sata_fsl: Remove unnecessary SCR cases
SCRs in the driver map to the standard values found in include/linux/ata.h,
so no need for individual scr_read/scr_write case statements duplicating
the natural value.
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Signed-off-by: Li Yang <leoli@freescale.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
-rw-r--r-- | drivers/ata/sata_fsl.c | 16 |
1 files changed, 2 insertions, 14 deletions
diff --git a/drivers/ata/sata_fsl.c b/drivers/ata/sata_fsl.c index 9e99cc88340f..e04fb75e58fb 100644 --- a/drivers/ata/sata_fsl.c +++ b/drivers/ata/sata_fsl.c | |||
@@ -472,16 +472,10 @@ static int sata_fsl_scr_write(struct ata_port *ap, unsigned int sc_reg_in, | |||
472 | 472 | ||
473 | switch (sc_reg_in) { | 473 | switch (sc_reg_in) { |
474 | case SCR_STATUS: | 474 | case SCR_STATUS: |
475 | sc_reg = 0; | ||
476 | break; | ||
477 | case SCR_ERROR: | 475 | case SCR_ERROR: |
478 | sc_reg = 1; | ||
479 | break; | ||
480 | case SCR_CONTROL: | 476 | case SCR_CONTROL: |
481 | sc_reg = 2; | ||
482 | break; | ||
483 | case SCR_ACTIVE: | 477 | case SCR_ACTIVE: |
484 | sc_reg = 3; | 478 | sc_reg = sc_reg_in; |
485 | break; | 479 | break; |
486 | default: | 480 | default: |
487 | return -EINVAL; | 481 | return -EINVAL; |
@@ -502,16 +496,10 @@ static int sata_fsl_scr_read(struct ata_port *ap, unsigned int sc_reg_in, | |||
502 | 496 | ||
503 | switch (sc_reg_in) { | 497 | switch (sc_reg_in) { |
504 | case SCR_STATUS: | 498 | case SCR_STATUS: |
505 | sc_reg = 0; | ||
506 | break; | ||
507 | case SCR_ERROR: | 499 | case SCR_ERROR: |
508 | sc_reg = 1; | ||
509 | break; | ||
510 | case SCR_CONTROL: | 500 | case SCR_CONTROL: |
511 | sc_reg = 2; | ||
512 | break; | ||
513 | case SCR_ACTIVE: | 501 | case SCR_ACTIVE: |
514 | sc_reg = 3; | 502 | sc_reg = sc_reg_in; |
515 | break; | 503 | break; |
516 | default: | 504 | default: |
517 | return -EINVAL; | 505 | return -EINVAL; |