Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-16 Thread Catalin Marinas
On Mon, Sep 15, 2014 at 11:51:14PM +0100, Christopher Covington wrote: Hi Sonny, On 09/15/2014 06:04 PM, Sonny Rao wrote: On Mon, Sep 15, 2014 at 2:52 PM, Sonny Rao sonny...@chromium.org wrote: On Mon, Sep 15, 2014 at 2:49 PM, Stephen Boyd sb...@codeaurora.org wrote: On 09/15/14 14:47,

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-16 Thread Catalin Marinas
On Mon, Sep 15, 2014 at 09:33:03PM +0100, Stephen Boyd wrote: On 09/15/14 04:10, Catalin Marinas wrote: On Fri, Sep 12, 2014 at 07:59:29PM +0100, Stephen Boyd wrote: On 09/12/14 05:14, Marc Zyngier wrote: We surely can handle the UNDEF and do something there. We just can't do it the way

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-16 Thread Christopher Covington
On 09/16/2014 06:42 AM, Catalin Marinas wrote: On Mon, Sep 15, 2014 at 11:51:14PM +0100, Christopher Covington wrote: Hi Sonny, On 09/15/2014 06:04 PM, Sonny Rao wrote: On Mon, Sep 15, 2014 at 2:52 PM, Sonny Rao sonny...@chromium.org wrote: On Mon, Sep 15, 2014 at 2:49 PM, Stephen Boyd

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-15 Thread Catalin Marinas
On Fri, Sep 12, 2014 at 07:59:29PM +0100, Stephen Boyd wrote: On 09/12/14 05:14, Marc Zyngier wrote: On 12/09/14 12:43, Christopher Covington wrote: On 09/11/2014 01:43 PM, Marc Zyngier wrote: On 11/09/14 18:29, Doug Anderson wrote: I did this in the past (again, see Sonny's thread),

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-15 Thread Stephen Boyd
On 09/15/14 04:10, Catalin Marinas wrote: On Fri, Sep 12, 2014 at 07:59:29PM +0100, Stephen Boyd wrote: On 09/12/14 05:14, Marc Zyngier wrote: We surely can handle the UNDEF and do something there. We just can't do it the way Doug described it above. I suggested doing that for something else

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-15 Thread Sonny Rao
On Mon, Sep 15, 2014 at 1:33 PM, Stephen Boyd sb...@codeaurora.org wrote: On 09/15/14 04:10, Catalin Marinas wrote: On Fri, Sep 12, 2014 at 07:59:29PM +0100, Stephen Boyd wrote: On 09/12/14 05:14, Marc Zyngier wrote: We surely can handle the UNDEF and do something there. We just can't do

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-15 Thread Stephen Boyd
On 09/15/14 14:47, Sonny Rao wrote: On Mon, Sep 15, 2014 at 1:33 PM, Stephen Boyd sb...@codeaurora.org wrote: On 09/15/14 04:10, Catalin Marinas wrote: On Fri, Sep 12, 2014 at 07:59:29PM +0100, Stephen Boyd wrote: On 09/12/14 05:14, Marc Zyngier wrote: We surely can handle the UNDEF and do

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-15 Thread Sonny Rao
On Mon, Sep 15, 2014 at 2:49 PM, Stephen Boyd sb...@codeaurora.org wrote: On 09/15/14 14:47, Sonny Rao wrote: On Mon, Sep 15, 2014 at 1:33 PM, Stephen Boyd sb...@codeaurora.org wrote: On 09/15/14 04:10, Catalin Marinas wrote: On Fri, Sep 12, 2014 at 07:59:29PM +0100, Stephen Boyd wrote: On

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-15 Thread Sonny Rao
On Mon, Sep 15, 2014 at 2:52 PM, Sonny Rao sonny...@chromium.org wrote: On Mon, Sep 15, 2014 at 2:49 PM, Stephen Boyd sb...@codeaurora.org wrote: On 09/15/14 14:47, Sonny Rao wrote: On Mon, Sep 15, 2014 at 1:33 PM, Stephen Boyd sb...@codeaurora.org wrote: On 09/15/14 04:10, Catalin Marinas

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-15 Thread Christopher Covington
Hi Sonny, On 09/15/2014 06:04 PM, Sonny Rao wrote: On Mon, Sep 15, 2014 at 2:52 PM, Sonny Rao sonny...@chromium.org wrote: On Mon, Sep 15, 2014 at 2:49 PM, Stephen Boyd sb...@codeaurora.org wrote: On 09/15/14 14:47, Sonny Rao wrote: On Mon, Sep 15, 2014 at 1:33 PM, Stephen Boyd

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-15 Thread Sonny Rao
On Mon, Sep 15, 2014 at 3:51 PM, Christopher Covington c...@codeaurora.org wrote: Hi Sonny, On 09/15/2014 06:04 PM, Sonny Rao wrote: On Mon, Sep 15, 2014 at 2:52 PM, Sonny Rao sonny...@chromium.org wrote: On Mon, Sep 15, 2014 at 2:49 PM, Stephen Boyd sb...@codeaurora.org wrote: On 09/15/14

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-12 Thread Marc Zyngier
On 12/09/14 01:01, Doug Anderson wrote: Stephen, On Thu, Sep 11, 2014 at 4:56 PM, Stephen Boyd sb...@codeaurora.org wrote: On 09/11/14 10:43, Marc Zyngier wrote: If I was suicidal, I'd suggest you could pass a parameter to the command line, interpreted by the timer code... But I since I'm

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-12 Thread Christopher Covington
Hi Marc, On 09/11/2014 01:43 PM, Marc Zyngier wrote: On 11/09/14 18:29, Doug Anderson wrote: Marc, On Thu, Sep 11, 2014 at 10:22 AM, Marc Zyngier marc.zyng...@arm.com wrote: We would need to run this code potentially at processor bringup and after suspend/resume, but that seems possible

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-12 Thread Marc Zyngier
Hi Christopher, On 12/09/14 12:43, Christopher Covington wrote: Hi Marc, On 09/11/2014 01:43 PM, Marc Zyngier wrote: On 11/09/14 18:29, Doug Anderson wrote: Marc, On Thu, Sep 11, 2014 at 10:22 AM, Marc Zyngier marc.zyng...@arm.com wrote: We would need to run this code potentially at

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-12 Thread Stephen Boyd
On 09/12/14 05:14, Marc Zyngier wrote: Hi Christopher, On 12/09/14 12:43, Christopher Covington wrote: Hi Marc, On 09/11/2014 01:43 PM, Marc Zyngier wrote: On 11/09/14 18:29, Doug Anderson wrote: I did this in the past (again, see Sonny's thread), but didn't consider myself knowledgeable

[PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-11 Thread Doug Anderson
Some 32-bit (ARMv7) systems are architected like this: * The firmware doesn't know and doesn't care about hypervisor mode and we don't want to add the complexity of hypervisor there. * The firmware isn't involved in SMP bringup or resume. * The ARCH timer come up with an uninitialized offset

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-11 Thread Doug Anderson
Will, On Thu, Sep 11, 2014 at 9:47 AM, Will Deacon will.dea...@arm.com wrote: On Thu, Sep 11, 2014 at 05:16:44PM +0100, Doug Anderson wrote: Some 32-bit (ARMv7) systems are architected like this: * The firmware doesn't know and doesn't care about hypervisor mode and we don't want to add

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-11 Thread Will Deacon
On Thu, Sep 11, 2014 at 05:59:53PM +0100, Doug Anderson wrote: On Thu, Sep 11, 2014 at 9:47 AM, Will Deacon will.dea...@arm.com wrote: I'd say `Only supported for ARM' to better match what we've done. Probably also worth mentioning that this relies on the hypervisor/firmware having set

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-11 Thread Doug Anderson
Hi, On Thu, Sep 11, 2014 at 10:00 AM, Marc Zyngier marc.zyng...@arm.com wrote: On 11/09/14 17:47, Will Deacon wrote: On Thu, Sep 11, 2014 at 05:16:44PM +0100, Doug Anderson wrote: Some 32-bit (ARMv7) systems are architected like this: * The firmware doesn't know and doesn't care about

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-11 Thread Doug Anderson
Will, On Thu, Sep 11, 2014 at 10:07 AM, Will Deacon will.dea...@arm.com wrote: On Thu, Sep 11, 2014 at 05:59:53PM +0100, Doug Anderson wrote: On Thu, Sep 11, 2014 at 9:47 AM, Will Deacon will.dea...@arm.com wrote: I'd say `Only supported for ARM' to better match what we've done. Probably

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-11 Thread Will Deacon
On Thu, Sep 11, 2014 at 05:16:44PM +0100, Doug Anderson wrote: Some 32-bit (ARMv7) systems are architected like this: * The firmware doesn't know and doesn't care about hypervisor mode and we don't want to add the complexity of hypervisor there. * The firmware isn't involved in SMP

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-11 Thread Marc Zyngier
On 11/09/14 18:11, Doug Anderson wrote: Hi, On Thu, Sep 11, 2014 at 10:00 AM, Marc Zyngier marc.zyng...@arm.com wrote: On 11/09/14 17:47, Will Deacon wrote: On Thu, Sep 11, 2014 at 05:16:44PM +0100, Doug Anderson wrote: Some 32-bit (ARMv7) systems are architected like this: * The firmware

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-11 Thread Marc Zyngier
On 11/09/14 18:29, Doug Anderson wrote: Marc, On Thu, Sep 11, 2014 at 10:22 AM, Marc Zyngier marc.zyng...@arm.com wrote: We would need to run this code potentially at processor bringup and after suspend/resume, but that seems possible too. Note that this would be an ARMv7 only thing (you

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-11 Thread Doug Anderson
Marc, On Thu, Sep 11, 2014 at 10:43 AM, Marc Zyngier marc.zyng...@arm.com wrote: On 11/09/14 18:29, Doug Anderson wrote: Marc, On Thu, Sep 11, 2014 at 10:22 AM, Marc Zyngier marc.zyng...@arm.com wrote: We would need to run this code potentially at processor bringup and after suspend/resume,

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-11 Thread Stephen Boyd
On 09/11/14 10:43, Marc Zyngier wrote: If I was suicidal, I'd suggest you could pass a parameter to the command line, interpreted by the timer code... But I since I'm not, let's pretend I haven't said anything... ;-) I did this in the past (again, see Sonny's thread), but didn't consider

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-11 Thread Doug Anderson
Stephen, On Thu, Sep 11, 2014 at 4:56 PM, Stephen Boyd sb...@codeaurora.org wrote: On 09/11/14 10:43, Marc Zyngier wrote: If I was suicidal, I'd suggest you could pass a parameter to the command line, interpreted by the timer code... But I since I'm not, let's pretend I haven't said

Re: [PATCH v2] clocksource: arch_timer: Allow the device tree to specify the physical timer

2014-09-11 Thread Sonny Rao
On Thu, Sep 11, 2014 at 6:17 PM, Stephen Boyd sb...@codeaurora.org wrote: On 09/11/14 17:14, Sonny Rao wrote: On Thu, Sep 11, 2014 at 4:56 PM, Stephen Boyd sb...@codeaurora.org wrote: Where does this platform jump to when a CPU comes up? Is it rockchip_secondary_startup()? I wonder if that