Commit 024811a2 authored by Damien Le Moal's avatar Damien Le Moal

ata: libata-core: Simplify ata_dev_set_xfermode()

The err_mask variable is not useful. Remove it.
Signed-off-by: default avatarDamien Le Moal <damien.lemoal@opensource.wdc.com>
parent 066de3b9
...@@ -4295,7 +4295,6 @@ static void ata_dev_xfermask(struct ata_device *dev) ...@@ -4295,7 +4295,6 @@ static void ata_dev_xfermask(struct ata_device *dev)
static unsigned int ata_dev_set_xfermode(struct ata_device *dev) static unsigned int ata_dev_set_xfermode(struct ata_device *dev)
{ {
struct ata_taskfile tf; struct ata_taskfile tf;
unsigned int err_mask;
/* set up set-features taskfile */ /* set up set-features taskfile */
ata_dev_dbg(dev, "set features - xfer mode\n"); ata_dev_dbg(dev, "set features - xfer mode\n");
...@@ -4317,10 +4316,11 @@ static unsigned int ata_dev_set_xfermode(struct ata_device *dev) ...@@ -4317,10 +4316,11 @@ static unsigned int ata_dev_set_xfermode(struct ata_device *dev)
else /* In the ancient relic department - skip all of this */ else /* In the ancient relic department - skip all of this */
return 0; return 0;
/* On some disks, this command causes spin-up, so we need longer timeout */ /*
err_mask = ata_exec_internal(dev, &tf, NULL, DMA_NONE, NULL, 0, 15000); * On some disks, this command causes spin-up, so we need longer
* timeout.
return err_mask; */
return ata_exec_internal(dev, &tf, NULL, DMA_NONE, NULL, 0, 15000);
} }
/** /**
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment