diff options
author | Michael Sevakis <jethead71@rockbox.org> | 2012-01-08 22:29:25 +0000 |
---|---|---|
committer | Michael Sevakis <jethead71@rockbox.org> | 2012-01-08 22:29:25 +0000 |
commit | 307cb049485cc20140b85aa78f8e2677e8df5851 (patch) | |
tree | eaadfd7266e4e97e69bccd68655140dd9f1ef061 /firmware/target/arm/as3525/sd-as3525.c | |
parent | 5e21bbf5757163725f4bd2909fc7aaa548f61fc3 (diff) | |
download | rockbox-307cb049485cc20140b85aa78f8e2677e8df5851.tar.gz rockbox-307cb049485cc20140b85aa78f8e2677e8df5851.zip |
AS3525v1/2: Enable nested handling of interrupts
Mostly for the sake of reducing latency for audio servicing where other service
routines can take a long time to complete, leading to occasional drops of a
few samples, especially in recording, where they are fairly frequent.
One mystery that remains is GPIOA IRQ being interrupted causes strange
undefined instruction exceptions, most easily produced on my Fuze V2 with a
scrollwheel. Making GPIOA the top ISR for now, thus not interruptible, cures it.
SVC mode is used during the actual calls. Hopefully the SVC stack size is
sufficient. Prologue and epilogue code only uses the IRQ stack and is large
enough.
Any routine code that should not be interrupted should disable IRQ itself from
here on in.
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@31642 a1c6a512-1295-4272-9138-f99709370657
Diffstat (limited to 'firmware/target/arm/as3525/sd-as3525.c')
-rw-r--r-- | firmware/target/arm/as3525/sd-as3525.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/firmware/target/arm/as3525/sd-as3525.c b/firmware/target/arm/as3525/sd-as3525.c index 17f1bfa11d..5bed36e51e 100644 --- a/firmware/target/arm/as3525/sd-as3525.c +++ b/firmware/target/arm/as3525/sd-as3525.c | |||
@@ -178,10 +178,12 @@ static int sd1_oneshot_callback(struct timeout *tmo) | |||
178 | void sd_gpioa_isr(void) | 178 | void sd_gpioa_isr(void) |
179 | { | 179 | { |
180 | static struct timeout sd1_oneshot; | 180 | static struct timeout sd1_oneshot; |
181 | |||
181 | if (GPIOA_MIS & EXT_SD_BITS) | 182 | if (GPIOA_MIS & EXT_SD_BITS) |
183 | { | ||
182 | timeout_register(&sd1_oneshot, sd1_oneshot_callback, (3*HZ/10), 0); | 184 | timeout_register(&sd1_oneshot, sd1_oneshot_callback, (3*HZ/10), 0); |
183 | /* acknowledge interrupt */ | 185 | GPIOA_IC = EXT_SD_BITS; /* acknowledge interrupt */ |
184 | GPIOA_IC = EXT_SD_BITS; | 186 | } |
185 | } | 187 | } |
186 | #endif /* HAVE_HOTSWAP */ | 188 | #endif /* HAVE_HOTSWAP */ |
187 | 189 | ||