Hi Joe, On 9 December 2016 at 05:10, Joe Perches <joe@xxxxxxxxxxx> wrote: > On Fri, 2016-12-09 at 01:33 +0800, fu.wei@xxxxxxxxxx wrote: >> From: Fu Wei <fu.wei@xxxxxxxxxx> >> >> This patch defines pr_fmt(fmt) for all pr_* functions, >> then the pr_* doesn't need to add "arch_timer:" everytime. > > trivia: > >> diff --git a/drivers/clocksource/arm_arch_timer.c b/drivers/clocksource/arm_arch_timer.c > [] >> @@ -966,7 +967,7 @@ static int __init arch_timer_mem_init(struct device_node *np) >> >> ret = -EINVAL; >> if (!irq) { >> - pr_err("arch_timer: Frame missing %s irq", >> + pr_err("Frame missing %s irq", >> arch_timer_mem_use_virtual ? "virt" : "phys"); >> goto out; >> } > > Missing terminating newline > > pr_err("Frame missing %s irq\n", yes, you are right , will fix it. :-) Thanks -- Best regards, Fu Wei Software Engineer Red Hat -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html