2009-09-22 14:05:41

by Alan Jenkins

Subject: [PATCH] 8139cp: fix duplicate loglevel in module load message

This was introduced by b93d58 "8139*: convert printk() to pr_<foo>()":

[ 2256252443 ] <6>8139cp: 10/100 PCI Ethernet driver v1.3 (Mar 22, 2004)

The "version" string is printed using pr_info(), so it doesn't need to
include a loglevel.

Signed-off-by: Alan Jenkins <[email protected]>
CC: Alexander Beregalov <[email protected]>
---
drivers/net/8139cp.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/net/8139cp.c b/drivers/net/8139cp.c
index d0dbbf3..6841a9a 100644
--- a/drivers/net/8139cp.c
+++ b/drivers/net/8139cp.c
@@ -87,7 +87,7 @@

/* These identify the driver base version and may not be removed. */
static char version[] =
-KERN_INFO DRV_NAME ": 10/100 PCI Ethernet driver v" DRV_VERSION " (" DRV_RELDATE ")\n";
+DRV_NAME ": 10/100 PCI Ethernet driver v" DRV_VERSION " (" DRV_RELDATE ")\n";

MODULE_AUTHOR("Jeff Garzik <[email protected]>");
MODULE_DESCRIPTION("RealTek RTL-8139C+ series 10/100 PCI Ethernet driver");
--
1.6.3.2

2009-09-22 21:25:39

by David Miller

Subject: Re: [PATCH] 8139cp: fix duplicate loglevel in module load message

From: Alan Jenkins <[email protected]>
Date: Tue, 22 Sep 2009 15:05:39 +0100

> This was introduced by b93d58 "8139*: convert printk() to pr_<foo>()":
>
> [ 2256252443 ] <6>8139cp: 10/100 PCI Ethernet driver v1.3 (Mar 22, 2004)
>
> The "version" string is printed using pr_info(), so it doesn't need to
> include a loglevel.
>
> Signed-off-by: Alan Jenkins <[email protected]>
> CC: Alexander Beregalov <[email protected]>

Applied.