From: Joe Perches <joe@perches.com>
To: "Luis R. Rodriguez" <rodrigue@qca.qualcomm.com>
Cc: davem@davemloft.net, netdev@vger.kernel.org,
linux-kernel@vger.kernel.org, mcgrof@frijolero.org,
qca-linux-team@qualcomm.com, nic-devel@qualcomm.com,
kgiori@qca.qualcomm.com, chris.snook@gmail.com,
mathieu@qca.qualcomm.com, bryanh@quicinc.com
Subject: Re: [PATCH] net: add QCA alx Ethernet driver
Date: Tue, 28 Feb 2012 19:52:26 -0800 [thread overview]
Message-ID: <1330487546.3742.1.camel@joe2Laptop> (raw)
In-Reply-To: <1330480210-30470-1-git-send-email-rodrigue@qca.qualcomm.com>
Even though this whole thing has been rejected,
here are some possible trivial logging corrections.
---
drivers/net/ethernet/atheros/alx/alf_cb.c | 2 +-
drivers/net/ethernet/atheros/alx/alx_main.c | 63 ++++++++++++---------------
drivers/net/ethernet/atheros/alx/alx_sw.h | 4 +-
3 files changed, 31 insertions(+), 38 deletions(-)
diff --git a/drivers/net/ethernet/atheros/alx/alf_cb.c b/drivers/net/ethernet/atheros/alx/alf_cb.c
index d267760..9e823f6 100644
--- a/drivers/net/ethernet/atheros/alx/alf_cb.c
+++ b/drivers/net/ethernet/atheros/alx/alf_cb.c
@@ -274,7 +274,7 @@ static int alf_check_phy_link(struct alx_hw *hw, u32 *speed, bool *link_up)
if (giga & L1F_GIGA_PSSR_DPLX)
*speed = ALX_LINK_SPEED_1GB_FULL;
else
- alx_hw_err(hw, "1000M half is invalid");
+ alx_hw_err(hw, "1000M half is invalid\n");
break;
case L1F_GIGA_PSSR_100MBS:
if (giga & L1F_GIGA_PSSR_DPLX)
diff --git a/drivers/net/ethernet/atheros/alx/alx_main.c b/drivers/net/ethernet/atheros/alx/alx_main.c
index a51c608..a599269 100644
--- a/drivers/net/ethernet/atheros/alx/alx_main.c
+++ b/drivers/net/ethernet/atheros/alx/alx_main.c
@@ -19,8 +19,7 @@
char alx_drv_name[] = "alx";
static const char alx_drv_description[] =
- "Qualcomm Atheros(R) "
- "AR813x/AR815x/AR816x PCI-E Ethernet Network Driver";
+ "Qualcomm Atheros(R) AR813x/AR815x/AR816x PCI-E Ethernet Network Driver";
/* alx_pci_tbl - PCI Device ID Table
*
@@ -142,22 +141,19 @@ static int alx_validate_mac_addr(u8 *mac_addr)
{
int retval = 0;
- if (mac_addr[0] & 0x01) {
- printk(KERN_DEBUG "MAC address is multicast\n");
+ if (is_broadcast_ether_addr(mac_addr)) {
+ pr_debug("MAC address is broadcast\n");
retval = -EADDRNOTAVAIL;
- } else if (mac_addr[0] == 0xff && mac_addr[1] == 0xff) {
- printk(KERN_DEBUG "MAC address is broadcast\n");
+ } else if (is_multicast_ether_addr(mac_addr)) {
+ pr_debug("MAC address is multicast\n");
retval = -EADDRNOTAVAIL;
- } else if (mac_addr[0] == 0 && mac_addr[1] == 0 &&
- mac_addr[2] == 0 && mac_addr[3] == 0 &&
- mac_addr[4] == 0 && mac_addr[5] == 0) {
- printk(KERN_DEBUG "MAC address is all zeros\n");
+ } else if (is_zero_ether_addr(mac_addr)) {
+ pr_debug("MAC address is all zeros\n");
retval = -EADDRNOTAVAIL;
}
return retval;
}
-
/*
* alx_set_mac_type - Sets MAC type
*/
@@ -459,7 +455,7 @@ static bool alx_get_rrdesc(struct alx_rx_queue *rxque,
if (likely(srrd->genr.nor != 1)) {
/* TODO support mul rfd*/
- printk(KERN_EMERG "Multi rfd not support yet!\n");
+ pr_emerg("Multi rfd not support yet!\n");
}
srrd->genr.update = 0;
@@ -1068,7 +1064,7 @@ static void alx_set_msix_flags(struct alx_msix_param *msix,
SET_MSIX_FLAG(RX7);
break;
default:
- printk(KERN_ERR "alx_set_msix_flags: rx error.");
+ pr_err("%s: rx error\n", __func__);
break;
}
} else if (type == alx_msix_type_tx) {
@@ -1086,7 +1082,7 @@ static void alx_set_msix_flags(struct alx_msix_param *msix,
SET_MSIX_FLAG(TX3);
break;
default:
- printk(KERN_ERR "alx_set_msix_flags: tx error.");
+ pr_err("%s: tx error\n", __func__);
break;
}
} else if (type == alx_msix_type_other) {
@@ -1104,7 +1100,7 @@ static void alx_set_msix_flags(struct alx_msix_param *msix,
SET_MSIX_FLAG(PHY);
break;
default:
- printk(KERN_ERR "alx_set_msix_flags: other error.");
+ pr_err("%s: other error\n", __func__);
break;
}
}
@@ -1742,11 +1738,11 @@ static int alx_alloc_all_rtx_queue(struct alx_adapter *adpt)
return 0;
err_alloc_rx_queue:
- alx_err(adpt, "goto err_alloc_rx_queue");
+ alx_err(adpt, "goto err_alloc_rx_queue\n");
for (que_idx = 0; que_idx < adpt->num_rxques; que_idx++)
kfree(adpt->rx_queue[que_idx]);
err_alloc_tx_queue:
- alx_err(adpt, "goto err_alloc_tx_queue");
+ alx_err(adpt, "goto err_alloc_tx_queue\n");
for (que_idx = 0; que_idx < adpt->num_txques; que_idx++)
kfree(adpt->tx_queue[que_idx]);
return -ENOMEM;
@@ -2976,17 +2972,17 @@ static void alx_link_task_routine(struct alx_adapter *adpt)
if (netif_carrier_ok(netdev))
return;
- link_desc = (hw->link_speed == ALX_LINK_SPEED_1GB_FULL) ?
+ link_desc = hw->link_speed == ALX_LINK_SPEED_1GB_FULL) ?
"1 Gbps Duplex Full" :
- (hw->link_speed == ALX_LINK_SPEED_100_FULL ?
- "100 Mbps Duplex Full" :
- (hw->link_speed == ALX_LINK_SPEED_100_HALF ?
- "100 Mbps Duplex Half" :
- (hw->link_speed == ALX_LINK_SPEED_10_FULL ?
- "10 Mbps Duplex Full" :
- (hw->link_speed == ALX_LINK_SPEED_10_HALF ?
- "10 Mbps Duplex HALF" :
- "unknown speed"))));
+ hw->link_speed == ALX_LINK_SPEED_100_FULL ?
+ "100 Mbps Duplex Full" :
+ hw->link_speed == ALX_LINK_SPEED_100_HALF ?
+ "100 Mbps Duplex Half" :
+ hw->link_speed == ALX_LINK_SPEED_10_FULL ?
+ "10 Mbps Duplex Full" :
+ hw->link_speed == ALX_LINK_SPEED_10_HALF ?
+ "10 Mbps Duplex HALF" :
+ "unknown speed");
netif_info(adpt, timer, adpt->netdev,
"NIC Link is Up %s\n", link_desc);
@@ -3171,8 +3167,7 @@ do_csum:
cso = skb_checksum_start_offset(skb);
if (unlikely(cso & 0x1)) {
- dev_err(&pdev->dev, "pay load offset should not be an "
- "event number\n");
+ dev_err(&pdev->dev, "pay load offset should not be an event number\n");
return -1;
} else {
css = cso + skb->csum_offset;
@@ -3642,16 +3637,14 @@ static int __devinit alx_init(struct pci_dev *pdev,
alx_set_register_info_special(adpt);
netif_dbg(adpt, probe, adpt->netdev,
- "num_msix_noque_intrs = %d, num_msix_rxque_intrs = %d, "
- "num_msix_txque_intrs = %d\n",
+ "num_msix_noque_intrs = %d, num_msix_rxque_intrs = %d, num_msix_txque_intrs = %d\n",
adpt->num_msix_noques, adpt->num_msix_rxques,
adpt->num_msix_txques);
netif_dbg(adpt, probe, adpt->netdev, "num_msix_all_intrs = %d\n",
adpt->num_msix_intrs);
netif_dbg(adpt, probe, adpt->netdev,
- "RX Queue Count = %u, HRX Queue Count = %u, "
- "SRX Queue Count = %u, TX Queue Count = %u\n",
+ "RX Queue Count = %u, HRX Queue Count = %u, SRX Queue Count = %u, TX Queue Count = %u\n",
adpt->num_rxques, adpt->num_hw_rxques, adpt->num_sw_rxques,
adpt->num_txques);
@@ -3719,7 +3712,7 @@ static int __devinit alx_init(struct pci_dev *pdev,
CHK_ADPT_FLAG(0, SRSS_EN) ? "Enable" : "Disable");
}
- printk(KERN_INFO "alx: Atheros Gigabit Network Connection\n");
+ pr_info("Atheros Gigabit Network Connection\n");
cards_found++;
return 0;
@@ -3882,7 +3875,7 @@ static int __init alx_init_module(void)
{
int retval;
- printk(KERN_INFO "%s\n", alx_drv_description);
+ pr_info("%s\n", alx_drv_description);
retval = pci_register_driver(&alx_driver);
return retval;
diff --git a/drivers/net/ethernet/atheros/alx/alx_sw.h b/drivers/net/ethernet/atheros/alx/alx_sw.h
index 3daa392..9118da5 100644
--- a/drivers/net/ethernet/atheros/alx/alx_sw.h
+++ b/drivers/net/ethernet/atheros/alx/alx_sw.h
@@ -479,8 +479,8 @@ extern int alc_init_hw_callbacks(struct alx_hw *hw);
extern int alf_init_hw_callbacks(struct alx_hw *hw);
/* Logging message functions */
-void __printf(3, 4) alx_hw_printk(const char *level, const struct alx_hw *hw,
- const char *fmt, ...);
+void __printf(3, 4)
+alx_hw_printk(const char *level, const struct alx_hw *hw, const char *fmt, ...);
#define alx_hw_err(_hw, _format, ...) \
alx_hw_printk(KERN_ERR, _hw, _format, ##__VA_ARGS__)
next prev parent reply other threads:[~2012-02-29 3:52 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-29 1:50 [PATCH] net: add QCA alx Ethernet driver Luis R. Rodriguez
2012-02-29 1:50 ` [PATCH] net: add new QCA alx ethernet driver which supercedes atl1c Luis R. Rodriguez
2012-02-29 1:58 ` David Miller
2012-02-29 1:53 ` [PATCH] net: add QCA alx Ethernet driver David Miller
2012-02-29 2:12 ` Luis R. Rodriguez
2012-02-29 2:15 ` David Miller
2012-02-29 2:19 ` Huang, Xiong
2012-02-29 2:28 ` David Miller
2012-02-29 3:11 ` Huang, Xiong
2012-02-29 3:32 ` David Miller
2012-02-29 7:30 ` Huang, Xiong
2012-02-29 9:38 ` Luis R. Rodriguez
2012-03-22 1:28 ` Luis R. Rodriguez
2012-03-22 9:27 ` Francois Romieu
2012-03-22 13:38 ` Luis R. Rodriguez
2012-03-30 17:10 ` Luis R. Rodriguez
2012-03-30 20:45 ` David Miller
2012-03-30 20:52 ` Luis R. Rodriguez
2012-02-29 3:32 ` Stephen Hemminger
2012-03-01 23:40 ` Francois Romieu
2012-03-02 0:56 ` Joe Perches
2012-02-29 3:52 ` Joe Perches [this message]
[not found] ` <CAA93jw4NwHaLazxK2Eo6a6TuvF3svqdZjTpHUwOyEHv-Q=busQ@mail.gmail.com>
2012-03-30 18:31 ` Luis R. Rodriguez
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1330487546.3742.1.camel@joe2Laptop \
--to=joe@perches.com \
--cc=bryanh@quicinc.com \
--cc=chris.snook@gmail.com \
--cc=davem@davemloft.net \
--cc=kgiori@qca.qualcomm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mathieu@qca.qualcomm.com \
--cc=mcgrof@frijolero.org \
--cc=netdev@vger.kernel.org \
--cc=nic-devel@qualcomm.com \
--cc=qca-linux-team@qualcomm.com \
--cc=rodrigue@qca.qualcomm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).