[PATCH v2] AHCI: use ATA_BUSY

[PATCH v2] AHCI: use ATA_BUSY

[Sergei Shtylyov]

ahci_hardreset() and ahci_p5wdh_hardreset() use bare numbers for the BSY bit of
the  ATA status  register, despite it's #define'd in <linux/ata.h>.

Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

---
The patch is against the 'for-3.11' branch of Tejun Heo's 'libata.git' repo. 

 drivers/ata/ahci.c    |    2 +-
 drivers/ata/libahci.c |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Index: libata/drivers/ata/ahci.c
===================================================================
--- libata.orig/drivers/ata/ahci.c
+++ libata/drivers/ata/ahci.c
@@ -583,7 +583,7 @@ static int ahci_p5wdh_hardreset(struct a
 
 	/* clear D2H reception area to properly wait for D2H FIS */
 	ata_tf_init(link->device, &tf);
-	tf.command = 0x80;
+	tf.command = ATA_BUSY;
 	ata_tf_to_fis(&tf, 0, 0, d2h_fis);
 
 	rc = sata_link_hardreset(link, sata_ehc_deb_timing(&link->eh_context),
Index: libata/drivers/ata/libahci.c
===================================================================
--- libata.orig/drivers/ata/libahci.c
+++ libata/drivers/ata/libahci.c
@@ -1413,7 +1413,7 @@ static int ahci_hardreset(struct ata_lin
 
 	/* clear D2H reception area to properly wait for D2H FIS */
 	ata_tf_init(link->device, &tf);
-	tf.command = 0x80;
+	tf.command = ATA_BUSY;
 	ata_tf_to_fis(&tf, 0, 0, d2h_fis);
 
 	rc = sata_link_hardreset(link, timing, deadline, &online,

Re: [PATCH v2] AHCI: use ATA_BUSY

[Tejun Heo]

On Sun, Jun 23, 2013 at 01:39:39AM +0400, Sergei Shtylyov wrote:
> ahci_hardreset() and ahci_p5wdh_hardreset() use bare numbers for the BSY bit of
> the  ATA status  register, despite it's #define'd in <linux/ata.h>.
> 
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

Applied to libata/for-3.11.

Thanks.

-- 
tejun