Commit ab3d408d authored by Florian Fainelli's avatar Florian Fainelli Committed by David S. Miller

net: dsa: Encapsulate legacy switch drivers into dsa_switch_driver

In preparation for making struct dsa_switch_ops const, encapsulate it
within a dsa_switch_driver which has a list pointer and a pointer to
dsa_switch_ops. This allows us to take the list_head pointer out of
dsa_switch_ops, which is written to by {un,}register_switch_driver.
Signed-off-by: default avatarFlorian Fainelli <f.fainelli@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 73095cb1
...@@ -261,16 +261,20 @@ static struct dsa_switch_ops mv88e6060_switch_ops = { ...@@ -261,16 +261,20 @@ static struct dsa_switch_ops mv88e6060_switch_ops = {
.phy_write = mv88e6060_phy_write, .phy_write = mv88e6060_phy_write,
}; };
static struct dsa_switch_driver mv88e6060_switch_drv = {
.ops = &mv88e6060_switch_ops,
};
static int __init mv88e6060_init(void) static int __init mv88e6060_init(void)
{ {
register_switch_driver(&mv88e6060_switch_ops); register_switch_driver(&mv88e6060_switch_drv);
return 0; return 0;
} }
module_init(mv88e6060_init); module_init(mv88e6060_init);
static void __exit mv88e6060_cleanup(void) static void __exit mv88e6060_cleanup(void)
{ {
unregister_switch_driver(&mv88e6060_switch_ops); unregister_switch_driver(&mv88e6060_switch_drv);
} }
module_exit(mv88e6060_cleanup); module_exit(mv88e6060_cleanup);
......
...@@ -4403,6 +4403,10 @@ static struct dsa_switch_ops mv88e6xxx_switch_ops = { ...@@ -4403,6 +4403,10 @@ static struct dsa_switch_ops mv88e6xxx_switch_ops = {
.port_mdb_dump = mv88e6xxx_port_mdb_dump, .port_mdb_dump = mv88e6xxx_port_mdb_dump,
}; };
static struct dsa_switch_driver mv88e6xxx_switch_drv = {
.ops = &mv88e6xxx_switch_ops,
};
static int mv88e6xxx_register_switch(struct mv88e6xxx_chip *chip, static int mv88e6xxx_register_switch(struct mv88e6xxx_chip *chip,
struct device_node *np) struct device_node *np)
{ {
...@@ -4565,7 +4569,7 @@ static struct mdio_driver mv88e6xxx_driver = { ...@@ -4565,7 +4569,7 @@ static struct mdio_driver mv88e6xxx_driver = {
static int __init mv88e6xxx_init(void) static int __init mv88e6xxx_init(void)
{ {
register_switch_driver(&mv88e6xxx_switch_ops); register_switch_driver(&mv88e6xxx_switch_drv);
return mdio_driver_register(&mv88e6xxx_driver); return mdio_driver_register(&mv88e6xxx_driver);
} }
module_init(mv88e6xxx_init); module_init(mv88e6xxx_init);
...@@ -4573,7 +4577,7 @@ module_init(mv88e6xxx_init); ...@@ -4573,7 +4577,7 @@ module_init(mv88e6xxx_init);
static void __exit mv88e6xxx_cleanup(void) static void __exit mv88e6xxx_cleanup(void)
{ {
mdio_driver_unregister(&mv88e6xxx_driver); mdio_driver_unregister(&mv88e6xxx_driver);
unregister_switch_driver(&mv88e6xxx_switch_ops); unregister_switch_driver(&mv88e6xxx_switch_drv);
} }
module_exit(mv88e6xxx_cleanup); module_exit(mv88e6xxx_cleanup);
......
...@@ -240,8 +240,6 @@ struct switchdev_obj_port_mdb; ...@@ -240,8 +240,6 @@ struct switchdev_obj_port_mdb;
struct switchdev_obj_port_vlan; struct switchdev_obj_port_vlan;
struct dsa_switch_ops { struct dsa_switch_ops {
struct list_head list;
/* /*
* Probing and setup. * Probing and setup.
*/ */
...@@ -390,8 +388,13 @@ struct dsa_switch_ops { ...@@ -390,8 +388,13 @@ struct dsa_switch_ops {
int (*cb)(struct switchdev_obj *obj)); int (*cb)(struct switchdev_obj *obj));
}; };
void register_switch_driver(struct dsa_switch_ops *type); struct dsa_switch_driver {
void unregister_switch_driver(struct dsa_switch_ops *type); struct list_head list;
struct dsa_switch_ops *ops;
};
void register_switch_driver(struct dsa_switch_driver *type);
void unregister_switch_driver(struct dsa_switch_driver *type);
struct mii_bus *dsa_host_dev_to_mii_bus(struct device *dev); struct mii_bus *dsa_host_dev_to_mii_bus(struct device *dev);
static inline bool dsa_uses_tagged_protocol(struct dsa_switch_tree *dst) static inline bool dsa_uses_tagged_protocol(struct dsa_switch_tree *dst)
......
...@@ -60,18 +60,18 @@ const struct dsa_device_ops *dsa_device_ops[DSA_TAG_LAST] = { ...@@ -60,18 +60,18 @@ const struct dsa_device_ops *dsa_device_ops[DSA_TAG_LAST] = {
static DEFINE_MUTEX(dsa_switch_drivers_mutex); static DEFINE_MUTEX(dsa_switch_drivers_mutex);
static LIST_HEAD(dsa_switch_drivers); static LIST_HEAD(dsa_switch_drivers);
void register_switch_driver(struct dsa_switch_ops *ops) void register_switch_driver(struct dsa_switch_driver *drv)
{ {
mutex_lock(&dsa_switch_drivers_mutex); mutex_lock(&dsa_switch_drivers_mutex);
list_add_tail(&ops->list, &dsa_switch_drivers); list_add_tail(&drv->list, &dsa_switch_drivers);
mutex_unlock(&dsa_switch_drivers_mutex); mutex_unlock(&dsa_switch_drivers_mutex);
} }
EXPORT_SYMBOL_GPL(register_switch_driver); EXPORT_SYMBOL_GPL(register_switch_driver);
void unregister_switch_driver(struct dsa_switch_ops *ops) void unregister_switch_driver(struct dsa_switch_driver *drv)
{ {
mutex_lock(&dsa_switch_drivers_mutex); mutex_lock(&dsa_switch_drivers_mutex);
list_del_init(&ops->list); list_del_init(&drv->list);
mutex_unlock(&dsa_switch_drivers_mutex); mutex_unlock(&dsa_switch_drivers_mutex);
} }
EXPORT_SYMBOL_GPL(unregister_switch_driver); EXPORT_SYMBOL_GPL(unregister_switch_driver);
...@@ -90,8 +90,10 @@ dsa_switch_probe(struct device *parent, struct device *host_dev, int sw_addr, ...@@ -90,8 +90,10 @@ dsa_switch_probe(struct device *parent, struct device *host_dev, int sw_addr,
mutex_lock(&dsa_switch_drivers_mutex); mutex_lock(&dsa_switch_drivers_mutex);
list_for_each(list, &dsa_switch_drivers) { list_for_each(list, &dsa_switch_drivers) {
struct dsa_switch_ops *ops; struct dsa_switch_ops *ops;
struct dsa_switch_driver *drv;
ops = list_entry(list, struct dsa_switch_ops, list); drv = list_entry(list, struct dsa_switch_driver, list);
ops = drv->ops;
name = ops->probe(parent, host_dev, sw_addr, priv); name = ops->probe(parent, host_dev, sw_addr, priv);
if (name != NULL) { if (name != NULL) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment