Messages in this thread | | | Subject | Re: [PATCH v5 2/2] clocksource/drivers/npcm: Add NPCM7xx timer driver | From | Daniel Lezcano <> | Date | Fri, 9 Mar 2018 14:25:03 +0100 |
| |
On 08/03/2018 23:37, Tomer Maimon wrote: > On 8 March 2018 at 17:33, Daniel Lezcano <daniel.lezcano@linaro.org> wrote: >> On 08/03/2018 16:24, Tomer Maimon wrote: >>> Add Nuvoton BMC NPCM7xx timer driver. >>> >>> The clocksource Enable 24-bit TIMER0 and TIMER1 counters, >>> while TIMER0 serve as clockevent and TIMER1 serve as clocksource. >>> >>> Signed-off-by: Tomer Maimon <tmaimon77@gmail.com> >>> Reviewed-by: Brendan Higgins <brendanhiggins@xxxxxxxxxx> >>> --- >> >> [ ... ] >> >>> + >>> + pr_info("Enabling NPCM7xx clocksource timer base: 0x%x, IRQ: %d ", >>> + (int)timer_of_base(&npcm7xx_to), timer_of_irq(&npcm7xx_to)); >> >> You can use '%p' instead of '0x%x' and remove the (int) cast. > I think we need to use in the pr_info %px, if I use %p I get in the > kernel boot print "(ptrval)" instead of the address. > please refer: https://lwn.net/Articles/740249/
Ah yes, right.
Thanks
-- Daniel
-- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
| |