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 wrote: On Mon, Sep 15, 2014 at 2:49 PM, Stephen Boyd wrote:

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 >

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 wrote: > >> On Mon, Sep 15, 2014 at 2:49 PM, Stephen Boyd wrote: > >>> On 09/15/14 14:47, Sonny Rao wrote: > On

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 Sonny Rao
On Mon, Sep 15, 2014 at 3:51 PM, 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 wrote: >>> On Mon, Sep 15, 2014 at 2:49 PM, Stephen Boyd wrote: On 09/15/14 14:47, Sonny Rao wrote: > On Mon, Sep 15,

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 wrote: >> On Mon, Sep 15, 2014 at 2:49 PM, Stephen Boyd wrote: >>> On 09/15/14 14:47, Sonny Rao wrote: On Mon, Sep 15, 2014 at 1:33 PM, Stephen Boyd wrote: > On 09/15/14 04:10, Catalin

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 wrote: > On Mon, Sep 15, 2014 at 2:49 PM, Stephen Boyd wrote: >> On 09/15/14 14:47, Sonny Rao wrote: >>> On Mon, Sep 15, 2014 at 1:33 PM, Stephen Boyd wrote: On 09/15/14 04:10, Catalin Marinas wrote: > On Fri, Sep 12, 2014 at 07:59:29PM +0100,

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 wrote: > On 09/15/14 14:47, Sonny Rao wrote: >> On Mon, Sep 15, 2014 at 1:33 PM, 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

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 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 1:33 PM, 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

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

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

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 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

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 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 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 wrote: We would need to run this code potentially at processor bringup and after suspend/resume, but that seems possible too. >>>

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 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,

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

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 wrote: > On 09/11/14 17:14, Sonny Rao wrote: > > On Thu, Sep 11, 2014 at 4:56 PM, Stephen Boyd wrote: >> >> >> Where does this platform jump to when a CPU comes up? Is it >> rockchip_secondary_startup()? I wonder if that path could have this >>

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 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 anything... ;-)

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 >>

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 wrote: > On 11/09/14 18:29, Doug Anderson wrote: >> Marc, >> >> On Thu, Sep 11, 2014 at 10:22 AM, Marc Zyngier 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-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 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 can't do

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:22 AM, Marc Zyngier 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 can't do that on ARMv8, > at all). Yes, of course. >>

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 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 Doug Anderson
Will, On Thu, Sep 11, 2014 at 10:07 AM, Will Deacon 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 wrote: >> > I'd say `Only supported for ARM' to better match what we've done. Probably >> > also worth mentioning that this

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 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 hypervisor

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 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 > > CNTHCTL.PL1PCEN

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 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 hypervisor mode and >> we don't want to add the complexity of hypervisor

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 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 the

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

[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
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

[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 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 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 Marc Zyngier
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 hypervisor mode and we don't want to add the complexity of hypervisor there. * The

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 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 Doug Anderson
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 can't do that on ARMv8, at all). Yes, of

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