From 7e4ae2d6285095794d73d2f2ce61404f61d4e633 Mon Sep 17 00:00:00 2001 From: Ansuel Smith Date: Tue, 17 May 2022 15:55:36 +0200 Subject: [PATCH 11/11] treewide: fix confusing printing of registered netdev Net core implementation changed and now printing the netdev name cause confusing printing if done before register_netdev. Move the old printing to dbg and add an additional info log right after register_netdev to give the user some info on correct nss-dp probe. Signed-off-by: Ansuel Smith --- hal/dp_ops/edma_dp/edma_v1/edma_data_plane.c | 4 ++-- nss_dp_main.c | 3 +++ 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/hal/dp_ops/edma_dp/edma_v1/edma_data_plane.c b/hal/dp_ops/edma_dp/edma_v1/edma_data_plane.c index 0af39c2..1d748db 100644 --- a/hal/dp_ops/edma_dp/edma_v1/edma_data_plane.c +++ b/hal/dp_ops/edma_dp/edma_v1/edma_data_plane.c @@ -822,8 +822,8 @@ static int edma_register_netdevice(struct net_device *netdev, uint32_t macid) return -EINVAL; } - netdev_info(netdev, "nss_dp_edma: Registering netdev %s(qcom-id:%d) with EDMA\n", - netdev->name, macid); + netdev_dbg(netdev, "nss_dp_edma: Registering netdev %s(qcom-id:%d) with EDMA\n", + netdev->name, macid); /* * We expect 'macid' to correspond to ports numbers on diff --git a/nss_dp_main.c b/nss_dp_main.c index c0ae9d6..441c300 100644 --- a/nss_dp_main.c +++ b/nss_dp_main.c @@ -875,6 +875,9 @@ static int32_t nss_dp_probe(struct platform_device *pdev) goto phy_setup_fail; } + netdev_info(netdev, "Registered netdev %s(qcom-id:%d)\n", + netdev->name, port_id); + dp_global_ctx.nss_dp[dp_priv->macid - 1] = dp_priv; dp_global_ctx.slowproto_acl_bm = 0; -- 2.34.1