According to the documentation, we should set the EXTCON_USB when
one SDP charger connector was reported.

Signed-off-by: Baolin Wang <baolin.w...@linaro.org>
Reviewed-by: Chanwoo Choi <cw00.c...@samsung.com>
---
Changes since v1:
 - Change extcon_set_cable_state_() to extcon_set_state_sync().
 - Add reviewed tag by Chanwoo Choi.
---
 drivers/phy/phy-rockchip-inno-usb2.c |    7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/drivers/phy/phy-rockchip-inno-usb2.c 
b/drivers/phy/phy-rockchip-inno-usb2.c
index 2f99ec9..4ea95c2 100644
--- a/drivers/phy/phy-rockchip-inno-usb2.c
+++ b/drivers/phy/phy-rockchip-inno-usb2.c
@@ -595,9 +595,14 @@ static void rockchip_usb2phy_otg_sm_work(struct 
work_struct *work)
                if (rport->vbus_attached != vbus_attach) {
                        rport->vbus_attached = vbus_attach;
 
-                       if (notify_charger && rphy->edev)
+                       if (notify_charger && rphy->edev) {
                                extcon_set_cable_state_(rphy->edev,
                                                        cable, vbus_attach);
+                               if (cable == EXTCON_CHG_USB_SDP)
+                                       extcon_set_state_sync(rphy->edev,
+                                                             EXTCON_USB,
+                                                             vbus_attach);
+                       }
                }
                break;
        case OTG_STATE_B_PERIPHERAL:
-- 
1.7.9.5

Reply via email to