Commit cd59e796 authored by Mark Brown's avatar Mark Brown

extcon: arizona: Allow additional debounce during microphone detection

Help mitigate against mechanical bounce during the initial detection by
allowing the configuration of an additional debounce on top of that the
hardware does during the initial phase of microphone detection operation.
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
parent 41a57850
...@@ -64,6 +64,7 @@ struct arizona_extcon_info { ...@@ -64,6 +64,7 @@ struct arizona_extcon_info {
bool micd_clamp; bool micd_clamp;
struct delayed_work hpdet_work; struct delayed_work hpdet_work;
struct delayed_work micd_detect_work;
struct delayed_work micd_timeout_work; struct delayed_work micd_timeout_work;
bool hpdet_active; bool hpdet_active;
...@@ -750,9 +751,11 @@ static void arizona_micd_timeout_work(struct work_struct *work) ...@@ -750,9 +751,11 @@ static void arizona_micd_timeout_work(struct work_struct *work)
mutex_unlock(&info->lock); mutex_unlock(&info->lock);
} }
static irqreturn_t arizona_micdet(int irq, void *data) static void arizona_micd_detect(struct work_struct *work)
{ {
struct arizona_extcon_info *info = data; struct arizona_extcon_info *info = container_of(work,
struct arizona_extcon_info,
micd_detect_work.work);
struct arizona *arizona = info->arizona; struct arizona *arizona = info->arizona;
unsigned int val = 0, lvl; unsigned int val = 0, lvl;
int ret, i, key; int ret, i, key;
...@@ -766,7 +769,7 @@ static irqreturn_t arizona_micdet(int irq, void *data) ...@@ -766,7 +769,7 @@ static irqreturn_t arizona_micdet(int irq, void *data)
if (ret != 0) { if (ret != 0) {
dev_err(arizona->dev, "Failed to read MICDET: %d\n", ret); dev_err(arizona->dev, "Failed to read MICDET: %d\n", ret);
mutex_unlock(&info->lock); mutex_unlock(&info->lock);
return IRQ_NONE; return;
} }
dev_dbg(arizona->dev, "MICDET: %x\n", val); dev_dbg(arizona->dev, "MICDET: %x\n", val);
...@@ -774,14 +777,14 @@ static irqreturn_t arizona_micdet(int irq, void *data) ...@@ -774,14 +777,14 @@ static irqreturn_t arizona_micdet(int irq, void *data)
if (!(val & ARIZONA_MICD_VALID)) { if (!(val & ARIZONA_MICD_VALID)) {
dev_warn(arizona->dev, "Microphone detection state invalid\n"); dev_warn(arizona->dev, "Microphone detection state invalid\n");
mutex_unlock(&info->lock); mutex_unlock(&info->lock);
return IRQ_NONE; return;
} }
} }
if (i == 10 && !(val & 0x7fc)) { if (i == 10 && !(val & 0x7fc)) {
dev_err(arizona->dev, "Failed to get valid MICDET value\n"); dev_err(arizona->dev, "Failed to get valid MICDET value\n");
mutex_unlock(&info->lock); mutex_unlock(&info->lock);
return IRQ_NONE; return;
} }
/* Due to jack detect this should never happen */ /* Due to jack detect this should never happen */
...@@ -890,6 +893,27 @@ static irqreturn_t arizona_micdet(int irq, void *data) ...@@ -890,6 +893,27 @@ static irqreturn_t arizona_micdet(int irq, void *data)
pm_runtime_mark_last_busy(info->dev); pm_runtime_mark_last_busy(info->dev);
mutex_unlock(&info->lock); mutex_unlock(&info->lock);
}
static irqreturn_t arizona_micdet(int irq, void *data)
{
struct arizona_extcon_info *info = data;
struct arizona *arizona = info->arizona;
int debounce = arizona->pdata.micd_detect_debounce;
cancel_delayed_work_sync(&info->micd_detect_work);
cancel_delayed_work_sync(&info->micd_timeout_work);
mutex_lock(&info->lock);
if (!info->detecting)
debounce = 0;
mutex_unlock(&info->lock);
if (debounce)
schedule_delayed_work(&info->micd_detect_work,
msecs_to_jiffies(debounce));
else
arizona_micd_detect(&info->micd_detect_work.work);
return IRQ_HANDLED; return IRQ_HANDLED;
} }
...@@ -1072,6 +1096,7 @@ static int arizona_extcon_probe(struct platform_device *pdev) ...@@ -1072,6 +1096,7 @@ static int arizona_extcon_probe(struct platform_device *pdev)
info->dev = &pdev->dev; info->dev = &pdev->dev;
info->last_jackdet = ~(ARIZONA_MICD_CLAMP_STS | ARIZONA_JD1_STS); info->last_jackdet = ~(ARIZONA_MICD_CLAMP_STS | ARIZONA_JD1_STS);
INIT_DELAYED_WORK(&info->hpdet_work, arizona_hpdet_work); INIT_DELAYED_WORK(&info->hpdet_work, arizona_hpdet_work);
INIT_DELAYED_WORK(&info->micd_detect_work, arizona_micd_detect);
INIT_DELAYED_WORK(&info->micd_timeout_work, arizona_micd_timeout_work); INIT_DELAYED_WORK(&info->micd_timeout_work, arizona_micd_timeout_work);
platform_set_drvdata(pdev, info); platform_set_drvdata(pdev, info);
......
...@@ -134,6 +134,9 @@ struct arizona_pdata { ...@@ -134,6 +134,9 @@ struct arizona_pdata {
/** GPIO used for mic isolation with HPDET */ /** GPIO used for mic isolation with HPDET */
int hpdet_id_gpio; int hpdet_id_gpio;
/** Extra debounce timeout used during initial mic detection (ms) */
int micd_detect_debounce;
/** GPIO for mic detection polarity */ /** GPIO for mic detection polarity */
int micd_pol_gpio; int micd_pol_gpio;
......
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