From 7249c8807c6e8f9d083d3909863097e8a44cc4fe Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B6rn=20Stenberg?= <bjorn@haxx.se>
Date: Mon, 2 Dec 2002 10:30:40 +0000
Subject: [PATCH] Updated power-on procedure

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@2896 a1c6a512-1295-4272-9138-f99709370657
---
 firmware/drivers/ata.c | 29 ++++++++++-------------------
 1 file changed, 10 insertions(+), 19 deletions(-)

diff --git a/firmware/drivers/ata.c b/firmware/drivers/ata.c
index 173dab9be3..f49040a794 100644
--- a/firmware/drivers/ata.c
+++ b/firmware/drivers/ata.c
@@ -102,6 +102,7 @@ static unsigned short identify_info[SECTOR_SIZE];
 
 static int ata_power_on(void);
 static int perform_soft_reset(void);
+static int set_multiple_mode(int sectors);
 
 static int wait_for_bsy(void) __attribute__ ((section (".icode")));
 static int wait_for_bsy(void)
@@ -524,21 +525,18 @@ int ata_hard_reset(void)
 {
     int ret;
     
-    mutex_lock(&ata_mtx);
-    
     PADR &= ~0x0200;
 
     sleep(2);
 
     PADR |= 0x0200;
 
-    ret = wait_for_rdy();
+    ret = wait_for_bsy();
 
     /* Massage the return code so it is 0 on success and -1 on failure */
     ret = ret?0:-1;
 
     sleeping = false;
-    mutex_unlock(&ata_mtx);
     return ret;
 }
 
@@ -582,26 +580,19 @@ int ata_soft_reset(void)
 
 static int ata_power_on(void)
 {
-    int ret;
-    int retry_count;
-    
     ide_power_enable(true);
-    sleep(HZ/2);
+    if( ata_hard_reset() )
+        return -1;
 
+    ATA_SELECT = SELECT_LBA;
     ATA_CONTROL = CONTROL_nIEN;
 
-    /* This little sucker can take up to 30 seconds */
-    retry_count = 8;
-    do
-    {
-        ret = wait_for_rdy();
-    } while(!ret && retry_count--);
-
-    /* Massage the return code so it is 0 on success and -1 on failure */
-    ret = ret?0:-1;
+    if (set_multiple_mode(multisectors))
+        return -2;
 
     sleeping = false;
-    return ret;
+    poweroff = false;
+    return 0;
 }
 
 static int master_slave_detect(void)
@@ -752,7 +743,7 @@ int ata_init(void)
         initialized = true;
     }
     if (set_multiple_mode(multisectors))
-       return -6;
+        return -6;
 
     ATA_SELECT = SELECT_LBA;
     ATA_CONTROL = CONTROL_nIEN;