net: phy: ti: Convert to U_BOOT_PHY_DRIVER()
authorMarek Vasut <marek.vasut+renesas@mailbox.org>
Sun, 19 Mar 2023 17:03:04 +0000 (18:03 +0100)
committerMarek Vasut <marek.vasut+renesas@mailbox.org>
Fri, 7 Apr 2023 12:18:50 +0000 (14:18 +0200)
Convert PHY driver to U_BOOT_PHY_DRIVER() macro and drop phy_register() init call.

Converted using sed
"s@^static struct phy_driver \(.*\)_driver = \+{@U_BOOT_PHY_DRIVER(\L\1) = {"

This particular PHY driver is slightly more spread out across additional
source files. Since the phy_register() calls are no longer necessary, all
the registration calls across those source files is dropped. Furthermore,
the Makefile can now be updated to only compile generic TI PHY support if
matching Kconfig symbol is enabled and the ifdeffery in the generic TI PHY
driver can be dropped.

Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
Acked-by: Michal Simek <michal.simek@amd.com>
Tested-by: Michal Simek <michal.simek@amd.com> #microblaze (MANUAL_RELOC)
drivers/net/phy/Makefile
drivers/net/phy/dp83867.c
drivers/net/phy/dp83869.c
drivers/net/phy/phy.c
drivers/net/phy/ti_phy_init.c
include/phy.h

index d38e99e..963d96e 100644 (file)
@@ -29,7 +29,7 @@ obj-$(CONFIG_PHY_NXP_TJA11XX) += nxp-tja11xx.o
 obj-$(CONFIG_PHY_REALTEK) += realtek.o
 obj-$(CONFIG_PHY_SMSC) += smsc.o
 obj-$(CONFIG_PHY_TERANETICS) += teranetics.o
-obj-$(CONFIG_PHY_TI) += ti_phy_init.o
+obj-$(CONFIG_PHY_TI_GENERIC) += ti_phy_init.o
 obj-$(CONFIG_PHY_TI_DP83867) += dp83867.o
 obj-$(CONFIG_PHY_TI_DP83869) += dp83869.o
 obj-$(CONFIG_PHY_XILINX) += xilinx_phy.o
index a45152b..b861bf7 100644 (file)
@@ -409,7 +409,7 @@ static int dp83867_probe(struct phy_device *phydev)
        return 0;
 }
 
-static struct phy_driver DP83867_driver = {
+U_BOOT_PHY_DRIVER(dp83867) = {
        .name = "TI DP83867",
        .uid = 0x2000a231,
        .mask = 0xfffffff0,
@@ -419,9 +419,3 @@ static struct phy_driver DP83867_driver = {
        .startup = &genphy_startup,
        .shutdown = &genphy_shutdown,
 };
-
-int phy_dp83867_init(void)
-{
-       phy_register(&DP83867_driver);
-       return 0;
-}
index 23dbf42..8d32d73 100644 (file)
@@ -473,7 +473,7 @@ static int dp83869_probe(struct phy_device *phydev)
        return 0;
 }
 
-static struct phy_driver DP83869_driver = {
+U_BOOT_PHY_DRIVER(dp83869) = {
        .name = "TI DP83869",
        .uid = 0x2000a0f1,
        .mask = 0xfffffff0,
@@ -485,9 +485,3 @@ static struct phy_driver DP83869_driver = {
        .readext = dp83869_readext,
        .writeext = dp83869_writeext
 };
-
-int phy_dp83869_init(void)
-{
-       phy_register(&DP83869_driver);
-       return 0;
-}
index b76cc51..404d61c 100644 (file)
@@ -514,9 +514,6 @@ int phy_init(void)
                phy_drv_reloc(drv);
 #endif
 
-#ifdef CONFIG_PHY_TI
-       phy_ti_init();
-#endif
 #ifdef CONFIG_PHY_VITESSE
        phy_vitesse_init();
 #endif
index 075b19a..a087819 100644 (file)
@@ -10,8 +10,7 @@
 #include <phy.h>
 #include "ti_phy_init.h"
 
-#ifdef CONFIG_PHY_TI_GENERIC
-static struct phy_driver dp83822_driver = {
+U_BOOT_PHY_DRIVER(dp83822) = {
        .name = "TI DP83822",
        .uid = 0x2000a240,
        .mask = 0xfffffff0,
@@ -21,7 +20,7 @@ static struct phy_driver dp83822_driver = {
        .shutdown = &genphy_shutdown,
 };
 
-static struct phy_driver dp83826nc_driver = {
+U_BOOT_PHY_DRIVER(dp83826nc) = {
        .name = "TI DP83826NC",
        .uid = 0x2000a110,
        .mask = 0xfffffff0,
@@ -31,7 +30,7 @@ static struct phy_driver dp83826nc_driver = {
        .shutdown = &genphy_shutdown,
 };
 
-static struct phy_driver dp83826c_driver = {
+U_BOOT_PHY_DRIVER(dp83826c) = {
        .name = "TI DP83826C",
        .uid = 0x2000a130,
        .mask = 0xfffffff0,
@@ -41,7 +40,7 @@ static struct phy_driver dp83826c_driver = {
        .shutdown = &genphy_shutdown,
 };
 
-static struct phy_driver dp83825s_driver = {
+U_BOOT_PHY_DRIVER(dp83825s) = {
        .name = "TI DP83825S",
        .uid = 0x2000a140,
        .mask = 0xfffffff0,
@@ -51,7 +50,7 @@ static struct phy_driver dp83825s_driver = {
        .shutdown = &genphy_shutdown,
 };
 
-static struct phy_driver dp83825i_driver = {
+U_BOOT_PHY_DRIVER(dp83825i) = {
        .name = "TI DP83825I",
        .uid = 0x2000a150,
        .mask = 0xfffffff0,
@@ -61,7 +60,7 @@ static struct phy_driver dp83825i_driver = {
        .shutdown = &genphy_shutdown,
 };
 
-static struct phy_driver dp83825m_driver = {
+U_BOOT_PHY_DRIVER(dp83825m) = {
        .name = "TI DP83825M",
        .uid = 0x2000a160,
        .mask = 0xfffffff0,
@@ -71,7 +70,7 @@ static struct phy_driver dp83825m_driver = {
        .shutdown = &genphy_shutdown,
 };
 
-static struct phy_driver dp83825cs_driver = {
+U_BOOT_PHY_DRIVER(dp83825cs) = {
        .name = "TI DP83825CS",
        .uid = 0x2000a170,
        .mask = 0xfffffff0,
@@ -80,26 +79,3 @@ static struct phy_driver dp83825cs_driver = {
        .startup = &genphy_startup,
        .shutdown = &genphy_shutdown,
 };
-#endif /* CONFIG_PHY_TI_GENERIC */
-
-int phy_ti_init(void)
-{
-#ifdef CONFIG_PHY_TI_DP83867
-       phy_dp83867_init();
-#endif
-
-#ifdef CONFIG_PHY_TI_DP83869
-       phy_dp83869_init();
-#endif
-
-#ifdef CONFIG_PHY_TI_GENERIC
-       phy_register(&dp83822_driver);
-       phy_register(&dp83825s_driver);
-       phy_register(&dp83825i_driver);
-       phy_register(&dp83825m_driver);
-       phy_register(&dp83825cs_driver);
-       phy_register(&dp83826c_driver);
-       phy_register(&dp83826nc_driver);
-#endif
-       return 0;
-}
index 2c6e167..e6508a1 100644 (file)
@@ -315,7 +315,6 @@ int gen10g_startup(struct phy_device *phydev);
 int gen10g_shutdown(struct phy_device *phydev);
 int gen10g_discover_mmds(struct phy_device *phydev);
 
-int phy_ti_init(void);
 int phy_vitesse_init(void);
 int phy_xilinx_init(void);
 int phy_xway_init(void);