Various coldfire fixes: (1) iAudio Bootloader: Check the status of main & remote power button vs. the respective hold switch, and shut down when on hold. On X5/M5 this check is not strictly necessary for the main unit, but left there to keep things uniform. (2) M3: Stop reading the ADC properly before leaving the bootloader, on RoLo, and on reboot, to make it work reliably after those transitions. (3) Disable all interrupt sources on system init to avoid premature ISR calls after enabling interrupts in general. (4) iAudios: Proper implementation of ide_powered(), avoiding nasty HDD clicks in the bootloader when powering down, e.g. because of Hold. (5) Slight optimisations.

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

5 files changed, 29 insertions, 14 deletions

diff --git a/firmware/target/coldfire/iaudio/m3/adc-target.h b/firmware/target/coldfire/iaudio/m3/adc-target.h
index e5ff818647..40ed226bea 100644
--- a/firmware/target/coldfire/iaudio/m3/adc-target.h
+++ b/firmware/target/coldfire/iaudio/m3/adc-target.h
@@ -26,4 +26,7 @@
 #define ADC_BATTERY 1
 #define ADC_REMOTE  2
 
+#define NEED_ADC_CLOSE
+void adc_close(void);
+
 #endif /* _ADC_TARGET_H_ */
diff --git a/firmware/target/coldfire/iaudio/m3/power-m3.c b/firmware/target/coldfire/iaudio/m3/power-m3.c
index 52a5de3460..a7f3fc46be 100644
--- a/firmware/target/coldfire/iaudio/m3/power-m3.c
+++ b/firmware/target/coldfire/iaudio/m3/power-m3.c
@@ -36,7 +36,7 @@ void power_init(void)
     /* Charger detect */
     and_l(~0x00000020, &GPIO1_ENABLE);
     or_l(0x00000020, &GPIO1_FUNCTION);
-                                                        
+
     /* FIXME: Just disable the multi-colour LED for now. */
     and_l(~0x00000210, &GPIO1_OUT);
     and_l(~0x00008000, &GPIO_OUT);
@@ -61,7 +61,7 @@ void ide_power_enable(bool on)
 
 bool ide_powered(void)
 {
-    return false;
+    return (GPIO_OUT & 0x00800000) != 0;
 }
 
 void power_off(void)
diff --git a/firmware/target/coldfire/iaudio/m5/power-m5.c b/firmware/target/coldfire/iaudio/m5/power-m5.c
index d33cbec8be..939cae02f2 100644
--- a/firmware/target/coldfire/iaudio/m5/power-m5.c
+++ b/firmware/target/coldfire/iaudio/m5/power-m5.c
@@ -38,23 +38,23 @@ void power_init(void)
 
 bool charger_inserted(void)
 {     
-    return (GPIO1_READ & 0x01000000)?true:false;
+    return (GPIO1_READ & 0x01000000) != 0;
 }
 
 void ide_power_enable(bool on)
 {
     /* GPOOD3 */
     int level = set_irq_level(HIGHEST_IRQ_LEVEL);
-    if(on)
-        pcf50606_write(0x3c, 0x07);
-    else
-        pcf50606_write(0x3c, 0x00);
+    pcf50606_write(0x3c, on ? 0x07 : 0x00);
     set_irq_level(level);
 }
 
 bool ide_powered(void)
 {
-    return false;
+    int level = set_irq_level(HIGHEST_IRQ_LEVEL);
+    int value = pcf50606_read(0x3c);
+    set_irq_level(level);
+    return (value & 0x07) != 0;
 }
 
 void power_off(void)
diff --git a/firmware/target/coldfire/iaudio/x5/power-x5.c b/firmware/target/coldfire/iaudio/x5/power-x5.c
index cfcb384a24..068b25f577 100644
--- a/firmware/target/coldfire/iaudio/x5/power-x5.c
+++ b/firmware/target/coldfire/iaudio/x5/power-x5.c
@@ -38,23 +38,23 @@ void power_init(void)
 
 bool charger_inserted(void)
 {     
-    return (GPIO1_READ & 0x01000000)?true:false;
+    return (GPIO1_READ & 0x01000000) != 0;
 }
 
 void ide_power_enable(bool on)
 {
     /* GPOOD3 */
     int level = set_irq_level(HIGHEST_IRQ_LEVEL);
-    if(on)
-        pcf50606_write(0x3c, 0x07);
-    else
-        pcf50606_write(0x3c, 0x00);
+    pcf50606_write(0x3c, on ? 0x07 : 0x00);
     set_irq_level(level);
 }
 
 bool ide_powered(void)
 {
-    return false;
+    int level = set_irq_level(HIGHEST_IRQ_LEVEL);
+    int value = pcf50606_read(0x3c);
+    set_irq_level(level);
+    return (value & 0x07) != 0;
 }
 
 void power_off(void)
diff --git a/firmware/target/coldfire/system-coldfire.c b/firmware/target/coldfire/system-coldfire.c
index 97e9d35aaa..60f7ab12ad 100644
--- a/firmware/target/coldfire/system-coldfire.c
+++ b/firmware/target/coldfire/system-coldfire.c
@@ -18,6 +18,7 @@
  ****************************************************************************/
 #include <stdio.h>
 #include "config.h"
+#include "adc.h"
 #include "system.h"
 #include "lcd.h"
 #include "font.h"
@@ -254,6 +255,16 @@ void system_init(void)
        what'll be the most useful for most things which the main thread
        will do. */
     coldfire_set_macsr(EMAC_FRACTIONAL | EMAC_SATURATE);
+    
+    IMR = 0x3ffff;
+    INTPRI1 = 0;
+    INTPRI2 = 0;
+    INTPRI3 = 0;
+    INTPRI4 = 0;
+    INTPRI5 = 0;
+    INTPRI6 = 0;
+    INTPRI7 = 0;
+    INTPRI8 = 0;
 
     /* Set INTBASE and SPURVEC */
     INTBASE = 64;
@@ -268,6 +279,7 @@ void system_init(void)
 
 void system_reboot (void)
 {
+    adc_close();
     set_cpu_frequency(0);
 
     asm(" move.w #0x2700,%sr");