Re: [PATCH 04/13] autofs4 - change printks AUTOFS defined prints

From: Ian Kent
Date: Wed Nov 05 2014 - 18:42:27 EST


On Wed, 2014-11-05 at 15:20 -0800, Joe Perches wrote:
>
> > But idea of using pr_xxx() and pr_fmt() (actually that's too open to
> > name clashes so it would need to be named something like autofs_pr_fmt()
> > anyway) looks like it results in less readable code so I'd really prefer
> > not to do that.
>
> Using pr_info/pr_debug (or any other pr_<level>) is a
> generic mechanism in the kernel. Adding a
> #define pr_fmt is also generic thing that works with
> all the pr_<level> uses in a specific compilation unit.

LOL, so you recommend I look more closely, I will.
In the meantime I'll send the other patches to Andrew.

Ian

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/