realtek: fix place of fdb/mdb info messages
authorINAGAKI Hiroshi <musashino.open@gmail.com>
Mon, 4 Apr 2022 12:57:43 +0000 (21:57 +0900)
committerSander Vanheule <sander@svanheule.net>
Sat, 8 Oct 2022 09:05:02 +0000 (11:05 +0200)
Those messages should be printed when entry was found (idx >= 0). Move
them to the right place to not print invalid entry indices.

Signed-off-by: INAGAKI Hiroshi <musashino.open@gmail.com>
[amden commit message]
Signed-off-by: Sander Vanheule <sander@svanheule.net>
target/linux/realtek/files-5.10/drivers/net/dsa/rtl83xx/dsa.c

index f54fdd298ac4e4957fb46be29244a913521c98f9..f08ede730b83cc3412095ce5df501704d1d3d747 100644 (file)
@@ -1689,8 +1689,8 @@ static int rtl83xx_port_fdb_del(struct dsa_switch *ds, int port,
 
        idx = rtl83xx_find_l2_hash_entry(priv, seed, true, &e);
 
-       pr_info("Found entry index %d, key %d and bucket %d\n", idx, idx >> 2, idx & 3);
        if (idx >= 0) {
+               pr_info("Found entry index %d, key %d and bucket %d\n", idx, idx >> 2, idx & 3);
                e.valid = false;
                dump_l2_entry(&e);
                priv->r->write_l2_entry_using_hash(idx >> 2, idx & 0x3, &e);
@@ -1870,8 +1870,8 @@ int rtl83xx_port_mdb_del(struct dsa_switch *ds, int port,
 
        idx = rtl83xx_find_l2_hash_entry(priv, seed, true, &e);
 
-       pr_debug("Found entry index %d, key %d and bucket %d\n", idx, idx >> 2, idx & 3);
        if (idx >= 0) {
+               pr_debug("Found entry index %d, key %d and bucket %d\n", idx, idx >> 2, idx & 3);
                portmask = rtl83xx_mc_group_del_port(priv, e.mc_portmask_index, port);
                if (!portmask) {
                        e.valid = false;