Re: [PATCH v14 3/9] clocksource/drivers/arm_arch_timer: Improve printk relevant code

2016-10-26 Thread Fu Wei
Hi Mark, On 20 October 2016 at 23:32, Mark Rutland wrote: > On Thu, Sep 29, 2016 at 02:17:11AM +0800, fu@linaro.org wrote: >> static void arch_timer_banner(unsigned type) >> { >> - pr_info("Architected %s%s%s timer(s) running at %lu.%02luMHz >> (%s%s%s).\n", >> - type

Re: [PATCH v14 3/9] clocksource/drivers/arm_arch_timer: Improve printk relevant code

2016-10-20 Thread Mark Rutland
On Thu, Sep 29, 2016 at 02:17:11AM +0800, fu@linaro.org wrote: > static void arch_timer_banner(unsigned type) > { > - pr_info("Architected %s%s%s timer(s) running at %lu.%02luMHz > (%s%s%s).\n", > - type & ARCH_CP15_TIMER ? "cp15" : "", > - type == (ARCH

[PATCH v14 3/9] clocksource/drivers/arm_arch_timer: Improve printk relevant code

2016-09-28 Thread fu . wei
From: Fu Wei This patch defines pr_fmt(fmt) for all pr_* functions, then the pr_* doesn't need to add "arch_timer:" everytime. Also delete some Blank Spaces in arch_timer_banner, according to the suggestion from checkpatch.pl. No functional change. Signed-off-by: Fu Wei --- drivers/clocksour