ahci: Make sending the SETFEATURES_XFER command optional
authorGabe Black <gabeblack@chromium.org>
Mon, 29 Oct 2012 05:23:52 +0000 (05:23 +0000)
committerTom Rini <trini@ti.com>
Fri, 2 Nov 2012 22:20:41 +0000 (15:20 -0700)
commite81058c05b762e6da012a7a3c22ee25c6c0bf71e
tree158a406416dd92c9592735e2078d0548407948c3
parent4e422bce8a51749ff3c67eec2c8caa6377cc35ee
ahci: Make sending the SETFEATURES_XFER command optional

This command doesn't really do anything when talking to a SATA device, and
sending it confuses some of them. This change makes sending the command
optional, and defaults to not. The situations where it should be sent are not
the common case.

With the standard SSD in the machine, here are some times with the option
turned off:
1. 8277
2. 8273
3. 8050

And turned on:
1. 8303
2. 8155
3. 8276

Sending that command seems to have no meaningful effect on performance.

This fixes problems with an SSD marked Toshiba NV6424, Taiwan 11159AE P
and TC58NVG5D2FTA10.

Signed-off-by: Gabe Black <gabeblack@chromium.org>
Signed-off-by: Taylor Hutt <thutt@chromium.org>
Signed-off-by: Simon Glass <sjg@chromium.org>
drivers/block/ahci.c