usb: chipidea: core: refactor ci_usb_role_switch_set()

Current code is redundant, refactor the code, no function change.

Signed-off-by: Xu Yang <xu.yang_2@nxp.com>
Link: https://patch.msgid.link/20260402071457.2516021-1-xu.yang_2@nxp.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/usb/chipidea/core.c b/drivers/usb/chipidea/core.c
index fac11f20cf0ac263c21d654b6acadb756b1dd2dc..87be716dff3e4149ce3eef03043c6865330d5f79 100644 (file)
--- a/drivers/usb/chipidea/core.c
+++ b/drivers/usb/chipidea/core.c
@@ -618,28 +618,13 @@ static int ci_usb_role_switch_set(struct usb_role_switch *sw,
        struct ci_hdrc *ci = usb_role_switch_get_drvdata(sw);
        struct ci_hdrc_cable *cable;
 
-       if (role == USB_ROLE_HOST) {
-               cable = &ci->platdata->id_extcon;
-               cable->changed = true;
-               cable->connected = true;
-               cable = &ci->platdata->vbus_extcon;
-               cable->changed = true;
-               cable->connected = false;
-       } else if (role == USB_ROLE_DEVICE) {
-               cable = &ci->platdata->id_extcon;
-               cable->changed = true;
-               cable->connected = false;
-               cable = &ci->platdata->vbus_extcon;
-               cable->changed = true;
-               cable->connected = true;
-       } else {
-               cable = &ci->platdata->id_extcon;
-               cable->changed = true;
-               cable->connected = false;
-               cable = &ci->platdata->vbus_extcon;
-               cable->changed = true;
-               cable->connected = false;
-       }
+       cable = &ci->platdata->id_extcon;
+       cable->changed = true;
+       cable->connected = (role == USB_ROLE_HOST);
+
+       cable = &ci->platdata->vbus_extcon;
+       cable->changed = true;
+       cable->connected = (role == USB_ROLE_DEVICE);
 
        ci_irq(ci);
        return 0;