libahci: wrong comments in ahci_do_softreset()
authorMinwoo Im <dn3108@gmail.com>
Sun, 11 Jun 2017 14:53:00 +0000 (23:53 +0900)
committerTejun Heo <tj@kernel.org>
Mon, 12 Jun 2017 18:07:49 +0000 (14:07 -0400)
AHCI 1.3.1 Spec says that software shall build two H2D register
FISes in the command list to send a software reset.
The comments in ahci_do_softreset() is currently D2H instead of H2D.

Signed-off-by: Minwoo Im <dn3108@gmail.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
drivers/ata/libahci.c

index 6154f0e2b81a9e88716781b556e88f8cfd40e02c..b3a685ad9b87538344275804a02e592f76765d85 100644 (file)
@@ -1400,7 +1400,7 @@ int ahci_do_softreset(struct ata_link *link, unsigned int *class,
 
        ata_tf_init(link->device, &tf);
 
-       /* issue the first D2H Register FIS */
+       /* issue the first H2D Register FIS */
        msecs = 0;
        now = jiffies;
        if (time_after(deadline, now))
@@ -1417,7 +1417,7 @@ int ahci_do_softreset(struct ata_link *link, unsigned int *class,
        /* spec says at least 5us, but be generous and sleep for 1ms */
        ata_msleep(ap, 1);
 
-       /* issue the second D2H Register FIS */
+       /* issue the second H2D Register FIS */
        tf.ctl &= ~ATA_SRST;
        ahci_exec_polled_cmd(ap, pmp, &tf, 0, 0, 0);