- Status Closed
- Percent Complete
- Task Type Patches
- Category Drivers
- Assigned To No-one
- Operating System Another
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#8687 - Use hardware interrupt handler table lookups available in DM320 interrupt controller
This is the modifications to system-dm320.c also included in FS#8686 .
Closed by kugel.
2011-06-05 11:35
Reason for closing: Out of Date
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
2011-06-05 11:35
Reason for closing: Out of Date
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
There doesn't seem to be interest
in this anymore. Open a new task if this
is wrong.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
Woops, spelling mistakes:
These are the …
As owner of a m:robe 500i (also DM320 platform), I’ll review this patch and make sure it doesn’t break anything on that side.
Short description:
This changes the way interrupts are handled to take advantage of the hardware looking of handler address(as described in the DM320 datasheet), it should save a few instruction in the IRQ/FIQ handlers and could save custom IRQ handler code bugs.
This patch is (partially) incorrect: after rereading the datasheet it seems EABASE0 (IO_INTC_ENTRY_TBA0 in the code) wasn’t filled in correctly; I’ll have to rework that one. But as I’ve disabled this in my patch for the ZVM it is low priority, so don’t expect it soon.
This patch uses the correct implementation (according to the datasheet), but isn’t working for some unknown reason.