netdev
[Top] [All Lists]

Re: [PATCH 2.6.4-rc2-bk4 2/2] pcnet32 use netif_msg_init

To: Don Fry <brazilnut@xxxxxxxxxx>
Subject: Re: [PATCH 2.6.4-rc2-bk4 2/2] pcnet32 use netif_msg_init
From: Jeff Garzik <jgarzik@xxxxxxxxx>
Date: Mon, 08 Mar 2004 21:19:35 -0500
Cc: tsbogend@xxxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx
In-reply-to: <200403081946.i28Jkrb10646@DYN318364BLD.beaverton.ibm.com>
References: <200403081946.i28Jkrb10646@DYN318364BLD.beaverton.ibm.com>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030703
Don Fry wrote:
@@ -2013,8 +2016,7 @@
{
printk(KERN_INFO "%s", version);
- if (debug >= 0 && debug < (sizeof(int) - 1))
- pcnet32_debug = 1 << debug;
+ pcnet32_debug = netif_msg_init(debug, 0);


Do you really want to default to silence? IMO define a PCNET32_DEF_MSG_ENABLE with a few of the critical bits like probe-time information.

        Jeff




<Prev in Thread] Current Thread [Next in Thread>