[openwrt/openwrt] realtek: dsa: silence log noise on route offload

[LEDE Commits]

svanheule pushed a commit to openwrt/openwrt.git, branch main:
https://git.openwrt.org/024e9dbace4aa41d713259e1e3c67538d425d577

commit 024e9dbace4aa41d713259e1e3c67538d425d577
Author: Bjørn Mork <bjorn at mork.no>
AuthorDate: Wed Feb 5 07:27:02 2025 +0100

    realtek: dsa: silence log noise on route offload
    
    Adding a static IPv4 route made the driver repeatedly print
    
     rtl83xx_l3_nexthop_update: Setting up fwding: ip 192.168.1.42, GW mac 0000001b21a7xxxx
     Route with id 3 to 192.168.99.0 / 24
     rtl83xx_l3_nexthop_update: total packets: 0
     Warning: TEMPLATE_FIELD_RANGE_CHK: not configured
    
    These messages are only useful to developers while debugging offloading.
    Demote to debug level, which in general is more useful for developers
    by allowing precise dynamic control.
    
    Signed-off-by: Bjørn Mork <bjorn at mork.no>
    Link: https://github.com/openwrt/openwrt/pull/17950
    Signed-off-by: Sander Vanheule <sander at svanheule.net>
---
 target/linux/realtek/files-6.6/drivers/net/dsa/rtl83xx/common.c  | 6 +++---
 target/linux/realtek/files-6.6/drivers/net/dsa/rtl83xx/rtl930x.c | 2 +-
 target/linux/realtek/files-6.6/drivers/net/dsa/rtl83xx/rtl931x.c | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/target/linux/realtek/files-6.6/drivers/net/dsa/rtl83xx/common.c b/target/linux/realtek/files-6.6/drivers/net/dsa/rtl83xx/common.c
index fe0980a1be..28b836f6b2 100644
--- a/target/linux/realtek/files-6.6/drivers/net/dsa/rtl83xx/common.c
+++ b/target/linux/realtek/files-6.6/drivers/net/dsa/rtl83xx/common.c
@@ -813,12 +813,12 @@ static int rtl83xx_l3_nexthop_update(struct rtl838x_switch_priv *priv,  __be32 i
 	}
 
 	rhl_for_each_entry_rcu(r, tmp, list, linkage) {
-		pr_info("%s: Setting up fwding: ip %pI4, GW mac %016llx\n",
+		pr_debug("%s: Setting up fwding: ip %pI4, GW mac %016llx\n",
 			__func__, &ip_addr, mac);
 
 		/* Reads the ROUTING table entry associated with the route */
 		priv->r->route_read(r->id, r);
-		pr_info("Route with id %d to %pI4 / %d\n", r->id, &r->dst_ip, r->prefix_len);
+		pr_debug("Route with id %d to %pI4 / %d\n", r->id, &r->dst_ip, r->prefix_len);
 
 		r->nh.mac = r->nh.gw = mac;
 		r->nh.port = priv->port_ignore;
@@ -865,7 +865,7 @@ static int rtl83xx_l3_nexthop_update(struct rtl838x_switch_priv *priv,  __be32 i
 			priv->r->pie_rule_add(priv, &r->pr);
 		} else {
 			int pkts = priv->r->packet_cntr_read(r->pr.packet_cntr);
-			pr_info("%s: total packets: %d\n", __func__, pkts);
+			pr_debug("%s: total packets: %d\n", __func__, pkts);
 
 			priv->r->pie_rule_write(priv, r->pr.id, &r->pr);
 		}
diff --git a/target/linux/realtek/files-6.6/drivers/net/dsa/rtl83xx/rtl930x.c b/target/linux/realtek/files-6.6/drivers/net/dsa/rtl83xx/rtl930x.c
index 793d762489..3524021e2f 100644
--- a/target/linux/realtek/files-6.6/drivers/net/dsa/rtl83xx/rtl930x.c
+++ b/target/linux/realtek/files-6.6/drivers/net/dsa/rtl83xx/rtl930x.c
@@ -1680,7 +1680,7 @@ static void rtl930x_write_pie_templated(u32 r[], struct pie_rule *pr, enum templ
 			data_m = pr->tcp_info_m;
 			break;
 		case TEMPLATE_FIELD_RANGE_CHK:
-			pr_warn("Warning: TEMPLATE_FIELD_RANGE_CHK: not configured\n");
+			pr_debug("TEMPLATE_FIELD_RANGE_CHK: not configured\n");
 			break;
 		default:
 			pr_info("%s: unknown field %d\n", __func__, field_type);
diff --git a/target/linux/realtek/files-6.6/drivers/net/dsa/rtl83xx/rtl931x.c b/target/linux/realtek/files-6.6/drivers/net/dsa/rtl83xx/rtl931x.c
index 2ba3a7819d..870dbde7a6 100644
--- a/target/linux/realtek/files-6.6/drivers/net/dsa/rtl83xx/rtl931x.c
+++ b/target/linux/realtek/files-6.6/drivers/net/dsa/rtl83xx/rtl931x.c
@@ -1077,10 +1077,10 @@ int rtl931x_pie_data_fill(enum template_field_id field_type, struct pie_rule *pr
 		*data_m = pr->tcp_info_m;
 		break;
 	case TEMPLATE_FIELD_RANGE_CHK:
-		pr_info("TEMPLATE_FIELD_RANGE_CHK: not configured\n");
+		pr_debug("TEMPLATE_FIELD_RANGE_CHK: not configured\n");
 		break;
 	default:
-		pr_info("%s: unknown field %d\n", __func__, field_type);
+		pr_debug("%s: unknown field %d\n", __func__, field_type);
 		return -1;
 	}