Finally: HDD power control for all PP502x iPods (4th gen Grayscale, Color, Minis, Video). * Wait a short time after powering on the HDD on all disk based targets, as that seems like a good idea. Shorten the excessive wait on initial power on.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@15444 a1c6a512-1295-4272-9138-f99709370657

2 files changed, 22 insertions, 9 deletions

diff --git a/firmware/target/arm/ipod/power-ipod.c b/firmware/target/arm/ipod/power-ipod.c
index 37ef418026..ae663d16e5 100644
--- a/firmware/target/arm/ipod/power-ipod.c
+++ b/firmware/target/arm/ipod/power-ipod.c
@@ -63,9 +63,19 @@ void ide_power_enable(bool on)
         GPIOC_OUTPUT_VAL &= ~0x40;
     else
         GPIOC_OUTPUT_VAL |= 0x40;
-#else
-    /* We do nothing on other iPods yet */
-    (void)on;
+#elif defined(IPOD_4G) || defined(IPOD_COLOR) \
+   || defined(IPOD_MINI) || defined(IPOD_MINI2G)
+    if (on)
+        GPIO_CLEAR_BITWISE(GPIOJ_OUTPUT_VAL, 0x04);
+    else
+        GPIO_SET_BITWISE(GPIOJ_OUTPUT_VAL, 0x04);
+#elif defined(IPOD_VIDEO)
+    if (on)
+        GPO32_VAL &= ~0x40000000;
+    else
+        GPO32_VAL |= 0x40000000;
+#else /* Nano */
+    (void)on;  /* Do nothing. */
 #endif
 }
 
@@ -73,9 +83,13 @@ bool ide_powered(void)
 {
 #if defined(IPOD_1G2G) || defined(IPOD_3G)
     return !(GPIOC_OUTPUT_VAL & 0x40);
-#else
-    /* pretend we are always powered - we don't turn it off on the ipod */
-    return true;
+#elif defined(IPOD_4G) || defined(IPOD_COLOR) \
+   || defined(IPOD_MINI) || defined(IPOD_MINI2G)
+    return !(GPIOJ_OUTPUT_VAL & 0x04);
+#elif defined(IPOD_VIDEO)
+    return !(GPO32_VAL & 0x40000000);
+#else /* Nano */
+    return true; /* Pretend we are always powered */
 #endif
 }
 
diff --git a/firmware/target/arm/iriver/h10/power-h10.c b/firmware/target/arm/iriver/h10/power-h10.c
index 9cbdb50080..7f3fb40b36 100644
--- a/firmware/target/arm/iriver/h10/power-h10.c
+++ b/firmware/target/arm/iriver/h10/power-h10.c
@@ -60,10 +60,9 @@ bool charger_inserted(void)
 void ide_power_enable(bool on)
 {
     if(on){
-        GPIOF_OUTPUT_VAL &=~ 0x1;
-        sleep(1);
+        GPIO_CLEAR_BITWISE(GPIOF_OUTPUT_VAL, 0x01);
     } else {
-        GPIOF_OUTPUT_VAL |= 0x1;
+        GPIO_SET_BITWISE(GPIOF_OUTPUT_VAL, 0x01);
     }
 }