Nyaaori/rockbox
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

MPIO HD300: Fix occasional scrollstrip hang

Browse source 
Scrollstrip works as quadrature encoder. We set pin change
interrupt for both edges of signal both lines and sample the state
in ISR. From time to time the sequence is out-of-sync which was
properly detected but erroneously handled leaving scrollstip
interrupts disabled.

Change-Id: I08e4f99c6c27df0f8180aa16e6e1d9e4203bafa7
Reviewed-on: http://gerrit.rockbox.org/782
Reviewed-by: Marcin Bukat <marcin.bukat@gmail.com>
Tested: Marcin Bukat <marcin.bukat@gmail.com>
This commit is contained in:
Marcin Bukat 2014-04-11 11:04:06 +02:00
parent a18c33a498
commit bd10245e89
1 changed files with 11 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
firmware/target/coldfire/mpio/hd300/button-hd300.c
View file

@ -90,20 +90,20 @@ void scrollstrip_isr(void)
/* read GPIO6 and GPIO7 state*/ /* read GPIO6 and GPIO7 state*/
new_scroll_lines = (GPIO_READ >> 6) & 0x03; new_scroll_lines = (GPIO_READ >> 6) & 0x03;
/* was it initialized? */
if ( prev_scroll_lines == -1 ) if ( prev_scroll_lines == -1 )
{ {
prev_scroll_lines = new_scroll_lines; prev_scroll_lines = new_scroll_lines;
ack_scrollstrip_interrupt(); goto end;
enable_scrollstrip_interrupts();
return;
} }
/* calculate the direction according to the sequence order */
scroll_dir = scroll_state[prev_scroll_lines][new_scroll_lines]; scroll_dir = scroll_state[prev_scroll_lines][new_scroll_lines];
prev_scroll_lines = new_scroll_lines; prev_scroll_lines = new_scroll_lines;
/* catch sequence error */ /* catch sequence error */
if (scroll_dir == BUTTON_NONE) if (scroll_dir == BUTTON_NONE)
return; goto end;
/* direction reversal */ /* direction reversal */
if (direction != scroll_dir) if (direction != scroll_dir)
@ -116,9 +116,7 @@ void scrollstrip_isr(void)
direction = scroll_dir; direction = scroll_dir;
count = 0; count = 0;
ack_scrollstrip_interrupt(); goto end;
enable_scrollstrip_interrupts();
return;
} }
/* poke backlight */ /* poke backlight */
@ -129,12 +127,9 @@ void scrollstrip_isr(void)
next_backlight_on = current_tick + HZ/4; next_backlight_on = current_tick + HZ/4;
} }
/* apply sensitivity filter */
if (++count < SLIDER_BASE_SENSITIVITY) if (++count < SLIDER_BASE_SENSITIVITY)
{ goto end;
ack_scrollstrip_interrupt();
enable_scrollstrip_interrupts();
return;
}
count = 0; count = 0;
@ -146,6 +141,10 @@ void scrollstrip_isr(void)
scroll.timeout = current_tick + SLIDER_REL_TIMEOUT; scroll.timeout = current_tick + SLIDER_REL_TIMEOUT;
scroll.rel = false; scroll.rel = false;
end:
/* acknowledge the interrupt
* and reenable scrollstrip interrupts
*/
ack_scrollstrip_interrupt(); ack_scrollstrip_interrupt();
enable_scrollstrip_interrupts(); enable_scrollstrip_interrupts();
} }