All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/2] arm: move HAVE_ARM_ARCH_TIMER selects into multi_v7 platforms
@ 2013-06-17  9:45 Mark Rutland
  2013-06-17  9:45 ` [PATCH 1/2] arm: vexpress: select HAVE_ARM_ARCH_TIMER Mark Rutland
  2013-06-17  9:45 ` [PATCH 2/2] arm: highbank: " Mark Rutland
  0 siblings, 2 replies; 6+ messages in thread
From: Mark Rutland @ 2013-06-17  9:45 UTC (permalink / raw)
  To: linux-arm-kernel

The multi_v7_defconfig used to select ARM_ARCH_TIMER, but was broken by
8a4da6e36c: "arm: arch_timer: move core to drivers/clocksource". Since
then some platforms haven't been getting the architected timer support
they want when building from a multi_v7_defconfig.

As not all platforms require architected timer support, rather than
selecting it in the multi_v7_defconfig, this patch adds selects to the
platforms requiring support.

Thanks,
Mark.

Mark Rutland (2):
  arm: vexpress: select HAVE_ARM_ARCH_TIMER
  arm: highbank: select HAVE_ARM_ARCH_TIMER

 arch/arm/mach-highbank/Kconfig | 1 +
 arch/arm/mach-vexpress/Kconfig | 1 +
 2 files changed, 2 insertions(+)

-- 
1.8.1.1

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/2] arm: vexpress: select HAVE_ARM_ARCH_TIMER
  2013-06-17  9:45 [PATCH 0/2] arm: move HAVE_ARM_ARCH_TIMER selects into multi_v7 platforms Mark Rutland
@ 2013-06-17  9:45 ` Mark Rutland
  2013-06-17 13:48   ` Pawel Moll
  2013-06-17  9:45 ` [PATCH 2/2] arm: highbank: " Mark Rutland
  1 sibling, 1 reply; 6+ messages in thread
From: Mark Rutland @ 2013-06-17  9:45 UTC (permalink / raw)
  To: linux-arm-kernel

When building a ARCH_MULTI_V7 kernel, support for the architected timers
won't be enabled out-of-the-box even when ARCH_VEXPRESS is selected,
which is unexpected and disappointing when using a coretile with
architected timers.

Fix this up by having ARCH_VEXPRESS select HAVE_ARM_ARCH_TIMER.

Signed-off-by: Mark Rutland <mark.rutland@arm.com>
Cc: Pawel Moll <pawel.moll@arm.com>
---
 arch/arm/mach-vexpress/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/mach-vexpress/Kconfig b/arch/arm/mach-vexpress/Kconfig
index 5907e10..e0f4974 100644
--- a/arch/arm/mach-vexpress/Kconfig
+++ b/arch/arm/mach-vexpress/Kconfig
@@ -9,6 +9,7 @@ config ARCH_VEXPRESS
 	select COMMON_CLK_VERSATILE
 	select CPU_V7
 	select GENERIC_CLOCKEVENTS
+	select HAVE_ARM_ARCH_TIMER
 	select HAVE_ARM_SCU if SMP
 	select HAVE_ARM_TWD if LOCAL_TIMERS
 	select HAVE_CLK
-- 
1.8.1.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 2/2] arm: highbank: select HAVE_ARM_ARCH_TIMER
  2013-06-17  9:45 [PATCH 0/2] arm: move HAVE_ARM_ARCH_TIMER selects into multi_v7 platforms Mark Rutland
  2013-06-17  9:45 ` [PATCH 1/2] arm: vexpress: select HAVE_ARM_ARCH_TIMER Mark Rutland
@ 2013-06-17  9:45 ` Mark Rutland
  2013-06-17 13:45   ` Rob Herring
  1 sibling, 1 reply; 6+ messages in thread
From: Mark Rutland @ 2013-06-17  9:45 UTC (permalink / raw)
  To: linux-arm-kernel

The ECX-2000 has architected timers, but support for them is not
selected by default.

Make MACH_HIGHBANK select HAVE_ARM_ARCH_TIMER, to ensure arhcited timer
support is enabled by default.

Signed-off-by: Mark Rutland <mark.rutland@arm.com>
Cc: Rob Herring <rob.herring@calxeda.com>
---
 arch/arm/mach-highbank/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/mach-highbank/Kconfig b/arch/arm/mach-highbank/Kconfig
index cd9fcb1..2bb896f 100644
--- a/arch/arm/mach-highbank/Kconfig
+++ b/arch/arm/mach-highbank/Kconfig
@@ -11,6 +11,7 @@ config ARCH_HIGHBANK
 	select COMMON_CLK
 	select CPU_V7
 	select GENERIC_CLOCKEVENTS
+	select HAVE_ARM_ARCH_TIMER
 	select HAVE_ARM_SCU
 	select HAVE_ARM_TWD if LOCAL_TIMERS
 	select HAVE_SMP
-- 
1.8.1.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 2/2] arm: highbank: select HAVE_ARM_ARCH_TIMER
  2013-06-17  9:45 ` [PATCH 2/2] arm: highbank: " Mark Rutland
@ 2013-06-17 13:45   ` Rob Herring
  0 siblings, 0 replies; 6+ messages in thread
From: Rob Herring @ 2013-06-17 13:45 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Jun 17, 2013 at 9:45 AM, Mark Rutland <mark.rutland@arm.com> wrote:
> The ECX-2000 has architected timers, but support for them is not
> selected by default.
>
> Make MACH_HIGHBANK select HAVE_ARM_ARCH_TIMER, to ensure arhcited timer
> support is enabled by default.
>
> Signed-off-by: Mark Rutland <mark.rutland@arm.com>
> Cc: Rob Herring <rob.herring@calxeda.com>

Acked-by: Rob Herring <rob.herring@calxeda.com>

> ---
>  arch/arm/mach-highbank/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/mach-highbank/Kconfig b/arch/arm/mach-highbank/Kconfig
> index cd9fcb1..2bb896f 100644
> --- a/arch/arm/mach-highbank/Kconfig
> +++ b/arch/arm/mach-highbank/Kconfig
> @@ -11,6 +11,7 @@ config ARCH_HIGHBANK
>         select COMMON_CLK
>         select CPU_V7
>         select GENERIC_CLOCKEVENTS
> +       select HAVE_ARM_ARCH_TIMER
>         select HAVE_ARM_SCU
>         select HAVE_ARM_TWD if LOCAL_TIMERS
>         select HAVE_SMP
> --
> 1.8.1.1
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/2] arm: vexpress: select HAVE_ARM_ARCH_TIMER
  2013-06-17  9:45 ` [PATCH 1/2] arm: vexpress: select HAVE_ARM_ARCH_TIMER Mark Rutland
@ 2013-06-17 13:48   ` Pawel Moll
  2013-06-17 14:02     ` Mark Rutland
  0 siblings, 1 reply; 6+ messages in thread
From: Pawel Moll @ 2013-06-17 13:48 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, 2013-06-17 at 10:45 +0100, Mark Rutland wrote:
> When building a ARCH_MULTI_V7 kernel, support for the architected timers
> won't be enabled out-of-the-box even when ARCH_VEXPRESS is selected,
> which is unexpected and disappointing when using a coretile with
> architected timers.
> 
> Fix this up by having ARCH_VEXPRESS select HAVE_ARM_ARCH_TIMER.
> 
> Signed-off-by: Mark Rutland <mark.rutland@arm.com>
> Cc: Pawel Moll <pawel.moll@arm.com>
> ---
>  arch/arm/mach-vexpress/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/mach-vexpress/Kconfig b/arch/arm/mach-vexpress/Kconfig
> index 5907e10..e0f4974 100644
> --- a/arch/arm/mach-vexpress/Kconfig
> +++ b/arch/arm/mach-vexpress/Kconfig
> @@ -9,6 +9,7 @@ config ARCH_VEXPRESS
>  	select COMMON_CLK_VERSATILE
>  	select CPU_V7
>  	select GENERIC_CLOCKEVENTS
> +	select HAVE_ARM_ARCH_TIMER
>  	select HAVE_ARM_SCU if SMP
>  	select HAVE_ARM_TWD if LOCAL_TIMERS
>  	select HAVE_CLK

Acked-by: Pawel Moll <pawel.moll@arm.com>

Do you want me to queue it for 3.12?

Pawe?

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/2] arm: vexpress: select HAVE_ARM_ARCH_TIMER
  2013-06-17 13:48   ` Pawel Moll
@ 2013-06-17 14:02     ` Mark Rutland
  0 siblings, 0 replies; 6+ messages in thread
From: Mark Rutland @ 2013-06-17 14:02 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Jun 17, 2013 at 02:48:05PM +0100, Pawel Moll wrote:
> On Mon, 2013-06-17 at 10:45 +0100, Mark Rutland wrote:
> > When building a ARCH_MULTI_V7 kernel, support for the architected timers
> > won't be enabled out-of-the-box even when ARCH_VEXPRESS is selected,
> > which is unexpected and disappointing when using a coretile with
> > architected timers.
> > 
> > Fix this up by having ARCH_VEXPRESS select HAVE_ARM_ARCH_TIMER.
> > 
> > Signed-off-by: Mark Rutland <mark.rutland@arm.com>
> > Cc: Pawel Moll <pawel.moll@arm.com>
> > ---
> >  arch/arm/mach-vexpress/Kconfig | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/arch/arm/mach-vexpress/Kconfig b/arch/arm/mach-vexpress/Kconfig
> > index 5907e10..e0f4974 100644
> > --- a/arch/arm/mach-vexpress/Kconfig
> > +++ b/arch/arm/mach-vexpress/Kconfig
> > @@ -9,6 +9,7 @@ config ARCH_VEXPRESS
> >  	select COMMON_CLK_VERSATILE
> >  	select CPU_V7
> >  	select GENERIC_CLOCKEVENTS
> > +	select HAVE_ARM_ARCH_TIMER
> >  	select HAVE_ARM_SCU if SMP
> >  	select HAVE_ARM_TWD if LOCAL_TIMERS
> >  	select HAVE_CLK
> 
> Acked-by: Pawel Moll <pawel.moll@arm.com>
> 
> Do you want me to queue it for 3.12?

That would be great, yes please :)

Thanks,
Mark.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2013-06-17 14:02 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-17  9:45 [PATCH 0/2] arm: move HAVE_ARM_ARCH_TIMER selects into multi_v7 platforms Mark Rutland
2013-06-17  9:45 ` [PATCH 1/2] arm: vexpress: select HAVE_ARM_ARCH_TIMER Mark Rutland
2013-06-17 13:48   ` Pawel Moll
2013-06-17 14:02     ` Mark Rutland
2013-06-17  9:45 ` [PATCH 2/2] arm: highbank: " Mark Rutland
2013-06-17 13:45   ` Rob Herring

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.