[PATCH v2 15/17] Staging: quickstart: Use pr_err and pr_info for logs
Szymon Janc
szymon at janc.net.pl
Sat Jan 14 22:20:56 UTC 2012
Hi,
> > +#define pr_fmt(fmt) KBUILD_MODNAME": " fmt
>
> Add a space please between KBUILD_MODNAME and the quoted string.
> #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
Will fix that in V3. (I'll wait few more days to see if there are some more
comments from other people)
> > @@ -207,9 +209,8 @@ static int quickstart_acpi_ghid(struct
> > quickstart_acpi *quickstart)
> >
> > quickstart->button->id = *(uint64_t *)buffer.pointer;
> > break;
> >
> > default:
> > - printk(KERN_ERR "quickstart: %s GHID method returned buffer "
> > - "of unexpected length %u\n",
> > - quickstart->button->name, buffer.length);
> > + pr_err("%s GHID method returned buffer of unexpected length %u"
> > + "\n", quickstart->button->name, buffer.length);
>
> Please try not to break format strings into multiple bits.
> It's very error prone and can make it harder to grep.
> It's OK to have the line with the format exceed 80 chars.
ditto :)
--
Szymon K. Janc
szymon at janc.net.pl
More information about the devel
mailing list