usb: common: usb-conn-gpio: use usb_role_string() to print role status
authorChunfeng Yun <chunfeng.yun@mediatek.com>
Tue, 25 May 2021 08:53:06 +0000 (16:53 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 27 May 2021 07:17:18 +0000 (09:17 +0200)
Use usb_role_string() to print role status, make the log readable.

Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
Link: https://lore.kernel.org/r/1621932786-9335-2-git-send-email-chunfeng.yun@mediatek.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/common/usb-conn-gpio.c

index dfbbc4f..0158148 100644 (file)
@@ -83,11 +83,11 @@ static void usb_conn_detect_cable(struct work_struct *work)
        else
                role = USB_ROLE_NONE;
 
-       dev_dbg(info->dev, "role %d/%d, gpios: id %d, vbus %d\n",
-               info->last_role, role, id, vbus);
+       dev_dbg(info->dev, "role %s -> %s, gpios: id %d, vbus %d\n",
+               usb_role_string(info->last_role), usb_role_string(role), id, vbus);
 
        if (info->last_role == role) {
-               dev_warn(info->dev, "repeated role: %d\n", role);
+               dev_warn(info->dev, "repeated role: %s\n", usb_role_string(role));
                return;
        }