From: Chanwoo Choi <cw00.c...@samsung.com>

This patch just uses the resource-managed extcon API when registering
the extcon notifier.

Signed-off-by: Chanwoo Choi <cw00.c...@samsung.com>
Acked-by: Maxime Ripard <maxime.rip...@free-electrons.com>
Signed-off-by: Bin Liu <b-...@ti.com>
---
 drivers/usb/musb/sunxi.c | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/drivers/usb/musb/sunxi.c b/drivers/usb/musb/sunxi.c
index 64545de4871f..05cab67d4106 100644
--- a/drivers/usb/musb/sunxi.c
+++ b/drivers/usb/musb/sunxi.c
@@ -251,14 +251,14 @@ static int sunxi_musb_init(struct musb *musb)
        writeb(SUNXI_MUSB_VEND0_PIO_MODE, musb->mregs + SUNXI_MUSB_VEND0);
 
        /* Register notifier before calling phy_init() */
-       ret = extcon_register_notifier(glue->extcon, EXTCON_USB_HOST,
-                                      &glue->host_nb);
+       ret = devm_extcon_register_notifier(glue->dev, glue->extcon,
+                                       EXTCON_USB_HOST, &glue->host_nb);
        if (ret)
                goto error_reset_assert;
 
        ret = phy_init(glue->phy);
        if (ret)
-               goto error_unregister_notifier;
+               goto error_reset_assert;
 
        musb->isr = sunxi_musb_interrupt;
 
@@ -267,9 +267,6 @@ static int sunxi_musb_init(struct musb *musb)
 
        return 0;
 
-error_unregister_notifier:
-       extcon_unregister_notifier(glue->extcon, EXTCON_USB_HOST,
-                                  &glue->host_nb);
 error_reset_assert:
        if (test_bit(SUNXI_MUSB_FL_HAS_RESET, &glue->flags))
                reset_control_assert(glue->rst);
@@ -293,9 +290,6 @@ static int sunxi_musb_exit(struct musb *musb)
 
        phy_exit(glue->phy);
 
-       extcon_unregister_notifier(glue->extcon, EXTCON_USB_HOST,
-                                  &glue->host_nb);
-
        if (test_bit(SUNXI_MUSB_FL_HAS_RESET, &glue->flags))
                reset_control_assert(glue->rst);
 
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to