Messages in this thread | | | Date | Fri, 17 Jun 2016 09:08:28 +0800 | From | Caesar Wang <> | Subject | Re: [PATCH v2 0/4] clocksource: rockchip/timer: Support rktimer for rk3399 |
| |
On 2016年06月16日 22:26, Daniel Lezcano wrote: > On 06/07/2016 06:54 AM, Caesar Wang wrote: >> This series patches had been tested on rockchip inside kernel. >> In order to support the rk3399 SoC timer and turn off interrupts and >> IPIs to >> save power in idle. >> Okay, it still works bootup on rk3288/other SoCs, even though many >> socs hasn't used >> the broadcast timer. >> >> History version: >> v1: >> https://lkml.org/lkml/2016/5/25/186 >> >> Easy to test for my borad. >> localhost / # cat /proc/interrupts >> CPU0 CPU1 CPU2 CPU3 CPU4 CPU5 >> 1: 0 0 0 0 0 0 >> GICv3 29 Edge arch_timer >> ... >> 5: 0 0 0 0 0 0 GICv3 >> 113 Level rk_timer >> .. >> >> localhost / # cat /proc/timer_list | grep event_handler >> get "event_handler: hrtimer_interrupt" >> event_handler: tick_handle_oneshot_broadcast >> event_handler: hrtimer_interrupt >> >> That should work for my board. > > Hi Caesar, > > I applied the patches on my tree. I humbly reworded the changelog, if > you disagree with the content, let me know.
Have a look at your tree. (https://git.linaro.org/people/daniel.lezcano/linux.git/shortlog/refs/heads/clockevents/next) That's LGTM.
> > Please, in the future, take care of the following: > > - patch prefix is 'clocksource/driver/<driver_name>: [A-Z].*' > > - the change log is important, it deserves some love. >
Thanks for pointing out, I will keep in mind it.
- Caesar
> Thanks. > > -- Daniel > >
-- caesar wang | software engineer | wxt@rock-chip.com
| |