On Tue, May 04, 2010 at 10:03:18AM -0400, Jarod Wilson wrote: > @@ -1205,7 +1204,7 @@ static u32 imon_panel_key_lookup(u64 hw_code) > if (imon_panel_key_table[i].hw_code == (code | 0xffee)) > break; > > - keycode = imon_panel_key_table[i % IMON_KEY_RELEASE_OFFSET].keycode; > + keycode = imon_panel_key_table[i].keycode; > > return keycode; > } There is still potentially a problem here because if we don't hit the break statement, then we're one past the end of the array. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html