diff options
author | Joe Perches <joe@perches.com> | 2011-08-14 08:16:20 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-08-17 18:47:23 -0400 |
commit | 94f05b0f60de32e6efa19310bd142f1519e2abdb (patch) | |
tree | 9d7f9b70833d1acdb0798d68b49c6cd309cefc58 /drivers/net | |
parent | 0f9dad10a040fa72c588db46a94c9e96545cc509 (diff) |
bnx2x: Coalesce pr_cont uses and fix DP typos
Uses of pr_cont should be avoided where reasonably possible
because they can be interleaved by other threads and processes.
Coalesce pr_cont uses.
Fix typos, duplicated words and spacing in DP uses caused
by split multi-line formats. Coalesce some of these
split formats. Add missing terminating newlines to DP uses.
Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Eilon Greenstein <eilong@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | 42 | ||||
-rw-r--r-- | drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.h | 4 | ||||
-rw-r--r-- | drivers/net/ethernet/broadcom/bnx2x/bnx2x_dcb.c | 2 | ||||
-rw-r--r-- | drivers/net/ethernet/broadcom/bnx2x/bnx2x_link.c | 153 | ||||
-rw-r--r-- | drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c | 33 | ||||
-rw-r--r-- | drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c | 23 |
6 files changed, 134 insertions, 123 deletions
diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c index d724a18b528..3254b9e7c2e 100644 --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | |||
@@ -953,15 +953,16 @@ void __bnx2x_link_report(struct bnx2x *bp) | |||
953 | netdev_err(bp->dev, "NIC Link is Down\n"); | 953 | netdev_err(bp->dev, "NIC Link is Down\n"); |
954 | return; | 954 | return; |
955 | } else { | 955 | } else { |
956 | const char *duplex; | ||
957 | const char *flow; | ||
958 | |||
956 | netif_carrier_on(bp->dev); | 959 | netif_carrier_on(bp->dev); |
957 | netdev_info(bp->dev, "NIC Link is Up, "); | ||
958 | pr_cont("%d Mbps ", cur_data.line_speed); | ||
959 | 960 | ||
960 | if (test_and_clear_bit(BNX2X_LINK_REPORT_FD, | 961 | if (test_and_clear_bit(BNX2X_LINK_REPORT_FD, |
961 | &cur_data.link_report_flags)) | 962 | &cur_data.link_report_flags)) |
962 | pr_cont("full duplex"); | 963 | duplex = "full"; |
963 | else | 964 | else |
964 | pr_cont("half duplex"); | 965 | duplex = "half"; |
965 | 966 | ||
966 | /* Handle the FC at the end so that only these flags would be | 967 | /* Handle the FC at the end so that only these flags would be |
967 | * possibly set. This way we may easily check if there is no FC | 968 | * possibly set. This way we may easily check if there is no FC |
@@ -970,16 +971,19 @@ void __bnx2x_link_report(struct bnx2x *bp) | |||
970 | if (cur_data.link_report_flags) { | 971 | if (cur_data.link_report_flags) { |
971 | if (test_bit(BNX2X_LINK_REPORT_RX_FC_ON, | 972 | if (test_bit(BNX2X_LINK_REPORT_RX_FC_ON, |
972 | &cur_data.link_report_flags)) { | 973 | &cur_data.link_report_flags)) { |
973 | pr_cont(", receive "); | ||
974 | if (test_bit(BNX2X_LINK_REPORT_TX_FC_ON, | 974 | if (test_bit(BNX2X_LINK_REPORT_TX_FC_ON, |
975 | &cur_data.link_report_flags)) | 975 | &cur_data.link_report_flags)) |
976 | pr_cont("& transmit "); | 976 | flow = "ON - receive & transmit"; |
977 | else | ||
978 | flow = "ON - receive"; | ||
977 | } else { | 979 | } else { |
978 | pr_cont(", transmit "); | 980 | flow = "ON - transmit"; |
979 | } | 981 | } |
980 | pr_cont("flow control ON"); | 982 | } else { |
983 | flow = "none"; | ||
981 | } | 984 | } |
982 | pr_cont("\n"); | 985 | netdev_info(bp->dev, "NIC Link is Up, %d Mbps %s duplex, Flow control: %s\n", |
986 | cur_data.line_speed, duplex, flow); | ||
983 | } | 987 | } |
984 | } | 988 | } |
985 | 989 | ||
@@ -2584,7 +2588,7 @@ netdev_tx_t bnx2x_start_xmit(struct sk_buff *skb, struct net_device *dev) | |||
2584 | #endif | 2588 | #endif |
2585 | 2589 | ||
2586 | /* enable this debug print to view the transmission queue being used | 2590 | /* enable this debug print to view the transmission queue being used |
2587 | DP(BNX2X_MSG_FP, "indices: txq %d, fp %d, txdata %d", | 2591 | DP(BNX2X_MSG_FP, "indices: txq %d, fp %d, txdata %d\n", |
2588 | txq_index, fp_index, txdata_index); */ | 2592 | txq_index, fp_index, txdata_index); */ |
2589 | 2593 | ||
2590 | /* locate the fastpath and the txdata */ | 2594 | /* locate the fastpath and the txdata */ |
@@ -2593,7 +2597,7 @@ netdev_tx_t bnx2x_start_xmit(struct sk_buff *skb, struct net_device *dev) | |||
2593 | 2597 | ||
2594 | /* enable this debug print to view the tranmission details | 2598 | /* enable this debug print to view the tranmission details |
2595 | DP(BNX2X_MSG_FP,"transmitting packet cid %d fp index %d txdata_index %d" | 2599 | DP(BNX2X_MSG_FP,"transmitting packet cid %d fp index %d txdata_index %d" |
2596 | " tx_data ptr %p fp pointer %p", | 2600 | " tx_data ptr %p fp pointer %p\n", |
2597 | txdata->cid, fp_index, txdata_index, txdata, fp); */ | 2601 | txdata->cid, fp_index, txdata_index, txdata, fp); */ |
2598 | 2602 | ||
2599 | if (unlikely(bnx2x_tx_avail(bp, txdata) < | 2603 | if (unlikely(bnx2x_tx_avail(bp, txdata) < |
@@ -2910,14 +2914,14 @@ int bnx2x_setup_tc(struct net_device *dev, u8 num_tc) | |||
2910 | /* requested to support too many traffic classes */ | 2914 | /* requested to support too many traffic classes */ |
2911 | if (num_tc > bp->max_cos) { | 2915 | if (num_tc > bp->max_cos) { |
2912 | DP(NETIF_MSG_TX_ERR, "support for too many traffic classes" | 2916 | DP(NETIF_MSG_TX_ERR, "support for too many traffic classes" |
2913 | " requested: %d. max supported is %d", | 2917 | " requested: %d. max supported is %d\n", |
2914 | num_tc, bp->max_cos); | 2918 | num_tc, bp->max_cos); |
2915 | return -EINVAL; | 2919 | return -EINVAL; |
2916 | } | 2920 | } |
2917 | 2921 | ||
2918 | /* declare amount of supported traffic classes */ | 2922 | /* declare amount of supported traffic classes */ |
2919 | if (netdev_set_num_tc(dev, num_tc)) { | 2923 | if (netdev_set_num_tc(dev, num_tc)) { |
2920 | DP(NETIF_MSG_TX_ERR, "failed to declare %d traffic classes", | 2924 | DP(NETIF_MSG_TX_ERR, "failed to declare %d traffic classes\n", |
2921 | num_tc); | 2925 | num_tc); |
2922 | return -EINVAL; | 2926 | return -EINVAL; |
2923 | } | 2927 | } |
@@ -2925,7 +2929,7 @@ int bnx2x_setup_tc(struct net_device *dev, u8 num_tc) | |||
2925 | /* configure priority to traffic class mapping */ | 2929 | /* configure priority to traffic class mapping */ |
2926 | for (prio = 0; prio < BNX2X_MAX_PRIORITY; prio++) { | 2930 | for (prio = 0; prio < BNX2X_MAX_PRIORITY; prio++) { |
2927 | netdev_set_prio_tc_map(dev, prio, bp->prio_to_cos[prio]); | 2931 | netdev_set_prio_tc_map(dev, prio, bp->prio_to_cos[prio]); |
2928 | DP(BNX2X_MSG_SP, "mapping priority %d to tc %d", | 2932 | DP(BNX2X_MSG_SP, "mapping priority %d to tc %d\n", |
2929 | prio, bp->prio_to_cos[prio]); | 2933 | prio, bp->prio_to_cos[prio]); |
2930 | } | 2934 | } |
2931 | 2935 | ||
@@ -2934,10 +2938,10 @@ int bnx2x_setup_tc(struct net_device *dev, u8 num_tc) | |||
2934 | This can be used for ets or pfc, and save the effort of setting | 2938 | This can be used for ets or pfc, and save the effort of setting |
2935 | up a multio class queue disc or negotiating DCBX with a switch | 2939 | up a multio class queue disc or negotiating DCBX with a switch |
2936 | netdev_set_prio_tc_map(dev, 0, 0); | 2940 | netdev_set_prio_tc_map(dev, 0, 0); |
2937 | DP(BNX2X_MSG_SP, "mapping priority %d to tc %d", 0, 0); | 2941 | DP(BNX2X_MSG_SP, "mapping priority %d to tc %d\n", 0, 0); |
2938 | for (prio = 1; prio < 16; prio++) { | 2942 | for (prio = 1; prio < 16; prio++) { |
2939 | netdev_set_prio_tc_map(dev, prio, 1); | 2943 | netdev_set_prio_tc_map(dev, prio, 1); |
2940 | DP(BNX2X_MSG_SP, "mapping priority %d to tc %d", prio, 1); | 2944 | DP(BNX2X_MSG_SP, "mapping priority %d to tc %d\n", prio, 1); |
2941 | } */ | 2945 | } */ |
2942 | 2946 | ||
2943 | /* configure traffic class to transmission queue mapping */ | 2947 | /* configure traffic class to transmission queue mapping */ |
@@ -2945,7 +2949,7 @@ int bnx2x_setup_tc(struct net_device *dev, u8 num_tc) | |||
2945 | count = BNX2X_NUM_ETH_QUEUES(bp); | 2949 | count = BNX2X_NUM_ETH_QUEUES(bp); |
2946 | offset = cos * MAX_TXQS_PER_COS; | 2950 | offset = cos * MAX_TXQS_PER_COS; |
2947 | netdev_set_tc_queue(dev, cos, count, offset); | 2951 | netdev_set_tc_queue(dev, cos, count, offset); |
2948 | DP(BNX2X_MSG_SP, "mapping tc %d to offset %d count %d", | 2952 | DP(BNX2X_MSG_SP, "mapping tc %d to offset %d count %d\n", |
2949 | cos, offset, count); | 2953 | cos, offset, count); |
2950 | } | 2954 | } |
2951 | 2955 | ||
@@ -3033,7 +3037,7 @@ static void bnx2x_free_fp_mem_at(struct bnx2x *bp, int fp_index) | |||
3033 | struct bnx2x_fp_txdata *txdata = &fp->txdata[cos]; | 3037 | struct bnx2x_fp_txdata *txdata = &fp->txdata[cos]; |
3034 | 3038 | ||
3035 | DP(BNX2X_MSG_SP, | 3039 | DP(BNX2X_MSG_SP, |
3036 | "freeing tx memory of fp %d cos %d cid %d", | 3040 | "freeing tx memory of fp %d cos %d cid %d\n", |
3037 | fp_index, cos, txdata->cid); | 3041 | fp_index, cos, txdata->cid); |
3038 | 3042 | ||
3039 | BNX2X_FREE(txdata->tx_buf_ring); | 3043 | BNX2X_FREE(txdata->tx_buf_ring); |
@@ -3115,7 +3119,7 @@ static int bnx2x_alloc_fp_mem_at(struct bnx2x *bp, int index) | |||
3115 | struct bnx2x_fp_txdata *txdata = &fp->txdata[cos]; | 3119 | struct bnx2x_fp_txdata *txdata = &fp->txdata[cos]; |
3116 | 3120 | ||
3117 | DP(BNX2X_MSG_SP, "allocating tx memory of " | 3121 | DP(BNX2X_MSG_SP, "allocating tx memory of " |
3118 | "fp %d cos %d", | 3122 | "fp %d cos %d\n", |
3119 | index, cos); | 3123 | index, cos); |
3120 | 3124 | ||
3121 | BNX2X_ALLOC(txdata->tx_buf_ring, | 3125 | BNX2X_ALLOC(txdata->tx_buf_ring, |
diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.h b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.h index 223bfeebc59..501a24b4767 100644 --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.h +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.h | |||
@@ -1289,7 +1289,7 @@ static inline void bnx2x_init_txdata(struct bnx2x *bp, | |||
1289 | txdata->txq_index = txq_index; | 1289 | txdata->txq_index = txq_index; |
1290 | txdata->tx_cons_sb = tx_cons_sb; | 1290 | txdata->tx_cons_sb = tx_cons_sb; |
1291 | 1291 | ||
1292 | DP(BNX2X_MSG_SP, "created tx data cid %d, txq %d", | 1292 | DP(BNX2X_MSG_SP, "created tx data cid %d, txq %d\n", |
1293 | txdata->cid, txdata->txq_index); | 1293 | txdata->cid, txdata->txq_index); |
1294 | } | 1294 | } |
1295 | 1295 | ||
@@ -1333,7 +1333,7 @@ static inline void bnx2x_init_fcoe_fp(struct bnx2x *bp) | |||
1333 | bnx2x_init_txdata(bp, &bnx2x_fcoe(bp, txdata[0]), | 1333 | bnx2x_init_txdata(bp, &bnx2x_fcoe(bp, txdata[0]), |
1334 | fp->cid, FCOE_TXQ_IDX(bp), BNX2X_FCOE_L2_TX_INDEX); | 1334 | fp->cid, FCOE_TXQ_IDX(bp), BNX2X_FCOE_L2_TX_INDEX); |
1335 | 1335 | ||
1336 | DP(BNX2X_MSG_SP, "created fcoe tx data (fp index %d)", fp->index); | 1336 | DP(BNX2X_MSG_SP, "created fcoe tx data (fp index %d)\n", fp->index); |
1337 | 1337 | ||
1338 | /* qZone id equals to FW (per path) client id */ | 1338 | /* qZone id equals to FW (per path) client id */ |
1339 | bnx2x_fcoe(bp, cl_qzone_id) = bnx2x_fp_qzone_id(fp); | 1339 | bnx2x_fcoe(bp, cl_qzone_id) = bnx2x_fp_qzone_id(fp); |
diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_dcb.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_dcb.c index a4ea35f6a45..38b5ca527a3 100644 --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_dcb.c +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_dcb.c | |||
@@ -350,7 +350,7 @@ static void bnx2x_dcbx_map_nw(struct bnx2x *bp) | |||
350 | if (cos_params[i].pri_bitmask & nw_prio) { | 350 | if (cos_params[i].pri_bitmask & nw_prio) { |
351 | /* extend the bitmask with unmapped */ | 351 | /* extend the bitmask with unmapped */ |
352 | DP(NETIF_MSG_LINK, | 352 | DP(NETIF_MSG_LINK, |
353 | "cos %d extended with 0x%08x", i, unmapped); | 353 | "cos %d extended with 0x%08x\n", i, unmapped); |
354 | cos_params[i].pri_bitmask |= unmapped; | 354 | cos_params[i].pri_bitmask |= unmapped; |
355 | break; | 355 | break; |
356 | } | 356 | } |
diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_link.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_link.c index d45b1555a60..e3de6fedf21 100644 --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_link.c +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_link.c | |||
@@ -694,8 +694,8 @@ static int bnx2x_ets_e3b0_disabled(const struct link_params *params, | |||
694 | struct bnx2x *bp = params->bp; | 694 | struct bnx2x *bp = params->bp; |
695 | 695 | ||
696 | if (!CHIP_IS_E3B0(bp)) { | 696 | if (!CHIP_IS_E3B0(bp)) { |
697 | DP(NETIF_MSG_LINK, "bnx2x_ets_e3b0_disabled the chip isn't E3B0" | 697 | DP(NETIF_MSG_LINK, |
698 | "\n"); | 698 | "bnx2x_ets_e3b0_disabled the chip isn't E3B0\n"); |
699 | return -EINVAL; | 699 | return -EINVAL; |
700 | } | 700 | } |
701 | 701 | ||
@@ -854,8 +854,8 @@ static int bnx2x_ets_e3b0_get_total_bw( | |||
854 | if (bnx2x_cos_state_bw == ets_params->cos[cos_idx].state) { | 854 | if (bnx2x_cos_state_bw == ets_params->cos[cos_idx].state) { |
855 | 855 | ||
856 | if (0 == ets_params->cos[cos_idx].params.bw_params.bw) { | 856 | if (0 == ets_params->cos[cos_idx].params.bw_params.bw) { |
857 | DP(NETIF_MSG_LINK, "bnx2x_ets_E3B0_config BW" | 857 | DP(NETIF_MSG_LINK, |
858 | "was set to 0\n"); | 858 | "bnx2x_ets_E3B0_config BW was set to 0\n"); |
859 | return -EINVAL; | 859 | return -EINVAL; |
860 | } | 860 | } |
861 | *total_bw += | 861 | *total_bw += |
@@ -866,12 +866,12 @@ static int bnx2x_ets_e3b0_get_total_bw( | |||
866 | /*Check taotl BW is valid */ | 866 | /*Check taotl BW is valid */ |
867 | if ((100 != *total_bw) || (0 == *total_bw)) { | 867 | if ((100 != *total_bw) || (0 == *total_bw)) { |
868 | if (0 == *total_bw) { | 868 | if (0 == *total_bw) { |
869 | DP(NETIF_MSG_LINK, "bnx2x_ets_E3B0_config toatl BW" | 869 | DP(NETIF_MSG_LINK, |
870 | "shouldn't be 0\n"); | 870 | "bnx2x_ets_E3B0_config toatl BW shouldn't be 0\n"); |
871 | return -EINVAL; | 871 | return -EINVAL; |
872 | } | 872 | } |
873 | DP(NETIF_MSG_LINK, "bnx2x_ets_E3B0_config toatl BW should be" | 873 | DP(NETIF_MSG_LINK, |
874 | "100\n"); | 874 | "bnx2x_ets_E3B0_config toatl BW should be 100\n"); |
875 | /** | 875 | /** |
876 | * We can handle a case whre the BW isn't 100 this can happen | 876 | * We can handle a case whre the BW isn't 100 this can happen |
877 | * if the TC are joined. | 877 | * if the TC are joined. |
@@ -908,13 +908,13 @@ static int bnx2x_ets_e3b0_sp_pri_to_cos_set(const struct link_params *params, | |||
908 | 908 | ||
909 | if (DCBX_INVALID_COS != sp_pri_to_cos[pri]) { | 909 | if (DCBX_INVALID_COS != sp_pri_to_cos[pri]) { |
910 | DP(NETIF_MSG_LINK, "bnx2x_ets_e3b0_sp_pri_to_cos_set invalid " | 910 | DP(NETIF_MSG_LINK, "bnx2x_ets_e3b0_sp_pri_to_cos_set invalid " |
911 | "parameter There can't be two COS's with" | 911 | "parameter There can't be two COS's with " |
912 | "the same strict pri\n"); | 912 | "the same strict pri\n"); |
913 | return -EINVAL; | 913 | return -EINVAL; |
914 | } | 914 | } |
915 | 915 | ||
916 | if (pri > max_num_of_cos) { | 916 | if (pri > max_num_of_cos) { |
917 | DP(NETIF_MSG_LINK, "bnx2x_ets_e3b0_sp_pri_to_cos_set invalid" | 917 | DP(NETIF_MSG_LINK, "bnx2x_ets_e3b0_sp_pri_to_cos_set invalid " |
918 | "parameter Illegal strict priority\n"); | 918 | "parameter Illegal strict priority\n"); |
919 | return -EINVAL; | 919 | return -EINVAL; |
920 | } | 920 | } |
@@ -1090,8 +1090,8 @@ int bnx2x_ets_e3b0_config(const struct link_params *params, | |||
1090 | u8 cos_entry = 0; | 1090 | u8 cos_entry = 0; |
1091 | 1091 | ||
1092 | if (!CHIP_IS_E3B0(bp)) { | 1092 | if (!CHIP_IS_E3B0(bp)) { |
1093 | DP(NETIF_MSG_LINK, "bnx2x_ets_e3b0_disabled the chip isn't E3B0" | 1093 | DP(NETIF_MSG_LINK, |
1094 | "\n"); | 1094 | "bnx2x_ets_e3b0_disabled the chip isn't E3B0\n"); |
1095 | return -EINVAL; | 1095 | return -EINVAL; |
1096 | } | 1096 | } |
1097 | 1097 | ||
@@ -1108,8 +1108,8 @@ int bnx2x_ets_e3b0_config(const struct link_params *params, | |||
1108 | bnx2x_status = bnx2x_ets_e3b0_get_total_bw(params, ets_params, | 1108 | bnx2x_status = bnx2x_ets_e3b0_get_total_bw(params, ets_params, |
1109 | &total_bw); | 1109 | &total_bw); |
1110 | if (0 != bnx2x_status) { | 1110 | if (0 != bnx2x_status) { |
1111 | DP(NETIF_MSG_LINK, "bnx2x_ets_E3B0_config get_total_bw failed " | 1111 | DP(NETIF_MSG_LINK, |
1112 | "\n"); | 1112 | "bnx2x_ets_E3B0_config get_total_bw failed\n"); |
1113 | return -EINVAL; | 1113 | return -EINVAL; |
1114 | } | 1114 | } |
1115 | 1115 | ||
@@ -1144,13 +1144,13 @@ int bnx2x_ets_e3b0_config(const struct link_params *params, | |||
1144 | cos_entry); | 1144 | cos_entry); |
1145 | 1145 | ||
1146 | } else { | 1146 | } else { |
1147 | DP(NETIF_MSG_LINK, "bnx2x_ets_e3b0_config cos state not" | 1147 | DP(NETIF_MSG_LINK, |
1148 | " valid\n"); | 1148 | "bnx2x_ets_e3b0_config cos state not valid\n"); |
1149 | return -EINVAL; | 1149 | return -EINVAL; |
1150 | } | 1150 | } |
1151 | if (0 != bnx2x_status) { | 1151 | if (0 != bnx2x_status) { |
1152 | DP(NETIF_MSG_LINK, "bnx2x_ets_e3b0_config set cos bw " | 1152 | DP(NETIF_MSG_LINK, |
1153 | "failed\n"); | 1153 | "bnx2x_ets_e3b0_config set cos bw failed\n"); |
1154 | return bnx2x_status; | 1154 | return bnx2x_status; |
1155 | } | 1155 | } |
1156 | } | 1156 | } |
@@ -1160,8 +1160,8 @@ int bnx2x_ets_e3b0_config(const struct link_params *params, | |||
1160 | sp_pri_to_cos); | 1160 | sp_pri_to_cos); |
1161 | 1161 | ||
1162 | if (0 != bnx2x_status) { | 1162 | if (0 != bnx2x_status) { |
1163 | DP(NETIF_MSG_LINK, "bnx2x_ets_E3B0_config set_pri_cli_reg " | 1163 | DP(NETIF_MSG_LINK, |
1164 | "failed\n"); | 1164 | "bnx2x_ets_E3B0_config set_pri_cli_reg failed\n"); |
1165 | return bnx2x_status; | 1165 | return bnx2x_status; |
1166 | } | 1166 | } |
1167 | 1167 | ||
@@ -1618,8 +1618,8 @@ static void bnx2x_xmac_init(struct bnx2x *bp, u32 max_speed) | |||
1618 | 1618 | ||
1619 | if (is_port4mode && (REG_RD(bp, MISC_REG_RESET_REG_2) & | 1619 | if (is_port4mode && (REG_RD(bp, MISC_REG_RESET_REG_2) & |
1620 | MISC_REGISTERS_RESET_REG_2_XMAC)) { | 1620 | MISC_REGISTERS_RESET_REG_2_XMAC)) { |
1621 | DP(NETIF_MSG_LINK, "XMAC already out of reset" | 1621 | DP(NETIF_MSG_LINK, |
1622 | " in 4-port mode\n"); | 1622 | "XMAC already out of reset in 4-port mode\n"); |
1623 | return; | 1623 | return; |
1624 | } | 1624 | } |
1625 | 1625 | ||
@@ -1642,13 +1642,13 @@ static void bnx2x_xmac_init(struct bnx2x *bp, u32 max_speed) | |||
1642 | /* Set the number of ports on the system side to 1 */ | 1642 | /* Set the number of ports on the system side to 1 */ |
1643 | REG_WR(bp, MISC_REG_XMAC_CORE_PORT_MODE, 0); | 1643 | REG_WR(bp, MISC_REG_XMAC_CORE_PORT_MODE, 0); |
1644 | if (max_speed == SPEED_10000) { | 1644 | if (max_speed == SPEED_10000) { |
1645 | DP(NETIF_MSG_LINK, "Init XMAC to 10G x 1" | 1645 | DP(NETIF_MSG_LINK, |
1646 | " port per path\n"); | 1646 | "Init XMAC to 10G x 1 port per path\n"); |
1647 | /* Set the number of ports on the Warp Core to 10G */ | 1647 | /* Set the number of ports on the Warp Core to 10G */ |
1648 | REG_WR(bp, MISC_REG_XMAC_PHY_PORT_MODE, 3); | 1648 | REG_WR(bp, MISC_REG_XMAC_PHY_PORT_MODE, 3); |
1649 | } else { | 1649 | } else { |
1650 | DP(NETIF_MSG_LINK, "Init XMAC to 20G x 2 ports" | 1650 | DP(NETIF_MSG_LINK, |
1651 | " per path\n"); | 1651 | "Init XMAC to 20G x 2 ports per path\n"); |
1652 | /* Set the number of ports on the Warp Core to 20G */ | 1652 | /* Set the number of ports on the Warp Core to 20G */ |
1653 | REG_WR(bp, MISC_REG_XMAC_PHY_PORT_MODE, 1); | 1653 | REG_WR(bp, MISC_REG_XMAC_PHY_PORT_MODE, 1); |
1654 | } | 1654 | } |
@@ -3959,8 +3959,8 @@ static void bnx2x_warpcore_set_sgmii_speed(struct bnx2x_phy *phy, | |||
3959 | val16 |= 0x0040; | 3959 | val16 |= 0x0040; |
3960 | break; | 3960 | break; |
3961 | default: | 3961 | default: |
3962 | DP(NETIF_MSG_LINK, "Speed not supported: 0x%x" | 3962 | DP(NETIF_MSG_LINK, |
3963 | "\n", phy->req_line_speed); | 3963 | "Speed not supported: 0x%x\n", phy->req_line_speed); |
3964 | return; | 3964 | return; |
3965 | } | 3965 | } |
3966 | 3966 | ||
@@ -4092,9 +4092,9 @@ static int bnx2x_get_mod_abs_int_cfg(struct bnx2x *bp, | |||
4092 | */ | 4092 | */ |
4093 | if ((cfg_pin < PIN_CFG_GPIO0_P0) || | 4093 | if ((cfg_pin < PIN_CFG_GPIO0_P0) || |
4094 | (cfg_pin > PIN_CFG_GPIO3_P1)) { | 4094 | (cfg_pin > PIN_CFG_GPIO3_P1)) { |
4095 | DP(NETIF_MSG_LINK, "ERROR: Invalid cfg pin %x for " | 4095 | DP(NETIF_MSG_LINK, |
4096 | "module detect indication\n", | 4096 | "ERROR: Invalid cfg pin %x for module detect indication\n", |
4097 | cfg_pin); | 4097 | cfg_pin); |
4098 | return -EINVAL; | 4098 | return -EINVAL; |
4099 | } | 4099 | } |
4100 | 4100 | ||
@@ -4222,8 +4222,9 @@ static void bnx2x_warpcore_config_init(struct bnx2x_phy *phy, | |||
4222 | break; | 4222 | break; |
4223 | 4223 | ||
4224 | default: | 4224 | default: |
4225 | DP(NETIF_MSG_LINK, "Unsupported Serdes Net Interface " | 4225 | DP(NETIF_MSG_LINK, |
4226 | "0x%x\n", serdes_net_if); | 4226 | "Unsupported Serdes Net Interface 0x%x\n", |
4227 | serdes_net_if); | ||
4227 | return; | 4228 | return; |
4228 | } | 4229 | } |
4229 | } | 4230 | } |
@@ -6127,8 +6128,8 @@ static int bnx2x_link_initialize(struct link_params *params, | |||
6127 | if (phy_index == EXT_PHY2 && | 6128 | if (phy_index == EXT_PHY2 && |
6128 | (bnx2x_phy_selection(params) == | 6129 | (bnx2x_phy_selection(params) == |
6129 | PORT_HW_CFG_PHY_SELECTION_FIRST_PHY)) { | 6130 | PORT_HW_CFG_PHY_SELECTION_FIRST_PHY)) { |
6130 | DP(NETIF_MSG_LINK, "Not initializing" | 6131 | DP(NETIF_MSG_LINK, |
6131 | " second phy\n"); | 6132 | "Not initializing second phy\n"); |
6132 | continue; | 6133 | continue; |
6133 | } | 6134 | } |
6134 | params->phy[phy_index].config_init( | 6135 | params->phy[phy_index].config_init( |
@@ -6447,8 +6448,8 @@ int bnx2x_link_update(struct link_params *params, struct link_vars *vars) | |||
6447 | */ | 6448 | */ |
6448 | if (active_external_phy == EXT_PHY1) { | 6449 | if (active_external_phy == EXT_PHY1) { |
6449 | if (params->phy[EXT_PHY2].phy_specific_func) { | 6450 | if (params->phy[EXT_PHY2].phy_specific_func) { |
6450 | DP(NETIF_MSG_LINK, "Disabling TX on" | 6451 | DP(NETIF_MSG_LINK, |
6451 | " EXT_PHY2\n"); | 6452 | "Disabling TX on EXT_PHY2\n"); |
6452 | params->phy[EXT_PHY2].phy_specific_func( | 6453 | params->phy[EXT_PHY2].phy_specific_func( |
6453 | ¶ms->phy[EXT_PHY2], | 6454 | ¶ms->phy[EXT_PHY2], |
6454 | params, DISABLE_TX); | 6455 | params, DISABLE_TX); |
@@ -7341,8 +7342,8 @@ static int bnx2x_8726_read_sfp_module_eeprom(struct bnx2x_phy *phy, | |||
7341 | u16 val = 0; | 7342 | u16 val = 0; |
7342 | u16 i; | 7343 | u16 i; |
7343 | if (byte_cnt > 16) { | 7344 | if (byte_cnt > 16) { |
7344 | DP(NETIF_MSG_LINK, "Reading from eeprom is" | 7345 | DP(NETIF_MSG_LINK, |
7345 | " is limited to 0xf\n"); | 7346 | "Reading from eeprom is limited to 0xf\n"); |
7346 | return -EINVAL; | 7347 | return -EINVAL; |
7347 | } | 7348 | } |
7348 | /* Set the read command byte count */ | 7349 | /* Set the read command byte count */ |
@@ -7413,8 +7414,8 @@ static int bnx2x_warpcore_read_sfp_module_eeprom(struct bnx2x_phy *phy, | |||
7413 | " addr %d, cnt %d\n", | 7414 | " addr %d, cnt %d\n", |
7414 | addr, byte_cnt);*/ | 7415 | addr, byte_cnt);*/ |
7415 | if (byte_cnt > 16) { | 7416 | if (byte_cnt > 16) { |
7416 | DP(NETIF_MSG_LINK, "Reading from eeprom is" | 7417 | DP(NETIF_MSG_LINK, |
7417 | " is limited to 16 bytes\n"); | 7418 | "Reading from eeprom is limited to 16 bytes\n"); |
7418 | return -EINVAL; | 7419 | return -EINVAL; |
7419 | } | 7420 | } |
7420 | 7421 | ||
@@ -7443,8 +7444,8 @@ static int bnx2x_8727_read_sfp_module_eeprom(struct bnx2x_phy *phy, | |||
7443 | u16 val, i; | 7444 | u16 val, i; |
7444 | 7445 | ||
7445 | if (byte_cnt > 16) { | 7446 | if (byte_cnt > 16) { |
7446 | DP(NETIF_MSG_LINK, "Reading from eeprom is" | 7447 | DP(NETIF_MSG_LINK, |
7447 | " is limited to 0xf\n"); | 7448 | "Reading from eeprom is limited to 0xf\n"); |
7448 | return -EINVAL; | 7449 | return -EINVAL; |
7449 | } | 7450 | } |
7450 | 7451 | ||
@@ -7591,13 +7592,14 @@ static int bnx2x_get_edc_mode(struct bnx2x_phy *phy, | |||
7591 | check_limiting_mode = 1; | 7592 | check_limiting_mode = 1; |
7592 | } else if (copper_module_type & | 7593 | } else if (copper_module_type & |
7593 | SFP_EEPROM_FC_TX_TECH_BITMASK_COPPER_PASSIVE) { | 7594 | SFP_EEPROM_FC_TX_TECH_BITMASK_COPPER_PASSIVE) { |
7594 | DP(NETIF_MSG_LINK, "Passive Copper" | 7595 | DP(NETIF_MSG_LINK, |
7595 | " cable detected\n"); | 7596 | "Passive Copper cable detected\n"); |
7596 | *edc_mode = | 7597 | *edc_mode = |
7597 | EDC_MODE_PASSIVE_DAC; | 7598 | EDC_MODE_PASSIVE_DAC; |
7598 | } else { | 7599 | } else { |
7599 | DP(NETIF_MSG_LINK, "Unknown copper-cable-" | 7600 | DP(NETIF_MSG_LINK, |
7600 | "type 0x%x !!!\n", copper_module_type); | 7601 | "Unknown copper-cable-type 0x%x !!!\n", |
7602 | copper_module_type); | ||
7601 | return -EINVAL; | 7603 | return -EINVAL; |
7602 | } | 7604 | } |
7603 | break; | 7605 | break; |
@@ -7635,8 +7637,8 @@ static int bnx2x_get_edc_mode(struct bnx2x_phy *phy, | |||
7635 | SFP_EEPROM_OPTIONS_ADDR, | 7637 | SFP_EEPROM_OPTIONS_ADDR, |
7636 | SFP_EEPROM_OPTIONS_SIZE, | 7638 | SFP_EEPROM_OPTIONS_SIZE, |
7637 | options) != 0) { | 7639 | options) != 0) { |
7638 | DP(NETIF_MSG_LINK, "Failed to read Option" | 7640 | DP(NETIF_MSG_LINK, |
7639 | " field from module EEPROM\n"); | 7641 | "Failed to read Option field from module EEPROM\n"); |
7640 | return -EINVAL; | 7642 | return -EINVAL; |
7641 | } | 7643 | } |
7642 | if ((options[0] & SFP_EEPROM_OPTIONS_LINEAR_RX_OUT_MASK)) | 7644 | if ((options[0] & SFP_EEPROM_OPTIONS_LINEAR_RX_OUT_MASK)) |
@@ -7677,15 +7679,15 @@ static int bnx2x_verify_sfp_module(struct bnx2x_phy *phy, | |||
7677 | FEATURE_CONFIG_BC_SUPPORTS_OPT_MDL_VRFY) { | 7679 | FEATURE_CONFIG_BC_SUPPORTS_OPT_MDL_VRFY) { |
7678 | /* Use first phy request only in case of non-dual media*/ | 7680 | /* Use first phy request only in case of non-dual media*/ |
7679 | if (DUAL_MEDIA(params)) { | 7681 | if (DUAL_MEDIA(params)) { |
7680 | DP(NETIF_MSG_LINK, "FW does not support OPT MDL " | 7682 | DP(NETIF_MSG_LINK, |
7681 | "verification\n"); | 7683 | "FW does not support OPT MDL verification\n"); |
7682 | return -EINVAL; | 7684 | return -EINVAL; |
7683 | } | 7685 | } |
7684 | cmd = DRV_MSG_CODE_VRFY_FIRST_PHY_OPT_MDL; | 7686 | cmd = DRV_MSG_CODE_VRFY_FIRST_PHY_OPT_MDL; |
7685 | } else { | 7687 | } else { |
7686 | /* No support in OPT MDL detection */ | 7688 | /* No support in OPT MDL detection */ |
7687 | DP(NETIF_MSG_LINK, "FW does not support OPT MDL " | 7689 | DP(NETIF_MSG_LINK, |
7688 | "verification\n"); | 7690 | "FW does not support OPT MDL verification\n"); |
7689 | return -EINVAL; | 7691 | return -EINVAL; |
7690 | } | 7692 | } |
7691 | 7693 | ||
@@ -7736,8 +7738,9 @@ static int bnx2x_wait_for_sfp_module_initialized(struct bnx2x_phy *phy, | |||
7736 | for (timeout = 0; timeout < 60; timeout++) { | 7738 | for (timeout = 0; timeout < 60; timeout++) { |
7737 | if (bnx2x_read_sfp_module_eeprom(phy, params, 1, 1, &val) | 7739 | if (bnx2x_read_sfp_module_eeprom(phy, params, 1, 1, &val) |
7738 | == 0) { | 7740 | == 0) { |
7739 | DP(NETIF_MSG_LINK, "SFP+ module initialization " | 7741 | DP(NETIF_MSG_LINK, |
7740 | "took %d ms\n", timeout * 5); | 7742 | "SFP+ module initialization took %d ms\n", |
7743 | timeout * 5); | ||
7741 | return 0; | 7744 | return 0; |
7742 | } | 7745 | } |
7743 | msleep(5); | 7746 | msleep(5); |
@@ -8506,8 +8509,8 @@ static int bnx2x_8726_config_init(struct bnx2x_phy *phy, | |||
8506 | /* Set TX PreEmphasis if needed */ | 8509 | /* Set TX PreEmphasis if needed */ |
8507 | if ((params->feature_config_flags & | 8510 | if ((params->feature_config_flags & |
8508 | FEATURE_CONFIG_OVERRIDE_PREEMPHASIS_ENABLED)) { | 8511 | FEATURE_CONFIG_OVERRIDE_PREEMPHASIS_ENABLED)) { |
8509 | DP(NETIF_MSG_LINK, "Setting TX_CTRL1 0x%x," | 8512 | DP(NETIF_MSG_LINK, |
8510 | "TX_CTRL2 0x%x\n", | 8513 | "Setting TX_CTRL1 0x%x, TX_CTRL2 0x%x\n", |
8511 | phy->tx_preemphasis[0], | 8514 | phy->tx_preemphasis[0], |
8512 | phy->tx_preemphasis[1]); | 8515 | phy->tx_preemphasis[1]); |
8513 | bnx2x_cl45_write(bp, phy, | 8516 | bnx2x_cl45_write(bp, phy, |
@@ -8788,8 +8791,8 @@ static void bnx2x_8727_handle_mod_abs(struct bnx2x_phy *phy, | |||
8788 | if (mod_abs & (1<<8)) { | 8791 | if (mod_abs & (1<<8)) { |
8789 | 8792 | ||
8790 | /* Module is absent */ | 8793 | /* Module is absent */ |
8791 | DP(NETIF_MSG_LINK, "MOD_ABS indication " | 8794 | DP(NETIF_MSG_LINK, |
8792 | "show module is absent\n"); | 8795 | "MOD_ABS indication show module is absent\n"); |
8793 | phy->media_type = ETH_PHY_NOT_PRESENT; | 8796 | phy->media_type = ETH_PHY_NOT_PRESENT; |
8794 | /* | 8797 | /* |
8795 | * 1. Set mod_abs to detect next module | 8798 | * 1. Set mod_abs to detect next module |
@@ -8816,8 +8819,8 @@ static void bnx2x_8727_handle_mod_abs(struct bnx2x_phy *phy, | |||
8816 | 8819 | ||
8817 | } else { | 8820 | } else { |
8818 | /* Module is present */ | 8821 | /* Module is present */ |
8819 | DP(NETIF_MSG_LINK, "MOD_ABS indication " | 8822 | DP(NETIF_MSG_LINK, |
8820 | "show module is present\n"); | 8823 | "MOD_ABS indication show module is present\n"); |
8821 | /* | 8824 | /* |
8822 | * First disable transmitter, and if the module is ok, the | 8825 | * First disable transmitter, and if the module is ok, the |
8823 | * module_detection will enable it | 8826 | * module_detection will enable it |
@@ -8908,8 +8911,9 @@ static u8 bnx2x_8727_read_status(struct bnx2x_phy *phy, | |||
8908 | if ((val1 & (1<<8)) == 0) { | 8911 | if ((val1 & (1<<8)) == 0) { |
8909 | if (!CHIP_IS_E1x(bp)) | 8912 | if (!CHIP_IS_E1x(bp)) |
8910 | oc_port = BP_PATH(bp) + (params->port << 1); | 8913 | oc_port = BP_PATH(bp) + (params->port << 1); |
8911 | DP(NETIF_MSG_LINK, "8727 Power fault has been detected" | 8914 | DP(NETIF_MSG_LINK, |
8912 | " on port %d\n", oc_port); | 8915 | "8727 Power fault has been detected on port %d\n", |
8916 | oc_port); | ||
8913 | netdev_err(bp->dev, "Error: Power fault on Port %d has" | 8917 | netdev_err(bp->dev, "Error: Power fault on Port %d has" |
8914 | " been detected and the power to " | 8918 | " been detected and the power to " |
8915 | "that SFP+ module has been removed" | 8919 | "that SFP+ module has been removed" |
@@ -9690,8 +9694,8 @@ static u8 bnx2x_848xx_read_status(struct bnx2x_phy *phy, | |||
9690 | MDIO_AN_REG_8481_EXPANSION_REG_RD_RW, | 9694 | MDIO_AN_REG_8481_EXPANSION_REG_RD_RW, |
9691 | &legacy_status); | 9695 | &legacy_status); |
9692 | 9696 | ||
9693 | DP(NETIF_MSG_LINK, "Legacy speed status" | 9697 | DP(NETIF_MSG_LINK, "Legacy speed status = 0x%x\n", |
9694 | " = 0x%x\n", legacy_status); | 9698 | legacy_status); |
9695 | link_up = ((legacy_status & (1<<11)) == (1<<11)); | 9699 | link_up = ((legacy_status & (1<<11)) == (1<<11)); |
9696 | if (link_up) { | 9700 | if (link_up) { |
9697 | legacy_speed = (legacy_status & (3<<9)); | 9701 | legacy_speed = (legacy_status & (3<<9)); |
@@ -9709,9 +9713,10 @@ static u8 bnx2x_848xx_read_status(struct bnx2x_phy *phy, | |||
9709 | else | 9713 | else |
9710 | vars->duplex = DUPLEX_HALF; | 9714 | vars->duplex = DUPLEX_HALF; |
9711 | 9715 | ||
9712 | DP(NETIF_MSG_LINK, "Link is up in %dMbps," | 9716 | DP(NETIF_MSG_LINK, |
9713 | " is_duplex_full= %d\n", vars->line_speed, | 9717 | "Link is up in %dMbps, is_duplex_full= %d\n", |
9714 | (vars->duplex == DUPLEX_FULL)); | 9718 | vars->line_speed, |
9719 | (vars->duplex == DUPLEX_FULL)); | ||
9715 | /* Check legacy speed AN resolution */ | 9720 | /* Check legacy speed AN resolution */ |
9716 | bnx2x_cl45_read(bp, phy, | 9721 | bnx2x_cl45_read(bp, phy, |
9717 | MDIO_AN_DEVAD, | 9722 | MDIO_AN_DEVAD, |
@@ -10286,9 +10291,10 @@ static u8 bnx2x_54618se_read_status(struct bnx2x_phy *phy, | |||
10286 | } else /* Should not happen */ | 10291 | } else /* Should not happen */ |
10287 | vars->line_speed = 0; | 10292 | vars->line_speed = 0; |
10288 | 10293 | ||
10289 | DP(NETIF_MSG_LINK, "Link is up in %dMbps," | 10294 | DP(NETIF_MSG_LINK, |
10290 | " is_duplex_full= %d\n", vars->line_speed, | 10295 | "Link is up in %dMbps, is_duplex_full= %d\n", |
10291 | (vars->duplex == DUPLEX_FULL)); | 10296 | vars->line_speed, |
10297 | (vars->duplex == DUPLEX_FULL)); | ||
10292 | 10298 | ||
10293 | /* Check legacy speed AN resolution */ | 10299 | /* Check legacy speed AN resolution */ |
10294 | bnx2x_cl22_read(bp, phy, | 10300 | bnx2x_cl22_read(bp, phy, |
@@ -11336,8 +11342,9 @@ static void bnx2x_phy_def_cfg(struct link_params *params, | |||
11336 | dev_info. | 11342 | dev_info. |
11337 | port_hw_config[params->port].speed_capability_mask)); | 11343 | port_hw_config[params->port].speed_capability_mask)); |
11338 | } | 11344 | } |
11339 | DP(NETIF_MSG_LINK, "Default config phy idx %x cfg 0x%x speed_cap_mask" | 11345 | DP(NETIF_MSG_LINK, |
11340 | " 0x%x\n", phy_index, link_config, phy->speed_cap_mask); | 11346 | "Default config phy idx %x cfg 0x%x speed_cap_mask 0x%x\n", |
11347 | phy_index, link_config, phy->speed_cap_mask); | ||
11341 | 11348 | ||
11342 | phy->req_duplex = DUPLEX_FULL; | 11349 | phy->req_duplex = DUPLEX_FULL; |
11343 | switch (link_config & PORT_FEATURE_LINK_SPEED_MASK) { | 11350 | switch (link_config & PORT_FEATURE_LINK_SPEED_MASK) { |
diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c index 173b258b855..e899e87f722 100644 --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c | |||
@@ -4388,7 +4388,7 @@ static inline void bnx2x_handle_rx_mode_eqe(struct bnx2x *bp) | |||
4388 | static inline struct bnx2x_queue_sp_obj *bnx2x_cid_to_q_obj( | 4388 | static inline struct bnx2x_queue_sp_obj *bnx2x_cid_to_q_obj( |
4389 | struct bnx2x *bp, u32 cid) | 4389 | struct bnx2x *bp, u32 cid) |
4390 | { | 4390 | { |
4391 | DP(BNX2X_MSG_SP, "retrieving fp from cid %d", cid); | 4391 | DP(BNX2X_MSG_SP, "retrieving fp from cid %d\n", cid); |
4392 | #ifdef BCM_CNIC | 4392 | #ifdef BCM_CNIC |
4393 | if (cid == BNX2X_FCOE_ETH_CID) | 4393 | if (cid == BNX2X_FCOE_ETH_CID) |
4394 | return &bnx2x_fcoe(bp, q_obj); | 4394 | return &bnx2x_fcoe(bp, q_obj); |
@@ -7176,7 +7176,7 @@ static inline void bnx2x_pf_q_prep_init(struct bnx2x *bp, | |||
7176 | /* set maximum number of COSs supported by this queue */ | 7176 | /* set maximum number of COSs supported by this queue */ |
7177 | init_params->max_cos = fp->max_cos; | 7177 | init_params->max_cos = fp->max_cos; |
7178 | 7178 | ||
7179 | DP(BNX2X_MSG_SP, "fp: %d setting queue params max cos to: %d", | 7179 | DP(BNX2X_MSG_SP, "fp: %d setting queue params max cos to: %d\n", |
7180 | fp->index, init_params->max_cos); | 7180 | fp->index, init_params->max_cos); |
7181 | 7181 | ||
7182 | /* set the context pointers queue object */ | 7182 | /* set the context pointers queue object */ |
@@ -7209,7 +7209,7 @@ int bnx2x_setup_tx_only(struct bnx2x *bp, struct bnx2x_fastpath *fp, | |||
7209 | 7209 | ||
7210 | DP(BNX2X_MSG_SP, "preparing to send tx-only ramrod for connection:" | 7210 | DP(BNX2X_MSG_SP, "preparing to send tx-only ramrod for connection:" |
7211 | "cos %d, primary cid %d, cid %d, " | 7211 | "cos %d, primary cid %d, cid %d, " |
7212 | "client id %d, sp-client id %d, flags %lx", | 7212 | "client id %d, sp-client id %d, flags %lx\n", |
7213 | tx_index, q_params->q_obj->cids[FIRST_TX_COS_INDEX], | 7213 | tx_index, q_params->q_obj->cids[FIRST_TX_COS_INDEX], |
7214 | q_params->q_obj->cids[tx_index], q_params->q_obj->cl_id, | 7214 | q_params->q_obj->cids[tx_index], q_params->q_obj->cl_id, |
7215 | tx_only_params->gen_params.spcl_id, tx_only_params->flags); | 7215 | tx_only_params->gen_params.spcl_id, tx_only_params->flags); |
@@ -7241,7 +7241,7 @@ int bnx2x_setup_queue(struct bnx2x *bp, struct bnx2x_fastpath *fp, | |||
7241 | int rc; | 7241 | int rc; |
7242 | u8 tx_index; | 7242 | u8 tx_index; |
7243 | 7243 | ||
7244 | DP(BNX2X_MSG_SP, "setting up queue %d", fp->index); | 7244 | DP(BNX2X_MSG_SP, "setting up queue %d\n", fp->index); |
7245 | 7245 | ||
7246 | /* reset IGU state skip FCoE L2 queue */ | 7246 | /* reset IGU state skip FCoE L2 queue */ |
7247 | if (!IS_FCOE_FP(fp)) | 7247 | if (!IS_FCOE_FP(fp)) |
@@ -7265,7 +7265,7 @@ int bnx2x_setup_queue(struct bnx2x *bp, struct bnx2x_fastpath *fp, | |||
7265 | return rc; | 7265 | return rc; |
7266 | } | 7266 | } |
7267 | 7267 | ||
7268 | DP(BNX2X_MSG_SP, "init complete"); | 7268 | DP(BNX2X_MSG_SP, "init complete\n"); |
7269 | 7269 | ||
7270 | 7270 | ||
7271 | /* Now move the Queue to the SETUP state... */ | 7271 | /* Now move the Queue to the SETUP state... */ |
@@ -7319,7 +7319,7 @@ static int bnx2x_stop_queue(struct bnx2x *bp, int index) | |||
7319 | struct bnx2x_queue_state_params q_params = {0}; | 7319 | struct bnx2x_queue_state_params q_params = {0}; |
7320 | int rc, tx_index; | 7320 | int rc, tx_index; |
7321 | 7321 | ||
7322 | DP(BNX2X_MSG_SP, "stopping queue %d cid %d", index, fp->cid); | 7322 | DP(BNX2X_MSG_SP, "stopping queue %d cid %d\n", index, fp->cid); |
7323 | 7323 | ||
7324 | q_params.q_obj = &fp->q_obj; | 7324 | q_params.q_obj = &fp->q_obj; |
7325 | /* We want to wait for completion in this context */ | 7325 | /* We want to wait for completion in this context */ |
@@ -7334,7 +7334,7 @@ static int bnx2x_stop_queue(struct bnx2x *bp, int index) | |||
7334 | /* ascertain this is a normal queue*/ | 7334 | /* ascertain this is a normal queue*/ |
7335 | txdata = &fp->txdata[tx_index]; | 7335 | txdata = &fp->txdata[tx_index]; |
7336 | 7336 | ||
7337 | DP(BNX2X_MSG_SP, "stopping tx-only queue %d", | 7337 | DP(BNX2X_MSG_SP, "stopping tx-only queue %d\n", |
7338 | txdata->txq_index); | 7338 | txdata->txq_index); |
7339 | 7339 | ||
7340 | /* send halt terminate on tx-only connection */ | 7340 | /* send halt terminate on tx-only connection */ |
@@ -10704,7 +10704,7 @@ static int __devinit bnx2x_init_one(struct pci_dev *pdev, | |||
10704 | return rc; | 10704 | return rc; |
10705 | } | 10705 | } |
10706 | 10706 | ||
10707 | DP(NETIF_MSG_DRV, "max_non_def_sbs %d", max_non_def_sbs); | 10707 | DP(NETIF_MSG_DRV, "max_non_def_sbs %d\n", max_non_def_sbs); |
10708 | 10708 | ||
10709 | rc = bnx2x_init_bp(bp); | 10709 | rc = bnx2x_init_bp(bp); |
10710 | if (rc) | 10710 | if (rc) |
@@ -10759,15 +10759,14 @@ static int __devinit bnx2x_init_one(struct pci_dev *pdev, | |||
10759 | 10759 | ||
10760 | bnx2x_get_pcie_width_speed(bp, &pcie_width, &pcie_speed); | 10760 | bnx2x_get_pcie_width_speed(bp, &pcie_width, &pcie_speed); |
10761 | 10761 | ||
10762 | netdev_info(dev, "%s (%c%d) PCI-E x%d %s found at mem %lx," | 10762 | netdev_info(dev, "%s (%c%d) PCI-E x%d %s found at mem %lx, IRQ %d, node addr %pM\n", |
10763 | " IRQ %d, ", board_info[ent->driver_data].name, | 10763 | board_info[ent->driver_data].name, |
10764 | (CHIP_REV(bp) >> 12) + 'A', (CHIP_METAL(bp) >> 4), | 10764 | (CHIP_REV(bp) >> 12) + 'A', (CHIP_METAL(bp) >> 4), |
10765 | pcie_width, | 10765 | pcie_width, |
10766 | ((!CHIP_IS_E2(bp) && pcie_speed == 2) || | 10766 | ((!CHIP_IS_E2(bp) && pcie_speed == 2) || |
10767 | (CHIP_IS_E2(bp) && pcie_speed == 1)) ? | 10767 | (CHIP_IS_E2(bp) && pcie_speed == 1)) ? |
10768 | "5GHz (Gen2)" : "2.5GHz", | 10768 | "5GHz (Gen2)" : "2.5GHz", |
10769 | dev->base_addr, bp->pdev->irq); | 10769 | dev->base_addr, bp->pdev->irq, dev->dev_addr); |
10770 | pr_cont("node addr %pM\n", dev->dev_addr); | ||
10771 | 10770 | ||
10772 | return 0; | 10771 | return 0; |
10773 | 10772 | ||
diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c index b4d9c16ff15..1f88c1913bb 100644 --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c | |||
@@ -3045,8 +3045,8 @@ static int bnx2x_mcast_setup_e1h(struct bnx2x *bp, | |||
3045 | break; | 3045 | break; |
3046 | 3046 | ||
3047 | case BNX2X_MCAST_CMD_DEL: | 3047 | case BNX2X_MCAST_CMD_DEL: |
3048 | DP(BNX2X_MSG_SP, "Invalidating multicast " | 3048 | DP(BNX2X_MSG_SP, |
3049 | "MACs configuration\n"); | 3049 | "Invalidating multicast MACs configuration\n"); |
3050 | 3050 | ||
3051 | /* clear the registry */ | 3051 | /* clear the registry */ |
3052 | memset(o->registry.aprox_match.vec, 0, | 3052 | memset(o->registry.aprox_match.vec, 0, |
@@ -4239,7 +4239,7 @@ static int bnx2x_queue_comp_cmd(struct bnx2x *bp, | |||
4239 | o->cids[BNX2X_PRIMARY_CID_INDEX], o->next_state); | 4239 | o->cids[BNX2X_PRIMARY_CID_INDEX], o->next_state); |
4240 | 4240 | ||
4241 | if (o->next_tx_only) /* print num tx-only if any exist */ | 4241 | if (o->next_tx_only) /* print num tx-only if any exist */ |
4242 | DP(BNX2X_MSG_SP, "primary cid %d: num tx-only cons %d", | 4242 | DP(BNX2X_MSG_SP, "primary cid %d: num tx-only cons %d\n", |
4243 | o->cids[BNX2X_PRIMARY_CID_INDEX], o->next_tx_only); | 4243 | o->cids[BNX2X_PRIMARY_CID_INDEX], o->next_tx_only); |
4244 | 4244 | ||
4245 | o->state = o->next_state; | 4245 | o->state = o->next_state; |
@@ -4301,7 +4301,7 @@ static void bnx2x_q_fill_init_general_data(struct bnx2x *bp, | |||
4301 | test_bit(BNX2X_Q_FLG_FCOE, flags) ? | 4301 | test_bit(BNX2X_Q_FLG_FCOE, flags) ? |
4302 | LLFC_TRAFFIC_TYPE_FCOE : LLFC_TRAFFIC_TYPE_NW; | 4302 | LLFC_TRAFFIC_TYPE_FCOE : LLFC_TRAFFIC_TYPE_NW; |
4303 | 4303 | ||
4304 | DP(BNX2X_MSG_SP, "flags: active %d, cos %d, stats en %d", | 4304 | DP(BNX2X_MSG_SP, "flags: active %d, cos %d, stats en %d\n", |
4305 | gen_data->activate_flg, gen_data->cos, gen_data->statistics_en_flg); | 4305 | gen_data->activate_flg, gen_data->cos, gen_data->statistics_en_flg); |
4306 | } | 4306 | } |
4307 | 4307 | ||
@@ -4454,7 +4454,7 @@ static void bnx2x_q_fill_setup_tx_only(struct bnx2x *bp, | |||
4454 | &data->tx, | 4454 | &data->tx, |
4455 | &cmd_params->params.tx_only.flags); | 4455 | &cmd_params->params.tx_only.flags); |
4456 | 4456 | ||
4457 | DP(BNX2X_MSG_SP, "cid %d, tx bd page lo %x hi %x",cmd_params->q_obj->cids[0], | 4457 | DP(BNX2X_MSG_SP, "cid %d, tx bd page lo %x hi %x\n",cmd_params->q_obj->cids[0], |
4458 | data->tx.tx_bd_page_base.lo, data->tx.tx_bd_page_base.hi); | 4458 | data->tx.tx_bd_page_base.lo, data->tx.tx_bd_page_base.hi); |
4459 | } | 4459 | } |
4460 | 4460 | ||
@@ -4501,9 +4501,9 @@ static inline int bnx2x_q_init(struct bnx2x *bp, | |||
4501 | 4501 | ||
4502 | /* Set CDU context validation values */ | 4502 | /* Set CDU context validation values */ |
4503 | for (cos = 0; cos < o->max_cos; cos++) { | 4503 | for (cos = 0; cos < o->max_cos; cos++) { |
4504 | DP(BNX2X_MSG_SP, "setting context validation. cid %d, cos %d", | 4504 | DP(BNX2X_MSG_SP, "setting context validation. cid %d, cos %d\n", |
4505 | o->cids[cos], cos); | 4505 | o->cids[cos], cos); |
4506 | DP(BNX2X_MSG_SP, "context pointer %p", init->cxts[cos]); | 4506 | DP(BNX2X_MSG_SP, "context pointer %p\n", init->cxts[cos]); |
4507 | bnx2x_set_ctx_validation(bp, init->cxts[cos], o->cids[cos]); | 4507 | bnx2x_set_ctx_validation(bp, init->cxts[cos], o->cids[cos]); |
4508 | } | 4508 | } |
4509 | 4509 | ||
@@ -4592,7 +4592,7 @@ static inline int bnx2x_q_send_setup_tx_only(struct bnx2x *bp, | |||
4592 | return -EINVAL; | 4592 | return -EINVAL; |
4593 | } | 4593 | } |
4594 | 4594 | ||
4595 | DP(BNX2X_MSG_SP, "parameters received: cos: %d sp-id: %d", | 4595 | DP(BNX2X_MSG_SP, "parameters received: cos: %d sp-id: %d\n", |
4596 | tx_only_params->gen_params.cos, | 4596 | tx_only_params->gen_params.cos, |
4597 | tx_only_params->gen_params.spcl_id); | 4597 | tx_only_params->gen_params.spcl_id); |
4598 | 4598 | ||
@@ -4603,7 +4603,7 @@ static inline int bnx2x_q_send_setup_tx_only(struct bnx2x *bp, | |||
4603 | bnx2x_q_fill_setup_tx_only(bp, params, rdata); | 4603 | bnx2x_q_fill_setup_tx_only(bp, params, rdata); |
4604 | 4604 | ||
4605 | DP(BNX2X_MSG_SP, "sending tx-only ramrod: cid %d, client-id %d," | 4605 | DP(BNX2X_MSG_SP, "sending tx-only ramrod: cid %d, client-id %d," |
4606 | "sp-client id %d, cos %d", | 4606 | "sp-client id %d, cos %d\n", |
4607 | o->cids[cid_index], | 4607 | o->cids[cid_index], |
4608 | rdata->general.client_id, | 4608 | rdata->general.client_id, |
4609 | rdata->general.sp_client_id, rdata->general.cos); | 4609 | rdata->general.sp_client_id, rdata->general.cos); |
@@ -5160,8 +5160,9 @@ static inline int bnx2x_func_state_change_comp(struct bnx2x *bp, | |||
5160 | return -EINVAL; | 5160 | return -EINVAL; |
5161 | } | 5161 | } |
5162 | 5162 | ||
5163 | DP(BNX2X_MSG_SP, "Completing command %d for func %d, setting state to " | 5163 | DP(BNX2X_MSG_SP, |
5164 | "%d\n", cmd, BP_FUNC(bp), o->next_state); | 5164 | "Completing command %d for func %d, setting state to %d\n", |
5165 | cmd, BP_FUNC(bp), o->next_state); | ||
5165 | 5166 | ||
5166 | o->state = o->next_state; | 5167 | o->state = o->next_state; |
5167 | o->next_state = BNX2X_F_STATE_MAX; | 5168 | o->next_state = BNX2X_F_STATE_MAX; |