This patch contains the following cleanups:
- make two needlessly global functions static
- kill an ancient version variable
Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]>
---
drivers/net/ne3210.c |9 +++--
1 files changed, 3 insertions(+), 6 deletions(-)
---
This patch contains the following cleanups:
- make two needlessly global functions static
- kill an ancient version variable
Signed-off-by: Adrian Bunk [EMAIL PROTECTED]
---
drivers/net/ne3210.c |9 +++--
1 files changed, 3 insertions(+), 6 deletions(-)
---
On Mon, Feb 21, 2005 at 03:39:54PM -0500, Jeff Garzik wrote:
> Adrian Bunk wrote:
> >@@ -197,7 +194,7 @@
> > ei_status.priv = phys_mem;
> >
> > if (ei_debug > 0)
> >-printk(version);
> >+printk("ne3210 driver");
>
>
> missing newline. Do something like "ns3210
On Mon, Feb 21, 2005 at 03:39:54PM -0500, Jeff Garzik wrote:
Adrian Bunk wrote:
@@ -197,7 +194,7 @@
ei_status.priv = phys_mem;
if (ei_debug 0)
-printk(version);
+printk(ne3210 driver);
missing newline. Do something like ns3210 __DATE__ loaded.\n
Adrian Bunk wrote:
@@ -197,7 +194,7 @@
ei_status.priv = phys_mem;
if (ei_debug > 0)
- printk(version);
+ printk("ne3210 driver");
missing newline. Do something like "ns3210 __DATE__ loaded.\n"
Ditto for seeq8002.
Jeff
-
To unsubscribe from this list: send the line "unsubscribe
On Sun, Feb 20, 2005 at 08:40:13PM -0500, Jeff Garzik wrote:
> Adrian Bunk wrote:
> >-if (ei_debug > 0)
> >-printk(version);
>
>
> I agree the version variable is outdated, but I disagree that the driver
> intro banner should be removed completely.
A few lines above, the driver
On Sun, Feb 20, 2005 at 08:40:13PM -0500, Jeff Garzik wrote:
Adrian Bunk wrote:
-if (ei_debug 0)
-printk(version);
I agree the version variable is outdated, but I disagree that the driver
intro banner should be removed completely.
A few lines above, the driver already
Adrian Bunk wrote:
@@ -197,7 +194,7 @@
ei_status.priv = phys_mem;
if (ei_debug 0)
- printk(version);
+ printk(ne3210 driver);
missing newline. Do something like ns3210 __DATE__ loaded.\n
Ditto for seeq8002.
Jeff
-
To unsubscribe from this list: send the line unsubscribe
Adrian Bunk wrote:
- if (ei_debug > 0)
- printk(version);
I agree the version variable is outdated, but I disagree that the driver
intro banner should be removed completely.
Jeff
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of
Adrian Bunk wrote:
- if (ei_debug 0)
- printk(version);
I agree the version variable is outdated, but I disagree that the driver
intro banner should be removed completely.
Jeff
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a
This patch contains the following cleanups:
- make two needlessly global functions static
- kill an ancient version variable
Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]>
---
drivers/net/ne3210.c | 10 ++
1 files changed, 2 insertions(+), 8 deletions(-)
---
This patch contains the following cleanups:
- make two needlessly global functions static
- kill an ancient version variable
Signed-off-by: Adrian Bunk [EMAIL PROTECTED]
---
drivers/net/ne3210.c | 10 ++
1 files changed, 2 insertions(+), 8 deletions(-)
---
12 matches
Mail list logo