realtek: fix place of fdb/mdb info messages
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>
This commit is contained in:
		 INAGAKI Hiroshi
					INAGAKI Hiroshi
				
			
				
					committed by
					
						 Sander Vanheule
						Sander Vanheule
					
				
			
			
				
	
			
			
			 Sander Vanheule
						Sander Vanheule
					
				
			
						parent
						
							3834e72fa3
						
					
				
				
					commit
					ff307f52f5
				
			| @@ -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); | 	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) { | 	if (idx >= 0) { | ||||||
|  | 		pr_info("Found entry index %d, key %d and bucket %d\n", idx, idx >> 2, idx & 3); | ||||||
| 		e.valid = false; | 		e.valid = false; | ||||||
| 		dump_l2_entry(&e); | 		dump_l2_entry(&e); | ||||||
| 		priv->r->write_l2_entry_using_hash(idx >> 2, idx & 0x3, &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); | 	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) { | 	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); | 		portmask = rtl83xx_mc_group_del_port(priv, e.mc_portmask_index, port); | ||||||
| 		if (!portmask) { | 		if (!portmask) { | ||||||
| 			e.valid = false; | 			e.valid = false; | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user