diff options
| author |   Vladimir Oltean <olteanv@gmail.com> | 2019-06-03 00:11:59 +0300 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2019-06-04 11:49:19 -0700 |
| commit | def846042f13cb0feabb86375e824c648543700a (patch) | |
| tree | b3f45aef9f1e74799f31468c0cdcef1d52a62846 /drivers/net/dsa/sja1105 | |
| parent | net: dsa: sja1105: Add P/Q/R/S support for dynamic L2 lookup operations (diff) | |
| download | linux-dev-def846042f13cb0feabb86375e824c648543700a.tar.xz linux-dev-def846042f13cb0feabb86375e824c648543700a.zip | |
net: dsa: sja1105: Make dynamic_config_read return -ENOENT if not found
Conceptually, if an entry is not found in the requested hardware table,
it is not an invalid request - so change the error returned
appropriately.
Signed-off-by: Vladimir Oltean <olteanv@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/dsa/sja1105')
| -rw-r--r-- | drivers/net/dsa/sja1105/sja1105_dynamic_config.c | 2 | ||||
| -rw-r--r-- | drivers/net/dsa/sja1105/sja1105_main.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/dsa/sja1105/sja1105_dynamic_config.c b/drivers/net/dsa/sja1105/sja1105_dynamic_config.c index 7db1f8258287..02a67df4437e 100644 --- a/drivers/net/dsa/sja1105/sja1105_dynamic_config.c +++ b/drivers/net/dsa/sja1105/sja1105_dynamic_config.c @@ -502,7 +502,7 @@ int sja1105_dynamic_config_read(struct sja1105_private *priv, * So don't error out in that case. */ if (!cmd.valident && blk_idx != BLK_IDX_MGMT_ROUTE) - return -EINVAL; + return -ENOENT; cpu_relax(); } while (cmd.valid && --retries); diff --git a/drivers/net/dsa/sja1105/sja1105_main.c b/drivers/net/dsa/sja1105/sja1105_main.c index c78d2def52f1..dc9803efdbbd 100644 --- a/drivers/net/dsa/sja1105/sja1105_main.c +++ b/drivers/net/dsa/sja1105/sja1105_main.c @@ -948,7 +948,7 @@ static int sja1105_fdb_dump(struct dsa_switch *ds, int port, rc = sja1105_dynamic_config_read(priv, BLK_IDX_L2_LOOKUP, i, &l2_lookup); /* No fdb entry at i, not an issue */ - if (rc == -EINVAL) + if (rc == -ENOENT) continue; if (rc) { dev_err(dev, "Failed to dump FDB: %d\n", rc); |