* [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init @ 2013-01-21 1:22 ` Thomas Abraham 0 siblings, 0 replies; 18+ messages in thread From: Thomas Abraham @ 2013-01-21 1:22 UTC (permalink / raw) To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r Cc: swarren-DDmLM1+adcrQT0dZR+AlfA, johnstul-r/Jw6+rmf7HQT0dZR+AlfA, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, rob.herring-bsGFqQB8/DxBDgjK7y7TUQ Add an entry in __clksrc_of_table so that ARMv7 architected timer is discoverable using call to clocksource_of_init. Signed-off-by: Thomas Abraham <thomas.abraham-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> --- arch/arm/kernel/arch_timer.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c index c8ef207..d21aada 100644 --- a/arch/arm/kernel/arch_timer.c +++ b/arch/arm/kernel/arch_timer.c @@ -504,6 +504,9 @@ int __init arch_timer_of_register(void) return arch_timer_register(); } +#ifdef CONFIG_CLKSRC_OF +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) +#endif int __init arch_timer_sched_clock_init(void) { -- 1.7.5.4 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init @ 2013-01-21 1:22 ` Thomas Abraham 0 siblings, 0 replies; 18+ messages in thread From: Thomas Abraham @ 2013-01-21 1:22 UTC (permalink / raw) To: linux-arm-kernel Add an entry in __clksrc_of_table so that ARMv7 architected timer is discoverable using call to clocksource_of_init. Signed-off-by: Thomas Abraham <thomas.abraham@linaro.org> --- arch/arm/kernel/arch_timer.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c index c8ef207..d21aada 100644 --- a/arch/arm/kernel/arch_timer.c +++ b/arch/arm/kernel/arch_timer.c @@ -504,6 +504,9 @@ int __init arch_timer_of_register(void) return arch_timer_register(); } +#ifdef CONFIG_CLKSRC_OF +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) +#endif int __init arch_timer_sched_clock_init(void) { -- 1.7.5.4 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init 2013-01-21 1:22 ` Thomas Abraham @ 2013-01-21 12:13 ` Mark Rutland -1 siblings, 0 replies; 18+ messages in thread From: Mark Rutland @ 2013-01-21 12:13 UTC (permalink / raw) To: Thomas Abraham Cc: johnstul, devicetree-discuss, swarren, rob.herring, linux-arm-kernel Hi Thomas, This looks really useful, and it's going to conflict with my arch_timer / arm_generic driver unification series [1]. I'm happy to take this as part of my series if that's ok with you? On Mon, Jan 21, 2013 at 01:22:06AM +0000, Thomas Abraham wrote: > Add an entry in __clksrc_of_table so that ARMv7 architected timer is > discoverable using call to clocksource_of_init. > > Signed-off-by: Thomas Abraham <thomas.abraham@linaro.org> > --- > arch/arm/kernel/arch_timer.c | 3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c > index c8ef207..d21aada 100644 > --- a/arch/arm/kernel/arch_timer.c > +++ b/arch/arm/kernel/arch_timer.c > @@ -504,6 +504,9 @@ int __init arch_timer_of_register(void) > > return arch_timer_register(); > } > +#ifdef CONFIG_CLKSRC_OF > +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) > +#endif > > int __init arch_timer_sched_clock_init(void) > { > -- > 1.7.5.4 > > _______________________________________________ > devicetree-discuss mailing list > devicetree-discuss@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/devicetree-discuss > Thanks, Mark. [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2013-January/142070.html ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init @ 2013-01-21 12:13 ` Mark Rutland 0 siblings, 0 replies; 18+ messages in thread From: Mark Rutland @ 2013-01-21 12:13 UTC (permalink / raw) To: linux-arm-kernel Hi Thomas, This looks really useful, and it's going to conflict with my arch_timer / arm_generic driver unification series [1]. I'm happy to take this as part of my series if that's ok with you? On Mon, Jan 21, 2013 at 01:22:06AM +0000, Thomas Abraham wrote: > Add an entry in __clksrc_of_table so that ARMv7 architected timer is > discoverable using call to clocksource_of_init. > > Signed-off-by: Thomas Abraham <thomas.abraham@linaro.org> > --- > arch/arm/kernel/arch_timer.c | 3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c > index c8ef207..d21aada 100644 > --- a/arch/arm/kernel/arch_timer.c > +++ b/arch/arm/kernel/arch_timer.c > @@ -504,6 +504,9 @@ int __init arch_timer_of_register(void) > > return arch_timer_register(); > } > +#ifdef CONFIG_CLKSRC_OF > +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) > +#endif > > int __init arch_timer_sched_clock_init(void) > { > -- > 1.7.5.4 > > _______________________________________________ > devicetree-discuss mailing list > devicetree-discuss at lists.ozlabs.org > https://lists.ozlabs.org/listinfo/devicetree-discuss > Thanks, Mark. [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2013-January/142070.html ^ permalink raw reply [flat|nested] 18+ messages in thread
[parent not found: <20130121121353.GH15707-NuALmloUBlrZROr8t4l/smS4ubULX0JqMm0uRHvK7Nw@public.gmane.org>]
* Re: [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init 2013-01-21 12:13 ` Mark Rutland @ 2013-01-21 18:19 ` Thomas Abraham -1 siblings, 0 replies; 18+ messages in thread From: Thomas Abraham @ 2013-01-21 18:19 UTC (permalink / raw) To: Mark Rutland Cc: johnstul-r/Jw6+rmf7HQT0dZR+AlfA, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, swarren-DDmLM1+adcrQT0dZR+AlfA, rob.herring-bsGFqQB8/DxBDgjK7y7TUQ, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r Hi Mark, On 21 January 2013 04:13, Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org> wrote: > Hi Thomas, > > This looks really useful, and it's going to conflict with my arch_timer / > arm_generic driver unification series [1]. > > I'm happy to take this as part of my series if that's ok with you? Sure, that's fine. This patch would probably be needed in 3.9 on Exynos platforms. So you could squash this change into your series. Thanks, Thomas. > > On Mon, Jan 21, 2013 at 01:22:06AM +0000, Thomas Abraham wrote: >> Add an entry in __clksrc_of_table so that ARMv7 architected timer is >> discoverable using call to clocksource_of_init. >> >> Signed-off-by: Thomas Abraham <thomas.abraham-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> >> --- >> arch/arm/kernel/arch_timer.c | 3 +++ >> 1 files changed, 3 insertions(+), 0 deletions(-) >> >> diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c >> index c8ef207..d21aada 100644 >> --- a/arch/arm/kernel/arch_timer.c >> +++ b/arch/arm/kernel/arch_timer.c >> @@ -504,6 +504,9 @@ int __init arch_timer_of_register(void) >> >> return arch_timer_register(); >> } >> +#ifdef CONFIG_CLKSRC_OF >> +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) >> +#endif >> >> int __init arch_timer_sched_clock_init(void) >> { >> -- >> 1.7.5.4 >> >> _______________________________________________ >> devicetree-discuss mailing list >> devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org >> https://lists.ozlabs.org/listinfo/devicetree-discuss >> > > Thanks, > Mark. > > [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2013-January/142070.html > ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init @ 2013-01-21 18:19 ` Thomas Abraham 0 siblings, 0 replies; 18+ messages in thread From: Thomas Abraham @ 2013-01-21 18:19 UTC (permalink / raw) To: linux-arm-kernel Hi Mark, On 21 January 2013 04:13, Mark Rutland <mark.rutland@arm.com> wrote: > Hi Thomas, > > This looks really useful, and it's going to conflict with my arch_timer / > arm_generic driver unification series [1]. > > I'm happy to take this as part of my series if that's ok with you? Sure, that's fine. This patch would probably be needed in 3.9 on Exynos platforms. So you could squash this change into your series. Thanks, Thomas. > > On Mon, Jan 21, 2013 at 01:22:06AM +0000, Thomas Abraham wrote: >> Add an entry in __clksrc_of_table so that ARMv7 architected timer is >> discoverable using call to clocksource_of_init. >> >> Signed-off-by: Thomas Abraham <thomas.abraham@linaro.org> >> --- >> arch/arm/kernel/arch_timer.c | 3 +++ >> 1 files changed, 3 insertions(+), 0 deletions(-) >> >> diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c >> index c8ef207..d21aada 100644 >> --- a/arch/arm/kernel/arch_timer.c >> +++ b/arch/arm/kernel/arch_timer.c >> @@ -504,6 +504,9 @@ int __init arch_timer_of_register(void) >> >> return arch_timer_register(); >> } >> +#ifdef CONFIG_CLKSRC_OF >> +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) >> +#endif >> >> int __init arch_timer_sched_clock_init(void) >> { >> -- >> 1.7.5.4 >> >> _______________________________________________ >> devicetree-discuss mailing list >> devicetree-discuss at lists.ozlabs.org >> https://lists.ozlabs.org/listinfo/devicetree-discuss >> > > Thanks, > Mark. > > [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2013-January/142070.html > ^ permalink raw reply [flat|nested] 18+ messages in thread
[parent not found: <1358731326-2979-1-git-send-email-thomas.abraham-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>]
* Re: [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init 2013-01-21 1:22 ` Thomas Abraham @ 2013-01-21 17:21 ` Stephen Warren -1 siblings, 0 replies; 18+ messages in thread From: Stephen Warren @ 2013-01-21 17:21 UTC (permalink / raw) To: Thomas Abraham Cc: johnstul-r/Jw6+rmf7HQT0dZR+AlfA, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, swarren-DDmLM1+adcrQT0dZR+AlfA, rob.herring-bsGFqQB8/DxBDgjK7y7TUQ, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r On 01/20/2013 06:22 PM, Thomas Abraham wrote: > Add an entry in __clksrc_of_table so that ARMv7 architected timer is > discoverable using call to clocksource_of_init. > diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c > +#ifdef CONFIG_CLKSRC_OF > +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) > +#endif I wonder if we shouldn't enhance include/linux/clocksource.h to define CLOCKSOURCE_OF_DECLARE even when !CONFIG_CLKSRC_OF; that way, drivers wouldn't need that ifdef. ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init @ 2013-01-21 17:21 ` Stephen Warren 0 siblings, 0 replies; 18+ messages in thread From: Stephen Warren @ 2013-01-21 17:21 UTC (permalink / raw) To: linux-arm-kernel On 01/20/2013 06:22 PM, Thomas Abraham wrote: > Add an entry in __clksrc_of_table so that ARMv7 architected timer is > discoverable using call to clocksource_of_init. > diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c > +#ifdef CONFIG_CLKSRC_OF > +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) > +#endif I wonder if we shouldn't enhance include/linux/clocksource.h to define CLOCKSOURCE_OF_DECLARE even when !CONFIG_CLKSRC_OF; that way, drivers wouldn't need that ifdef. ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init 2013-01-21 17:21 ` Stephen Warren @ 2013-01-21 18:25 ` Marc Zyngier -1 siblings, 0 replies; 18+ messages in thread From: Marc Zyngier @ 2013-01-21 18:25 UTC (permalink / raw) To: Stephen Warren Cc: swarren, johnstul, devicetree-discuss, rob.herring, Thomas Abraham, linux-arm-kernel On Mon, 21 Jan 2013 10:21:08 -0700, Stephen Warren <swarren@wwwdotorg.org> wrote: > On 01/20/2013 06:22 PM, Thomas Abraham wrote: >> Add an entry in __clksrc_of_table so that ARMv7 architected timer is >> discoverable using call to clocksource_of_init. > >> diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c > >> +#ifdef CONFIG_CLKSRC_OF >> +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", >> arch_timer_of_register) >> +#endif > > I wonder if we shouldn't enhance include/linux/clocksource.h to define > CLOCKSOURCE_OF_DECLARE even when !CONFIG_CLKSRC_OF; that way, drivers > wouldn't need that ifdef. Indeed. That would be a lot nicer. M. -- Fast, cheap, reliable. Pick two. ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init @ 2013-01-21 18:25 ` Marc Zyngier 0 siblings, 0 replies; 18+ messages in thread From: Marc Zyngier @ 2013-01-21 18:25 UTC (permalink / raw) To: linux-arm-kernel On Mon, 21 Jan 2013 10:21:08 -0700, Stephen Warren <swarren@wwwdotorg.org> wrote: > On 01/20/2013 06:22 PM, Thomas Abraham wrote: >> Add an entry in __clksrc_of_table so that ARMv7 architected timer is >> discoverable using call to clocksource_of_init. > >> diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c > >> +#ifdef CONFIG_CLKSRC_OF >> +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", >> arch_timer_of_register) >> +#endif > > I wonder if we shouldn't enhance include/linux/clocksource.h to define > CLOCKSOURCE_OF_DECLARE even when !CONFIG_CLKSRC_OF; that way, drivers > wouldn't need that ifdef. Indeed. That would be a lot nicer. M. -- Fast, cheap, reliable. Pick two. ^ permalink raw reply [flat|nested] 18+ messages in thread
[parent not found: <50FD7904.1070201-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>]
* Re: [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init 2013-01-21 17:21 ` Stephen Warren @ 2013-01-21 18:30 ` Thomas Abraham -1 siblings, 0 replies; 18+ messages in thread From: Thomas Abraham @ 2013-01-21 18:30 UTC (permalink / raw) To: Stephen Warren Cc: johnstul-r/Jw6+rmf7HQT0dZR+AlfA, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, swarren-DDmLM1+adcrQT0dZR+AlfA, rob.herring-bsGFqQB8/DxBDgjK7y7TUQ, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r Hi Stephen, On 21 January 2013 09:21, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote: > On 01/20/2013 06:22 PM, Thomas Abraham wrote: >> Add an entry in __clksrc_of_table so that ARMv7 architected timer is >> discoverable using call to clocksource_of_init. > >> diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c > >> +#ifdef CONFIG_CLKSRC_OF >> +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) >> +#endif > > I wonder if we shouldn't enhance include/linux/clocksource.h to define > CLOCKSOURCE_OF_DECLARE even when !CONFIG_CLKSRC_OF; that way, drivers > wouldn't need that ifdef. Yes, it will be helpful to have a !CONFIG_CLKSRC_OF version of CLOCKSOURCE_OF_DECLARE. And can CONFIG_CLKSRC_OF be enabled by default for all ARM platforms? Thanks, Thomas. ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init @ 2013-01-21 18:30 ` Thomas Abraham 0 siblings, 0 replies; 18+ messages in thread From: Thomas Abraham @ 2013-01-21 18:30 UTC (permalink / raw) To: linux-arm-kernel Hi Stephen, On 21 January 2013 09:21, Stephen Warren <swarren@wwwdotorg.org> wrote: > On 01/20/2013 06:22 PM, Thomas Abraham wrote: >> Add an entry in __clksrc_of_table so that ARMv7 architected timer is >> discoverable using call to clocksource_of_init. > >> diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c > >> +#ifdef CONFIG_CLKSRC_OF >> +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) >> +#endif > > I wonder if we shouldn't enhance include/linux/clocksource.h to define > CLOCKSOURCE_OF_DECLARE even when !CONFIG_CLKSRC_OF; that way, drivers > wouldn't need that ifdef. Yes, it will be helpful to have a !CONFIG_CLKSRC_OF version of CLOCKSOURCE_OF_DECLARE. And can CONFIG_CLKSRC_OF be enabled by default for all ARM platforms? Thanks, Thomas. ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init 2013-01-21 18:30 ` Thomas Abraham @ 2013-01-22 11:04 ` Mark Rutland -1 siblings, 0 replies; 18+ messages in thread From: Mark Rutland @ 2013-01-22 11:04 UTC (permalink / raw) To: Thomas Abraham Cc: swarren, Stephen Warren, johnstul, devicetree-discuss, rob.herring, linux-arm-kernel On Mon, Jan 21, 2013 at 06:30:47PM +0000, Thomas Abraham wrote: > Hi Stephen, > > On 21 January 2013 09:21, Stephen Warren <swarren@wwwdotorg.org> wrote: > > On 01/20/2013 06:22 PM, Thomas Abraham wrote: > >> Add an entry in __clksrc_of_table so that ARMv7 architected timer is > >> discoverable using call to clocksource_of_init. > > > >> diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c > > > >> +#ifdef CONFIG_CLKSRC_OF > >> +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) > >> +#endif > > > > I wonder if we shouldn't enhance include/linux/clocksource.h to define > > CLOCKSOURCE_OF_DECLARE even when !CONFIG_CLKSRC_OF; that way, drivers > > wouldn't need that ifdef. > > Yes, it will be helpful to have a !CONFIG_CLKSRC_OF version of > CLOCKSOURCE_OF_DECLARE. And can CONFIG_CLKSRC_OF be enabled by default > for all ARM platforms? > > Thanks, > Thomas. Thomas, Once the above is worked out, could you send me the updated patch with a pointer to whichever {branch,patch}(es) I need for CLKSRC_OF? Thanks, Mark. ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init @ 2013-01-22 11:04 ` Mark Rutland 0 siblings, 0 replies; 18+ messages in thread From: Mark Rutland @ 2013-01-22 11:04 UTC (permalink / raw) To: linux-arm-kernel On Mon, Jan 21, 2013 at 06:30:47PM +0000, Thomas Abraham wrote: > Hi Stephen, > > On 21 January 2013 09:21, Stephen Warren <swarren@wwwdotorg.org> wrote: > > On 01/20/2013 06:22 PM, Thomas Abraham wrote: > >> Add an entry in __clksrc_of_table so that ARMv7 architected timer is > >> discoverable using call to clocksource_of_init. > > > >> diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c > > > >> +#ifdef CONFIG_CLKSRC_OF > >> +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) > >> +#endif > > > > I wonder if we shouldn't enhance include/linux/clocksource.h to define > > CLOCKSOURCE_OF_DECLARE even when !CONFIG_CLKSRC_OF; that way, drivers > > wouldn't need that ifdef. > > Yes, it will be helpful to have a !CONFIG_CLKSRC_OF version of > CLOCKSOURCE_OF_DECLARE. And can CONFIG_CLKSRC_OF be enabled by default > for all ARM platforms? > > Thanks, > Thomas. Thomas, Once the above is worked out, could you send me the updated patch with a pointer to whichever {branch,patch}(es) I need for CLKSRC_OF? Thanks, Mark. ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init 2013-01-22 11:04 ` Mark Rutland @ 2013-01-30 17:19 ` Mark Rutland -1 siblings, 0 replies; 18+ messages in thread From: Mark Rutland @ 2013-01-30 17:19 UTC (permalink / raw) To: Thomas Abraham Cc: johnstul, devicetree-discuss, swarren, linux-arm-kernel, rob.herring Thomas, On Tue, Jan 22, 2013 at 11:04:56AM +0000, Mark Rutland wrote: > On Mon, Jan 21, 2013 at 06:30:47PM +0000, Thomas Abraham wrote: > > Hi Stephen, > > > > On 21 January 2013 09:21, Stephen Warren <swarren@wwwdotorg.org> wrote: > > > On 01/20/2013 06:22 PM, Thomas Abraham wrote: > > >> Add an entry in __clksrc_of_table so that ARMv7 architected timer is > > >> discoverable using call to clocksource_of_init. > > > > > >> diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c > > > > > >> +#ifdef CONFIG_CLKSRC_OF > > >> +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) > > >> +#endif > > > > > > I wonder if we shouldn't enhance include/linux/clocksource.h to define > > > CLOCKSOURCE_OF_DECLARE even when !CONFIG_CLKSRC_OF; that way, drivers > > > wouldn't need that ifdef. > > > > Yes, it will be helpful to have a !CONFIG_CLKSRC_OF version of > > CLOCKSOURCE_OF_DECLARE. And can CONFIG_CLKSRC_OF be enabled by default > > for all ARM platforms? > > > > Thanks, > > Thomas. > > Thomas, > > Once the above is worked out, could you send me the updated patch with a > pointer to whichever {branch,patch}(es) I need for CLKSRC_OF? Is there any news on this? I'd like to be able to push out a branch ready for merging soon. I've also realised that platforms using of_clocksource_init won't get the architected timer registered as a sched_clock, which would seem like a loss. I'm not sure what the relative requirements are for sched_clock and clocksource, so there may be some platforms that want it and some that don't. I see for exynos5440 the architected timer is regsitered as a clock{source,_event_device}, but not as a sched_clock, and omap5 is similar, but may register dmtimer_read_sched_clock first. We can probably leave it out for now, but it's something we'll need to consider in future. Thanks, Mark. ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init @ 2013-01-30 17:19 ` Mark Rutland 0 siblings, 0 replies; 18+ messages in thread From: Mark Rutland @ 2013-01-30 17:19 UTC (permalink / raw) To: linux-arm-kernel Thomas, On Tue, Jan 22, 2013 at 11:04:56AM +0000, Mark Rutland wrote: > On Mon, Jan 21, 2013 at 06:30:47PM +0000, Thomas Abraham wrote: > > Hi Stephen, > > > > On 21 January 2013 09:21, Stephen Warren <swarren@wwwdotorg.org> wrote: > > > On 01/20/2013 06:22 PM, Thomas Abraham wrote: > > >> Add an entry in __clksrc_of_table so that ARMv7 architected timer is > > >> discoverable using call to clocksource_of_init. > > > > > >> diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c > > > > > >> +#ifdef CONFIG_CLKSRC_OF > > >> +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) > > >> +#endif > > > > > > I wonder if we shouldn't enhance include/linux/clocksource.h to define > > > CLOCKSOURCE_OF_DECLARE even when !CONFIG_CLKSRC_OF; that way, drivers > > > wouldn't need that ifdef. > > > > Yes, it will be helpful to have a !CONFIG_CLKSRC_OF version of > > CLOCKSOURCE_OF_DECLARE. And can CONFIG_CLKSRC_OF be enabled by default > > for all ARM platforms? > > > > Thanks, > > Thomas. > > Thomas, > > Once the above is worked out, could you send me the updated patch with a > pointer to whichever {branch,patch}(es) I need for CLKSRC_OF? Is there any news on this? I'd like to be able to push out a branch ready for merging soon. I've also realised that platforms using of_clocksource_init won't get the architected timer registered as a sched_clock, which would seem like a loss. I'm not sure what the relative requirements are for sched_clock and clocksource, so there may be some platforms that want it and some that don't. I see for exynos5440 the architected timer is regsitered as a clock{source,_event_device}, but not as a sched_clock, and omap5 is similar, but may register dmtimer_read_sched_clock first. We can probably leave it out for now, but it's something we'll need to consider in future. Thanks, Mark. ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init 2013-01-30 17:19 ` Mark Rutland @ 2013-01-30 17:49 ` Stephen Warren -1 siblings, 0 replies; 18+ messages in thread From: Stephen Warren @ 2013-01-30 17:49 UTC (permalink / raw) To: Mark Rutland Cc: swarren, johnstul, devicetree-discuss, rob.herring, Thomas Abraham, linux-arm-kernel On 01/30/2013 10:19 AM, Mark Rutland wrote: > Thomas, > > On Tue, Jan 22, 2013 at 11:04:56AM +0000, Mark Rutland wrote: >> On Mon, Jan 21, 2013 at 06:30:47PM +0000, Thomas Abraham wrote: >>> Hi Stephen, >>> >>> On 21 January 2013 09:21, Stephen Warren <swarren@wwwdotorg.org> wrote: >>>> On 01/20/2013 06:22 PM, Thomas Abraham wrote: >>>>> Add an entry in __clksrc_of_table so that ARMv7 architected timer is >>>>> discoverable using call to clocksource_of_init. >>>> >>>>> diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c >>>> >>>>> +#ifdef CONFIG_CLKSRC_OF >>>>> +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) >>>>> +#endif >>>> >>>> I wonder if we shouldn't enhance include/linux/clocksource.h to define >>>> CLOCKSOURCE_OF_DECLARE even when !CONFIG_CLKSRC_OF; that way, drivers >>>> wouldn't need that ifdef. >>> >>> Yes, it will be helpful to have a !CONFIG_CLKSRC_OF version of >>> CLOCKSOURCE_OF_DECLARE. And can CONFIG_CLKSRC_OF be enabled by default >>> for all ARM platforms? >>> >>> Thanks, >>> Thomas. >> >> Thomas, >> >> Once the above is worked out, could you send me the updated patch with a >> pointer to whichever {branch,patch}(es) I need for CLKSRC_OF? > > Is there any news on this? I'd like to be able to push out a branch ready for > merging soon. I just sent a patch for this. ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init @ 2013-01-30 17:49 ` Stephen Warren 0 siblings, 0 replies; 18+ messages in thread From: Stephen Warren @ 2013-01-30 17:49 UTC (permalink / raw) To: linux-arm-kernel On 01/30/2013 10:19 AM, Mark Rutland wrote: > Thomas, > > On Tue, Jan 22, 2013 at 11:04:56AM +0000, Mark Rutland wrote: >> On Mon, Jan 21, 2013 at 06:30:47PM +0000, Thomas Abraham wrote: >>> Hi Stephen, >>> >>> On 21 January 2013 09:21, Stephen Warren <swarren@wwwdotorg.org> wrote: >>>> On 01/20/2013 06:22 PM, Thomas Abraham wrote: >>>>> Add an entry in __clksrc_of_table so that ARMv7 architected timer is >>>>> discoverable using call to clocksource_of_init. >>>> >>>>> diff --git a/arch/arm/kernel/arch_timer.c b/arch/arm/kernel/arch_timer.c >>>> >>>>> +#ifdef CONFIG_CLKSRC_OF >>>>> +CLOCKSOURCE_OF_DECLARE(armv7_timer, "arm,armv7-timer", arch_timer_of_register) >>>>> +#endif >>>> >>>> I wonder if we shouldn't enhance include/linux/clocksource.h to define >>>> CLOCKSOURCE_OF_DECLARE even when !CONFIG_CLKSRC_OF; that way, drivers >>>> wouldn't need that ifdef. >>> >>> Yes, it will be helpful to have a !CONFIG_CLKSRC_OF version of >>> CLOCKSOURCE_OF_DECLARE. And can CONFIG_CLKSRC_OF be enabled by default >>> for all ARM platforms? >>> >>> Thanks, >>> Thomas. >> >> Thomas, >> >> Once the above is worked out, could you send me the updated patch with a >> pointer to whichever {branch,patch}(es) I need for CLKSRC_OF? > > Is there any news on this? I'd like to be able to push out a branch ready for > merging soon. I just sent a patch for this. ^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2013-01-30 17:49 UTC | newest] Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2013-01-21 1:22 [PATCH] ARM: architected timers: allow dt based discovery using clocksource_of_init Thomas Abraham 2013-01-21 1:22 ` Thomas Abraham 2013-01-21 12:13 ` Mark Rutland 2013-01-21 12:13 ` Mark Rutland [not found] ` <20130121121353.GH15707-NuALmloUBlrZROr8t4l/smS4ubULX0JqMm0uRHvK7Nw@public.gmane.org> 2013-01-21 18:19 ` Thomas Abraham 2013-01-21 18:19 ` Thomas Abraham [not found] ` <1358731326-2979-1-git-send-email-thomas.abraham-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2013-01-21 17:21 ` Stephen Warren 2013-01-21 17:21 ` Stephen Warren 2013-01-21 18:25 ` Marc Zyngier 2013-01-21 18:25 ` Marc Zyngier [not found] ` <50FD7904.1070201-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> 2013-01-21 18:30 ` Thomas Abraham 2013-01-21 18:30 ` Thomas Abraham 2013-01-22 11:04 ` Mark Rutland 2013-01-22 11:04 ` Mark Rutland 2013-01-30 17:19 ` Mark Rutland 2013-01-30 17:19 ` Mark Rutland 2013-01-30 17:49 ` Stephen Warren 2013-01-30 17:49 ` Stephen Warren
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.