Re: [PATCH] MIPS: Add severity levels to printk statements during kernel setup.

[Geert Uytterhoeven <geert@linux-m68k.org>]

On Sat, 26 Jul 2008, Mike Crowe wrote:
> On Fri, 25 Jul 2008, I wrote:
> >> --- a/arch/mips/kernel/setup.c
> >> +++ b/arch/mips/kernel/setup.c
> >> @@ -78,7 +78,7 @@ void __init add_memory_region(phys_t start, phys_t size, 
> >> long type)
> >>  
> >>    /* Sanity check */
> >>    if (start + size < start) {
> >> -          printk("Trying to add an invalid memory region, skipped\n");
> >> +          printk(KERN_WARNING "Trying to add an invalid memory region, 
> >> skipped\n");
>  
> On Fri, Jul 25, 2008 at 08:04:57PM +0200, Geert Uytterhoeven wrote:
> > Why not convert to pr_warning(), while you're at it?
> 
> I can do. I'm just a bit behind the times. :-)
> 
> Should I use pr_{warning,err,info} everywhere rather than printk? Is

Yes, please.

> it worth fixing up the other calls to printk that I didn't need to
> "fix"?

If you have the time, you can do that. It also makes those (typically
too long) lines a bit shorter.

Gr{oetje,eeting}s,

                                                Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                                            -- Linus Torvalds