Commit a0c02161 authored by Vivien Didelot's avatar Vivien Didelot Committed by David S. Miller

net: dsa: variable number of ports

Change the ports[DSA_MAX_PORTS] array of the dsa_switch structure for a
zero-length array, allocated at the same time as the dsa_switch
structure itself. A dsa_switch_alloc() helper is provided for that.

This commit brings no functional change yet since we pass DSA_MAX_PORTS
as the number of ports for the moment. Future patches can update the DSA
drivers separately to support dynamic number of ports.
Signed-off-by: default avatarVivien Didelot <vivien.didelot@savoirfairelinux.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 1f5d492a
...@@ -1790,14 +1790,15 @@ struct b53_device *b53_switch_alloc(struct device *base, ...@@ -1790,14 +1790,15 @@ struct b53_device *b53_switch_alloc(struct device *base,
struct dsa_switch *ds; struct dsa_switch *ds;
struct b53_device *dev; struct b53_device *dev;
ds = devm_kzalloc(base, sizeof(*ds) + sizeof(*dev), GFP_KERNEL); ds = dsa_switch_alloc(base, DSA_MAX_PORTS);
if (!ds) if (!ds)
return NULL; return NULL;
dev = (struct b53_device *)(ds + 1); dev = devm_kzalloc(base, sizeof(*dev), GFP_KERNEL);
if (!dev)
return NULL;
ds->priv = dev; ds->priv = dev;
ds->dev = base;
dev->dev = base; dev->dev = base;
dev->ds = ds; dev->ds = ds;
......
...@@ -4361,11 +4361,10 @@ static int mv88e6xxx_register_switch(struct mv88e6xxx_chip *chip) ...@@ -4361,11 +4361,10 @@ static int mv88e6xxx_register_switch(struct mv88e6xxx_chip *chip)
struct device *dev = chip->dev; struct device *dev = chip->dev;
struct dsa_switch *ds; struct dsa_switch *ds;
ds = devm_kzalloc(dev, sizeof(*ds), GFP_KERNEL); ds = dsa_switch_alloc(dev, DSA_MAX_PORTS);
if (!ds) if (!ds)
return -ENOMEM; return -ENOMEM;
ds->dev = dev;
ds->priv = chip; ds->priv = chip;
ds->ops = &mv88e6xxx_switch_ops; ds->ops = &mv88e6xxx_switch_ops;
......
...@@ -954,12 +954,11 @@ qca8k_sw_probe(struct mdio_device *mdiodev) ...@@ -954,12 +954,11 @@ qca8k_sw_probe(struct mdio_device *mdiodev)
if (id != QCA8K_ID_QCA8337) if (id != QCA8K_ID_QCA8337)
return -ENODEV; return -ENODEV;
priv->ds = devm_kzalloc(&mdiodev->dev, sizeof(*priv->ds), GFP_KERNEL); priv->ds = dsa_switch_alloc(&mdiodev->dev, DSA_MAX_PORTS);
if (!priv->ds) if (!priv->ds)
return -ENOMEM; return -ENOMEM;
priv->ds->priv = priv; priv->ds->priv = priv;
priv->ds->dev = &mdiodev->dev;
priv->ds->ops = &qca8k_switch_ops; priv->ds->ops = &qca8k_switch_ops;
mutex_init(&priv->reg_mutex); mutex_init(&priv->reg_mutex);
dev_set_drvdata(&mdiodev->dev, priv); dev_set_drvdata(&mdiodev->dev, priv);
......
...@@ -190,8 +190,11 @@ struct dsa_switch { ...@@ -190,8 +190,11 @@ struct dsa_switch {
u32 cpu_port_mask; u32 cpu_port_mask;
u32 enabled_port_mask; u32 enabled_port_mask;
u32 phys_mii_mask; u32 phys_mii_mask;
struct dsa_port ports[DSA_MAX_PORTS];
struct mii_bus *slave_mii_bus; struct mii_bus *slave_mii_bus;
/* Dynamically allocated ports, keep last */
size_t num_ports;
struct dsa_port ports[];
}; };
static inline bool dsa_is_cpu_port(struct dsa_switch *ds, int p) static inline bool dsa_is_cpu_port(struct dsa_switch *ds, int p)
...@@ -386,6 +389,7 @@ static inline bool dsa_uses_tagged_protocol(struct dsa_switch_tree *dst) ...@@ -386,6 +389,7 @@ static inline bool dsa_uses_tagged_protocol(struct dsa_switch_tree *dst)
return dst->rcv != NULL; return dst->rcv != NULL;
} }
struct dsa_switch *dsa_switch_alloc(struct device *dev, size_t n);
void dsa_unregister_switch(struct dsa_switch *ds); void dsa_unregister_switch(struct dsa_switch *ds);
int dsa_register_switch(struct dsa_switch *ds, struct device *dev); int dsa_register_switch(struct dsa_switch *ds, struct device *dev);
#ifdef CONFIG_PM_SLEEP #ifdef CONFIG_PM_SLEEP
......
...@@ -347,8 +347,8 @@ dsa_switch_setup(struct dsa_switch_tree *dst, int index, ...@@ -347,8 +347,8 @@ dsa_switch_setup(struct dsa_switch_tree *dst, int index,
/* /*
* Allocate and initialise switch state. * Allocate and initialise switch state.
*/ */
ds = devm_kzalloc(parent, sizeof(*ds), GFP_KERNEL); ds = dsa_switch_alloc(parent, DSA_MAX_PORTS);
if (ds == NULL) if (!ds)
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);
ds->dst = dst; ds->dst = dst;
...@@ -356,7 +356,6 @@ dsa_switch_setup(struct dsa_switch_tree *dst, int index, ...@@ -356,7 +356,6 @@ dsa_switch_setup(struct dsa_switch_tree *dst, int index,
ds->cd = cd; ds->cd = cd;
ds->ops = ops; ds->ops = ops;
ds->priv = priv; ds->priv = priv;
ds->dev = parent;
ret = dsa_switch_setup_one(ds, parent); ret = dsa_switch_setup_one(ds, parent);
if (ret) if (ret)
......
...@@ -666,6 +666,22 @@ static int _dsa_register_switch(struct dsa_switch *ds, struct device *dev) ...@@ -666,6 +666,22 @@ static int _dsa_register_switch(struct dsa_switch *ds, struct device *dev)
return err; return err;
} }
struct dsa_switch *dsa_switch_alloc(struct device *dev, size_t n)
{
size_t size = sizeof(struct dsa_switch) + n * sizeof(struct dsa_port);
struct dsa_switch *ds;
ds = devm_kzalloc(dev, size, GFP_KERNEL);
if (!ds)
return NULL;
ds->dev = dev;
ds->num_ports = n;
return ds;
}
EXPORT_SYMBOL_GPL(dsa_switch_alloc);
int dsa_register_switch(struct dsa_switch *ds, struct device *dev) int dsa_register_switch(struct dsa_switch *ds, struct device *dev)
{ {
int err; int err;
......
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