Re: [PATCH] mac8390: fix pr_info() calls and change return code

David Miller
2010-04-16T05:53:39Z

From: Finn Thain <fthain@telegraphics.com.au>
Date: Fri, 16 Apr 2010 14:21:00 +1000 (EST)

>
> @@ -668,11 +668,11 @@
> {
> unsigned char *target = nubus_slot_addr(IRQ2SLOT(dev->irq));
> if (ei_debug > 1)
> - pr_info("Need to reset the NS8390 t=%lu...", jiffies);
> + printk(KERN_DEBUG "Need to reset the NS8390 t=%lu...", jiffies);
> ei_status.txing = 0;
> target[0xC0000] = 0;
> if (ei_debug > 1)
> - pr_cont("reset complete\n");
> + printk(KERN_CONT "reset complete\n");
> return;

You're missing the whole point of using pr_info() et al. in that it
includes the bits we define for pr_fmt at the top of the file.

Also, you write absolutely no commit log message entry for your
change explaining why you're doing the things you are doing.

And finally you are doing two completely unrelated things at one
(changing error return values and changing log message levels).