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(-) --- 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(struc 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 --- a/nss_dp_main.c +++ b/nss_dp_main.c @@ -875,6 +875,9 @@ static int32_t nss_dp_probe(struct platf 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;