drivers: net: phy: at803x code cleanup on register and unregister driver

Make use of phy_drivers_register/phy_drivers_unregister to register/unregister
multiple phy drivers in a single module.

Cc: Matus Ujhelyi <ujhelyi.m@gmail.com>
Signed-off-by: Mugunthan V N <mugunthanvnm@ti.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Mugunthan V N 2013-06-03 20:10:04 +00:00 committed by David S. Miller
parent d862e54614
commit 317420ab23

View file

@ -108,8 +108,9 @@ static int at803x_config_init(struct phy_device *phydev)
return 0; return 0;
} }
/* ATHEROS 8035 */ static struct phy_driver at803x_driver[] = {
static struct phy_driver at8035_driver = { {
/* ATHEROS 8035 */
.phy_id = 0x004dd072, .phy_id = 0x004dd072,
.name = "Atheros 8035 ethernet", .name = "Atheros 8035 ethernet",
.phy_id_mask = 0xffffffef, .phy_id_mask = 0xffffffef,
@ -121,10 +122,8 @@ static struct phy_driver at8035_driver = {
.driver = { .driver = {
.owner = THIS_MODULE, .owner = THIS_MODULE,
}, },
}; }, {
/* ATHEROS 8030 */
/* ATHEROS 8030 */
static struct phy_driver at8030_driver = {
.phy_id = 0x004dd076, .phy_id = 0x004dd076,
.name = "Atheros 8030 ethernet", .name = "Atheros 8030 ethernet",
.phy_id_mask = 0xffffffef, .phy_id_mask = 0xffffffef,
@ -136,32 +135,18 @@ static struct phy_driver at8030_driver = {
.driver = { .driver = {
.owner = THIS_MODULE, .owner = THIS_MODULE,
}, },
}; } };
static int __init atheros_init(void) static int __init atheros_init(void)
{ {
int ret; return phy_drivers_register(at803x_driver,
ARRAY_SIZE(at803x_driver));
ret = phy_driver_register(&at8035_driver);
if (ret)
goto err1;
ret = phy_driver_register(&at8030_driver);
if (ret)
goto err2;
return 0;
err2:
phy_driver_unregister(&at8035_driver);
err1:
return ret;
} }
static void __exit atheros_exit(void) static void __exit atheros_exit(void)
{ {
phy_driver_unregister(&at8035_driver); return phy_drivers_unregister(at803x_driver,
phy_driver_unregister(&at8030_driver); ARRAY_SIZE(at803x_driver));
} }
module_init(atheros_init); module_init(atheros_init);