On 1/9/2019 11:16 PM, Stephen Boyd wrote:
Quoting Raju P L S S S N (2019-01-08 21:34:32)
On 1/7/2019 9:47 PM, Raju P L S S S N wrote:
On 1/4/2019 2:49 AM, Stephen Boyd wrote:
I'd hope that each RSC's PDC timer wakes up the owner of the RSC so that
we can use the sysreg based timers and ignore the MMIO based timers
here. This isn't a very important distinction to make though, so if you
have to use the MMIO timers then it just means some extra DT things need
to be done to relate the MMIO timers with the RSC that has the timer
that needs to be programmed too.
Either way we would need a way to either hook the ARM architected timer
driver in the kernel, or reimplement the bit of code needed to implement
the clockevent based on the ARM architected timer that programs the ARM
timer and the PDC timer together.
Could you please provide some more details on the implementation?
Hi Stephen,
Regardless of implementation options about application processor
subsytem PDC timer, I think there is a need to differentiate the fact
that for application processor subsystem PDC timer programming is done
by SW but not for display processor subsystem as HW sleep solver takes
care of PDC timer during sleep entry/exit. How about having a dt
property like qcom,pdc-timer-mode = "solver"/"sw" ? The current patch
introduced client based model using regmap to achieve this
differentiation between these two subsystems. By using the dt property,
once rpmh-src driver instance for application subsystem can have PDC
timer programing implemented. Let me know if there is another way.
For implementation of PDC timer, I see the following based on above
discussion -
1. Take the existing cpu_pm_notify approach and move the current series
approach of programing PDC timer registers to RSC driver. This wouldn't
involve any changes in clock_event_framework/broadcast framework.
2. Add api hooks (like reading the next wake up programmed) to ARM
architecture timer driver so that the value is copied to PDC timer using
the api with in RSC driver based on cpu_pm_notifications.
3. Changes in clockevent to program both ARM mem timer & PDC timer
together. Could you please share some more details on this?
Please let me know if the first approach is ok.
The first approach requires that we expose internals of the clockevent
and broadcast timer information to drivers. From my perspective it looks
like a weird kludge to workaround the fact that the broadcast timer
doesn't actually work on this platform. That's why I'm suggesting that
you fix the broadcast timer on your platform to actually work, and do
that within the clockevent/broadcast layers instead of indirecting that
through cpu_pm notifiers.
That could be done by making a PDC clockevent that has some DT binding
of a property pointing to an MMIO timer frame and then reimplementing
the MMIO timer code in the PDC driver on top of the frame/register
region it pulls out of there. Or it could be written in reverse by
having the generic MMIO timer driver point to the PDC somehow and
implement some platform specific API to pass that information to the
real wakeup programming part in PDC.
I'm leaning toward the first approach where PDC is the clockevent and
that uses the MMIO timer on the backend to do what it needs to program a
wakeup. That way you can mandate the usage of the physical timer and
keep this quirk away from the ARM timer driver. It also makes the idea
of a qcom,pdc-timer-mode sort of useless because the PDC will have a
property that points to the timer frame and that will mean "use this for
broadcast wakeup". I'm not sure how the ARM folks feel about this
though. It would probably require some sort of ARM timer API that lets
us program the MMIO timer frame from the PDC driver. So exporting
arch_timer_reg_write() and making that always inlined to optimize hot
paths would be required.
Regarding the first approach -
If PDC clk_evt_dev is created and registered to broadcast framework, it
means it will replace MMIO timer event device in broadcast framework (as
only one broadcast event timer can exist). So in registration phase, the
tick broadcast framework would configure broadcat event handler for
PDC(clk_evt_dev).
Even if we have some way of exporting arch_timer_reg_write() to program
the wakeup for MMIO timer, the irq handler for MMIO timer is responsible
to invoke event_handler. However, since MMIO timer is replaced by PDC
timer event device, event handler will not be able to inform broadcast
framework. What can be done about this?
