diff options
| author |   Ansuel Smith <ansuelsmth@gmail.com> | 2021-11-22 16:23:40 +0100 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2021-11-22 15:35:16 +0000 |
| commit | b9133f3ef5a2659730cf47a74bd0a9259f1cf8ff (patch) | |
| tree | 5bbeb67eb0fc36e11c868d85fb7c943e5c4ec700 /drivers/net/dsa/qca8k.c | |
| parent | Merge branch 'skbuff-struct-group' (diff) | |
| download | linux-dev-b9133f3ef5a2659730cf47a74bd0a9259f1cf8ff.tar.xz linux-dev-b9133f3ef5a2659730cf47a74bd0a9259f1cf8ff.zip | |
net: dsa: qca8k: remove redundant check in parse_port_config
The very next check for port 0 and 6 already makes sure we don't go out
of bounds with the ports_config delay table.
Remove the redundant check.
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
Reviewed-by: Vladimir Oltean <olteanv@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/dsa/qca8k.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/dsa/qca8k.c b/drivers/net/dsa/qca8k.c index a429c9750add..bfffc1fb7016 100644 --- a/drivers/net/dsa/qca8k.c +++ b/drivers/net/dsa/qca8k.c @@ -983,7 +983,7 @@ qca8k_parse_port_config(struct qca8k_priv *priv) u32 delay; /* We have 2 CPU port. Check them */ - for (port = 0; port < QCA8K_NUM_PORTS && cpu_port_index < QCA8K_NUM_CPU_PORTS; port++) { + for (port = 0; port < QCA8K_NUM_PORTS; port++) { /* Skip every other port */ if (port != 0 && port != 6) continue; |