Re: [PATCH 11/16] phy: rockchip: usbdp: Drop rockchip_u3phy_uboot_init()

2024-05-06 Thread Kever Yang



On 2024/5/5 03:43, Jonas Karlman wrote:

Remove the rockchip_u3phy_uboot_init() function, it has no caller and is
not needed with proper driver model use.

Signed-off-by: Jonas Karlman 

Reviewed-by: Kever Yang 

Thanks,
- Kever

---
  drivers/phy/rockchip/phy-rockchip-usbdp.c | 24 ---
  1 file changed, 24 deletions(-)

diff --git a/drivers/phy/rockchip/phy-rockchip-usbdp.c 
b/drivers/phy/rockchip/phy-rockchip-usbdp.c
index 8e5821069757..bf0fb6d8288f 100644
--- a/drivers/phy/rockchip/phy-rockchip-usbdp.c
+++ b/drivers/phy/rockchip/phy-rockchip-usbdp.c
@@ -616,30 +616,6 @@ static const struct phy_ops rockchip_u3phy_ops = {
.exit   = rockchip_u3phy_exit,
  };
  
-int rockchip_u3phy_uboot_init(void)

-{
-   struct udevice *udev;
-   struct rockchip_udphy *udphy;
-   int ret;
-
-   ret = uclass_get_device_by_driver(UCLASS_PHY,
- DM_DRIVER_GET(rockchip_udphy_u3_port),
- );
-   if (ret) {
-   pr_err("%s: get u3-port failed: %d\n", __func__, ret);
-   return ret;
-   }
-
-   /* DP only or high-speed, disable U3 port */
-   udphy = dev_get_priv(udev->parent);
-   if (!(udphy->mode & UDPHY_MODE_USB) || udphy->hs) {
-   udphy_u3_port_disable(udphy, true);
-   return 0;
-   }
-
-   return udphy_power_on(udphy, UDPHY_MODE_USB);
-}
-
  static int rockchip_udphy_probe(struct udevice *dev)
  {
struct rockchip_udphy *udphy = dev_get_priv(dev);


[PATCH 11/16] phy: rockchip: usbdp: Drop rockchip_u3phy_uboot_init()

2024-05-04 Thread Jonas Karlman
Remove the rockchip_u3phy_uboot_init() function, it has no caller and is
not needed with proper driver model use.

Signed-off-by: Jonas Karlman 
---
 drivers/phy/rockchip/phy-rockchip-usbdp.c | 24 ---
 1 file changed, 24 deletions(-)

diff --git a/drivers/phy/rockchip/phy-rockchip-usbdp.c 
b/drivers/phy/rockchip/phy-rockchip-usbdp.c
index 8e5821069757..bf0fb6d8288f 100644
--- a/drivers/phy/rockchip/phy-rockchip-usbdp.c
+++ b/drivers/phy/rockchip/phy-rockchip-usbdp.c
@@ -616,30 +616,6 @@ static const struct phy_ops rockchip_u3phy_ops = {
.exit   = rockchip_u3phy_exit,
 };
 
-int rockchip_u3phy_uboot_init(void)
-{
-   struct udevice *udev;
-   struct rockchip_udphy *udphy;
-   int ret;
-
-   ret = uclass_get_device_by_driver(UCLASS_PHY,
- DM_DRIVER_GET(rockchip_udphy_u3_port),
- );
-   if (ret) {
-   pr_err("%s: get u3-port failed: %d\n", __func__, ret);
-   return ret;
-   }
-
-   /* DP only or high-speed, disable U3 port */
-   udphy = dev_get_priv(udev->parent);
-   if (!(udphy->mode & UDPHY_MODE_USB) || udphy->hs) {
-   udphy_u3_port_disable(udphy, true);
-   return 0;
-   }
-
-   return udphy_power_on(udphy, UDPHY_MODE_USB);
-}
-
 static int rockchip_udphy_probe(struct udevice *dev)
 {
struct rockchip_udphy *udphy = dev_get_priv(dev);
-- 
2.43.2