Skip to content

Commit

Permalink
net: dsa: remove NULL check for priv and platform data
Browse files Browse the repository at this point in the history
Because the uclass has the "*_auto" properties set, the driver model
will take care of allocating the private structures for us and they
can't be NULL. Drop the checks.

Signed-off-by: Michael Walle <michael@walle.cc>
Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Reviewed-by: Ramon Fried <rfried.dev@gmail.com>
Reviewed-by: Priyanka Jain <priyanka.jain@nxp.com>
  • Loading branch information
mwalle authored and p-priyanka-jain committed Mar 5, 2021
1 parent e5d7d11 commit 108157c
Showing 1 changed file with 7 additions and 14 deletions.
21 changes: 7 additions & 14 deletions net/dsa-uclass.c
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@ int dsa_set_tagging(struct udevice *dev, ushort headroom, ushort tailroom)
{
struct dsa_priv *priv;

if (!dev || !dev_get_uclass_priv(dev))
return -ENODEV;
if (!dev)
return -EINVAL;

if (headroom + tailroom > DSA_MAX_OVR)
return -EINVAL;
Expand All @@ -47,11 +47,13 @@ int dsa_set_tagging(struct udevice *dev, ushort headroom, ushort tailroom)
/* returns the DSA master Ethernet device */
struct udevice *dsa_get_master(struct udevice *dev)
{
struct dsa_priv *priv = dev_get_uclass_priv(dev);
struct dsa_priv *priv;

if (!priv)
if (!dev)
return NULL;

priv = dev_get_uclass_priv(dev);

return priv->master_dev;
}

Expand All @@ -67,9 +69,6 @@ static int dsa_port_start(struct udevice *pdev)
struct dsa_ops *ops = dsa_get_ops(dev);
int err;

if (!priv)
return -ENODEV;

if (!master) {
dev_err(pdev, "DSA master Ethernet device not found!\n");
return -EINVAL;
Expand Down Expand Up @@ -101,9 +100,6 @@ static void dsa_port_stop(struct udevice *pdev)
struct udevice *master = dsa_get_master(dev);
struct dsa_ops *ops = dsa_get_ops(dev);

if (!priv)
return;

if (ops->port_disable) {
struct dsa_port_pdata *port_pdata;

Expand Down Expand Up @@ -347,7 +343,7 @@ static int dsa_post_bind(struct udevice *dev)
ofnode node = dev_ofnode(dev), pnode;
int i, err, first_err = 0;

if (!pdata || !ofnode_valid(node))
if (!ofnode_valid(node))
return -ENODEV;

pdata->master_node = ofnode_null();
Expand Down Expand Up @@ -459,9 +455,6 @@ static int dsa_pre_probe(struct udevice *dev)
struct dsa_pdata *pdata = dev_get_uclass_plat(dev);
struct dsa_priv *priv = dev_get_uclass_priv(dev);

if (!pdata || !priv)
return -ENODEV;

priv->num_ports = pdata->num_ports;
priv->cpu_port = pdata->cpu_port;
priv->cpu_port_fixed_phy = fixed_phy_create(pdata->cpu_port_node);
Expand Down

0 comments on commit 108157c

Please sign in to comment.