summaryrefslogtreecommitdiff
path: root/drivers
diff options
context:
space:
mode:
authorJonghwa Lee <jonghwa3.lee@samsung.com>2014-03-17 17:44:06 +0900
committerChanho Park <chanho61.park@samsung.com>2014-03-20 17:44:20 +0900
commit15905cff739df8fd4d674397ced269f17728c52c (patch)
tree454f2ebd01e06c58062ef5eb3cd8ef3ff23903e5 /drivers
parentf6cc1c1abe0a80a0f6d4fa6c42d983c43b9df963 (diff)
downloadlinux-3.10-15905cff739df8fd4d674397ced269f17728c52c.tar.gz
linux-3.10-15905cff739df8fd4d674397ced269f17728c52c.tar.bz2
linux-3.10-15905cff739df8fd4d674397ced269f17728c52c.zip
extcon: max77693: Fix inaccurate extcon event for JIG with USB cable
Dmitry's patch 'extcon: max77693: Force using UART~' fixes problem when device is connected JIG with USB cable. However, with the patch, driver sends inaccurate event "USB" even muic keeps path to UART not USB. This would make users confused. This patch keeps Dmitry's modification working and also bypasses signaling event "USB" when device is connected to JIG with USB cable. Change-Id: Ia5f3328e5b0827378178502c70cb90da48e9e3f7 Signed-off-by: Jonghwa Lee <jonghwa3.lee@samsung.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/extcon/extcon-max77693.c23
1 files changed, 12 insertions, 11 deletions
diff --git a/drivers/extcon/extcon-max77693.c b/drivers/extcon/extcon-max77693.c
index 903c5078f12..b08a1e882e4 100644
--- a/drivers/extcon/extcon-max77693.c
+++ b/drivers/extcon/extcon-max77693.c
@@ -293,18 +293,10 @@ static int max77693_muic_set_path(struct max77693_muic_info *info,
return ret;
}
- if (attached) {
- if (info->prev_cable_type ==
- MAX77693_MUIC_ADC_FACTORY_MODE_UART_OFF) {
- /* if cable_type is jig, then force UART */
- dev_info(info->dev, "For jig force using UART path\n");
- ctrl1 = CONTROL1_SW_UART;
- } else {
- ctrl1 = val;
- }
- } else {
+ if (attached)
+ ctrl1 = val;
+ else
ctrl1 = CONTROL1_SW_OPEN;
- }
ret = max77693_update_reg(info->max77693->regmap_muic,
MAX77693_MUIC_REG_CTRL1, ctrl1, COMP_SW_MASK);
@@ -922,6 +914,15 @@ static int max77693_muic_chg_handler(struct max77693_muic_info *info)
*/
break;
case MAX77693_CHARGER_TYPE_USB:
+ /* If UART is still connected, do not set path to USB */
+ if (cable_type == MAX77693_MUIC_ADC_FACTORY_MODE_UART_OFF) {
+ ret = max77693_muic_set_path(info, info->path_uart,
+ attached);
+ if (ret < 0)
+ return ret;
+ break;
+ }
+
/* Only USB cable, PATH:AP_USB */
ret = max77693_muic_set_path(info, info->path_usb, attached);
if (ret < 0)