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@xxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html