Commit 41a57850 authored by Mark Brown's avatar Mark Brown

extcon: arizona: Clear existing button reports before reporting new one

If the user moves directly from one button to another then we won't get a
no buttons pressed event and will therefore end up reporting that two
buttons are simultaneously pressed which isn't supported by the hardware.
Make sure we clear any existing button reports before reporting any new
ones.
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
parent 939c5671
...@@ -852,6 +852,10 @@ static irqreturn_t arizona_micdet(int irq, void *data) ...@@ -852,6 +852,10 @@ static irqreturn_t arizona_micdet(int irq, void *data)
lvl = val & ARIZONA_MICD_LVL_MASK; lvl = val & ARIZONA_MICD_LVL_MASK;
lvl >>= ARIZONA_MICD_LVL_SHIFT; lvl >>= ARIZONA_MICD_LVL_SHIFT;
for (i = 0; i < info->num_micd_ranges; i++)
input_report_key(info->input,
info->micd_ranges[i].key, 0);
WARN_ON(!lvl); WARN_ON(!lvl);
WARN_ON(ffs(lvl) - 1 >= info->num_micd_ranges); WARN_ON(ffs(lvl) - 1 >= info->num_micd_ranges);
if (lvl && ffs(lvl) - 1 < info->num_micd_ranges) { if (lvl && ffs(lvl) - 1 < info->num_micd_ranges) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment