summary refs log tree commit diff
path: root/drivers/extcon
diff options
context:
space:
mode:
authorChanwoo Choi <cw00.choi@samsung.com>2012-12-06 21:36:18 +0900
committerChanwoo Choi <cw00.choi@samsung.com>2013-01-15 15:42:16 +0900
commit0e2738f59c6db185a70a683059980bd2296571ca (patch)
tree83f85242972facb889d7b75ce62287ba8190a0ac /drivers/extcon
parent2b75799f5ae9f31ea9778003591fd295d3bc3159 (diff)
downloadlinux-0e2738f59c6db185a70a683059980bd2296571ca.tar.gz
extcon: max77693: Fix bug when detecting MHL/Dock-Audio with USB/TA cable
This patch fix bug that muic couldn't detect MHL/Dock-Audio with USB/TA
cable on exception situation. I explain detail case on following:

When MHL(with USB/TA cable) or Dock-Audio with USB/TA cable is attached,
the MUIC device happen following two interrupt.
- 'MAX77693_MUIC_IRQ_INT1_ADC' for detecting MHL/Dock-Audio.
- 'MAX77693_MUIC_IRQ_INT2_CHGTYP' for detecting USB/TA cable connected to
MHL/Dock-Audio. Always, happen eariler MAX77693_MUIC_IRQ_INT1_ADC interrupt
than MAX77693_MUIC_IRQ_INT2_CHGTYP interrupt.

If user attach MHL with USB/TA cable and immediately detach MHL with USB/TA
cable before MAX77693_MUIC_IRQ_INT2_CHGTYP interrupt is happened, USB/TA
connected to MHL cable remain connected state to target. But USB/TA connected
to MHL cable isn't connected to target. user be faced with unusual action.
So, driver should check this situation in spite of that, previous charger type
is N/A.

Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
Signed-off-by: Myungjoo Ham <myungjoo.ham@samsung.com>
Diffstat (limited to 'drivers/extcon')
-rw-r--r--drivers/extcon/extcon-max77693.c20
1 files changed, 20 insertions, 0 deletions
diff --git a/drivers/extcon/extcon-max77693.c b/drivers/extcon/extcon-max77693.c
index abab068adc35..28eff88fca18 100644
--- a/drivers/extcon/extcon-max77693.c
+++ b/drivers/extcon/extcon-max77693.c
@@ -737,6 +737,7 @@ static int max77693_muic_chg_handler(struct max77693_muic_info *info)
 
 	switch (chg_type) {
 	case MAX77693_CHARGER_TYPE_USB:
+	case MAX77693_CHARGER_TYPE_NONE:
 		/*
 		 * MHL_TA(USB/TA) with MHL cable
 		 * - MHL cable include two port(HDMI line and separate micro
@@ -778,6 +779,25 @@ static int max77693_muic_chg_handler(struct max77693_muic_info *info)
 			goto out;
 		}
 
+		/*
+		 * When MHL(with USB/TA cable) or Dock-Audio with USB/TA cable
+		 * is attached, muic device happen below two interrupt.
+		 * - 'MAX77693_MUIC_IRQ_INT1_ADC' for detecting MHL/Dock-Audio.
+		 * - 'MAX77693_MUIC_IRQ_INT2_CHGTYP' for detecting USB/TA cable
+		 *   connected to MHL or Dock-Audio.
+		 * Always, happen eariler MAX77693_MUIC_IRQ_INT1_ADC interrupt
+		 * than MAX77693_MUIC_IRQ_INT2_CHGTYP interrupt.
+		 *
+		 * If user attach MHL (with USB/TA cable and immediately detach
+		 * MHL with USB/TA cable before MAX77693_MUIC_IRQ_INT2_CHGTYP
+		 * interrupt is happened, USB/TA cable remain connected state to
+		 * target. But USB/TA cable isn't connected to target. The user
+		 * be face with unusual action. So, driver should check this
+		 * situation in spite of, that previous charger type is N/A.
+		 */
+		if (chg_type == MAX77693_CHARGER_TYPE_NONE)
+			break;
+
 		/* Only USB cable, PATH:AP_USB */
 		ret = max77693_muic_set_path(info, CONTROL1_SW_USB, attached);
 		if (ret < 0)