Revert "usb: cdns3-starfive: Add extcon usb gadget state with always connected" 01/318801/1
authorMarek Szyprowski <m.szyprowski@samsung.com>
Tue, 8 Oct 2024 10:32:29 +0000 (12:32 +0200)
committerMarek Szyprowski <m.szyprowski@samsung.com>
Tue, 8 Oct 2024 11:01:25 +0000 (13:01 +0200)
Since adding a standalone Tizen-specific kernel extcon-usb-fixed module,
which reports USB cable as always connected, the extcon hack/workaround
in the Cadence USB3 driver is no longer needed.

This reverts commit 7e96f1653ad39babf34301a9561937c93ec332fb.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Change-Id: I03aed75616b89ffc5dfcf190b066926155a3d964

drivers/usb/cdns3/Kconfig
drivers/usb/cdns3/cdns3-starfive.c

index b139e66dfbf3ede59e3fe9b2f032ce56165e166a..0a514b5915272a000c0c313451e58c2fe012ac5d 100644 (file)
@@ -131,14 +131,3 @@ config USB_CDNSP_HOST
          standard XHCI driver.
 
 endif
-
-if USB_CDNS3_STARFIVE
-
-config USB_CDNS3_STARFIVE_EXTCON
-       bool "Extcon USB state support on Cadence USB3 of Starfive SoC"
-       depends on EXTCON
-       help
-         Say Y here to enable external connector event of the StarFive SoC
-         platforms that contain Cadence USB3 controller core.
-
-endif
index 8c5e0284747b4c65ec7c837d4518f3ff33c45219..7d4abc44abfea1fe53c52a5cf49f78bf5d435059 100644 (file)
@@ -18,9 +18,6 @@
 #include <linux/reset.h>
 #include <linux/regmap.h>
 #include <linux/usb/otg.h>
-#if IS_ENABLED(CONFIG_USB_CDNS3_STARFIVE_EXTCON)
-#include <linux/extcon-provider.h>
-#endif
 #include "core.h"
 
 #define USB_STRAP_HOST                 BIT(17)
@@ -45,9 +42,6 @@ struct cdns_starfive {
        u32 stg_usb_mode;
        struct regmap *sys_syscon;
        u32 sys_offset;
-#if IS_ENABLED(CONFIG_USB_CDNS3_STARFIVE_EXTCON)
-       struct extcon_dev               *edev;
-#endif
 };
 
 static void cdns_mode_init(struct platform_device *pdev,
@@ -125,20 +119,10 @@ static void cdns_clk_rst_deinit(struct cdns_starfive *data)
        clk_bulk_disable_unprepare(data->num_clks, data->clks);
 }
 
-#if IS_ENABLED(CONFIG_USB_CDNS3_STARFIVE_EXTCON)
-static const unsigned int supported_cable[] = {
-       EXTCON_USB,
-       EXTCON_NONE,
-};
-#endif
-
 static int cdns_starfive_probe(struct platform_device *pdev)
 {
        struct device *dev = &pdev->dev;
        struct cdns_starfive *data;
-#if IS_ENABLED(CONFIG_USB_CDNS3_STARFIVE_EXTCON)
-       struct extcon_dev *edev;
-#endif
        unsigned int args;
        int ret;
 
@@ -147,20 +131,6 @@ static int cdns_starfive_probe(struct platform_device *pdev)
                return -ENOMEM;
 
        data->dev = dev;
-#if IS_ENABLED(CONFIG_USB_CDNS3_STARFIVE_EXTCON)
-       edev = devm_extcon_dev_allocate(dev, supported_cable);
-       if (IS_ERR(edev))
-               return PTR_ERR(edev);
-
-       ret = devm_extcon_dev_register(dev, edev);
-       if (ret)
-               return ret;
-
-       /* Set starfive cdns3 usb always connected because of hw limitation */
-       extcon_set_state_sync(edev, EXTCON_USB, true);
-
-       data->edev = edev;
-#endif
 
        data->stg_syscon =
                syscon_regmap_lookup_by_phandle_args(pdev->dev.of_node,