* [GIT PULL] Qualcomm Driver Updates for v4.7
@ 2016-04-01 19:59 Andy Gross
2016-04-13 18:10 ` Olof Johansson
0 siblings, 1 reply; 6+ messages in thread
From: Andy Gross @ 2016-04-01 19:59 UTC (permalink / raw)
To: linux-arm-kernel
The following changes since commit f55532a0c0b8bb6148f4e07853b876ef73bc69ca:
Linux 4.6-rc1 (2016-03-26 16:03:24 -0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/agross/linux.git tags/qcom-drivers-for-4.7
for you to fetch changes up to d24a7dab5b8fa6717d7a3917a634b561dcee312f:
ARM: qcom: Drop ARCH_MSM* configs (2016-03-31 17:37:31 -0500)
----------------------------------------------------------------
Qualcomm ARM Based Driver Updates for v4.7
* Drop ARCH_MSM* configs and fixup clocksource support
----------------------------------------------------------------
Stephen Boyd (2):
ARM: qcom: Make an option for qcom 'a-family' platforms
ARM: qcom: Drop ARCH_MSM* configs
arch/arm/Makefile | 3 +--
arch/arm/mach-qcom/Kconfig | 17 +++++++++--------
2 files changed, 10 insertions(+), 10 deletions(-)
^ permalink raw reply [flat|nested] 6+ messages in thread* [GIT PULL] Qualcomm Driver Updates for v4.7
2016-04-01 19:59 [GIT PULL] Qualcomm Driver Updates for v4.7 Andy Gross
@ 2016-04-13 18:10 ` Olof Johansson
2016-04-13 18:21 ` Stephen Boyd
0 siblings, 1 reply; 6+ messages in thread
From: Olof Johansson @ 2016-04-13 18:10 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Apr 01, 2016 at 02:59:36PM -0500, Andy Gross wrote:
> The following changes since commit f55532a0c0b8bb6148f4e07853b876ef73bc69ca:
>
> Linux 4.6-rc1 (2016-03-26 16:03:24 -0700)
>
> are available in the git repository at:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/agross/linux.git tags/qcom-drivers-for-4.7
Are you sure? Last branch was all drivers/soc contents, which _I_ merged
into next/drivers. This is all arch/arm contents, which _I_ would merge
into next/soc. :)
> for you to fetch changes up to d24a7dab5b8fa6717d7a3917a634b561dcee312f:
>
> ARM: qcom: Drop ARCH_MSM* configs (2016-03-31 17:37:31 -0500)
>
> ----------------------------------------------------------------
> Qualcomm ARM Based Driver Updates for v4.7
>
> * Drop ARCH_MSM* configs and fixup clocksource support
>
> ----------------------------------------------------------------
> Stephen Boyd (2):
> ARM: qcom: Make an option for qcom 'a-family' platforms
> ARM: qcom: Drop ARCH_MSM* configs
THis does:
-config ARCH_MSM8974
- bool "Enable support for MSM8974"
- select HAVE_ARM_ARCH_TIMER
... and I don't see the HAVE_ARCH_ARM_TIMER added anywhere. So it looks
like you might need to pick it for ARCH_QCOM_A_FAMILY now?
-Olof
^ permalink raw reply [flat|nested] 6+ messages in thread
* [GIT PULL] Qualcomm Driver Updates for v4.7
2016-04-13 18:10 ` Olof Johansson
@ 2016-04-13 18:21 ` Stephen Boyd
2016-04-13 18:41 ` Olof Johansson
0 siblings, 1 reply; 6+ messages in thread
From: Stephen Boyd @ 2016-04-13 18:21 UTC (permalink / raw)
To: linux-arm-kernel
On 04/13, Olof Johansson wrote:
> THis does:
>
> -config ARCH_MSM8974
> - bool "Enable support for MSM8974"
> - select HAVE_ARM_ARCH_TIMER
>
>
> ... and I don't see the HAVE_ARCH_ARM_TIMER added anywhere. So it looks
> like you might need to pick it for ARCH_QCOM_A_FAMILY now?
>
That was part of the original series[1]. Splitting up patches
into topic branches is problematic here it seems.
[1] http://lkml.kernel.org/r/1448485478-21699-3-git-send-email-sboyd at codeaurora.org
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
^ permalink raw reply [flat|nested] 6+ messages in thread
* [GIT PULL] Qualcomm Driver Updates for v4.7
2016-04-13 18:21 ` Stephen Boyd
@ 2016-04-13 18:41 ` Olof Johansson
2016-04-13 19:11 ` Stephen Boyd
0 siblings, 1 reply; 6+ messages in thread
From: Olof Johansson @ 2016-04-13 18:41 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Apr 13, 2016 at 11:21:56AM -0700, Stephen Boyd wrote:
> On 04/13, Olof Johansson wrote:
> > THis does:
> >
> > -config ARCH_MSM8974
> > - bool "Enable support for MSM8974"
> > - select HAVE_ARM_ARCH_TIMER
> >
> >
> > ... and I don't see the HAVE_ARCH_ARM_TIMER added anywhere. So it looks
> > like you might need to pick it for ARCH_QCOM_A_FAMILY now?
> >
>
> That was part of the original series[1]. Splitting up patches
> into topic branches is problematic here it seems.
>
> [1] http://lkml.kernel.org/r/1448485478-21699-3-git-send-email-sboyd at codeaurora.org
The HAVE.* options are usually expected to be set through selects, not
necessarily driven by a user choosing them. So I think adding a select here is
the right way to do it.
HAVE.* options are usually used for dependencies for the user-exposed options.
I.e. only ask if you want arch timers if HAVE_ARM_ARCH_TIMERS is set, etc.
-Olof
^ permalink raw reply [flat|nested] 6+ messages in thread
* [GIT PULL] Qualcomm Driver Updates for v4.7
2016-04-13 18:41 ` Olof Johansson
@ 2016-04-13 19:11 ` Stephen Boyd
2016-04-16 19:59 ` Arnd Bergmann
0 siblings, 1 reply; 6+ messages in thread
From: Stephen Boyd @ 2016-04-13 19:11 UTC (permalink / raw)
To: linux-arm-kernel
On 04/13, Olof Johansson wrote:
> On Wed, Apr 13, 2016 at 11:21:56AM -0700, Stephen Boyd wrote:
> > That was part of the original series[1]. Splitting up patches
> > into topic branches is problematic here it seems.
> >
> > [1] http://lkml.kernel.org/r/1448485478-21699-3-git-send-email-sboyd at codeaurora.org
>
> The HAVE.* options are usually expected to be set through selects, not
> necessarily driven by a user choosing them. So I think adding a select here is
> the right way to do it.
>
> HAVE.* options are usually used for dependencies for the user-exposed options.
> I.e. only ask if you want arch timers if HAVE_ARM_ARCH_TIMERS is set, etc.
>
Agreed. Perhaps the name is bad? I don't know the history of the
Kconfig, but HAVE_ARM_ARCH_TIMER is a user visible option, so
selecting it in Kconfig language doesn't make much sense to me.
Some other platforms are following the same design where it's
part of the defconfig while others are selecting it from Kconfig.
Mass confusion has set in.
If we want to change the design to be selected by all platforms
then we'll need to make HAVE_ARM_ARCH_TIMER into a hidden Kconfig
option so that users can't turn it off. In fact, we may want to
just obliterate the Kconfig entirely and have users select
ARM_ARCH_TIMER directly. Everything is multi-platform now, right?
If so then GENERIC_CLOCKEVENTS is selected all the time on CPU_V7
and the CPU_V7 dependency is not helping much.
Honestly, I view these clocksource selects as the only blocker in
the effort to get rid of machine type Kconfigs. Maybe that isn't
a good goal though.
--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
^ permalink raw reply [flat|nested] 6+ messages in thread
* [GIT PULL] Qualcomm Driver Updates for v4.7
2016-04-13 19:11 ` Stephen Boyd
@ 2016-04-16 19:59 ` Arnd Bergmann
0 siblings, 0 replies; 6+ messages in thread
From: Arnd Bergmann @ 2016-04-16 19:59 UTC (permalink / raw)
To: linux-arm-kernel
On Wednesday 13 April 2016 12:11:08 Stephen Boyd wrote:
>
> Honestly, I view these clocksource selects as the only blocker in
> the effort to get rid of machine type Kconfigs. Maybe that isn't
> a good goal though.
I agree with you, it would be nice if we could just make them
all configurable and not need any machine type Kconfigs (though
a lot of platforms will continue using them for other reasons).
A patch like the one below (as I think you suggested) would be
nice too, but requires people to update their defconfig files.
In the mainline kernel, we have four defconfig files that
enable CONFIG_HAVE_ARM_ARCH_TIMER manually, so those all need
to be changed along with the Kconfig files, and everyone that
has their own out of tree defconfig file (not a full .config)
runs into the same issue.
It's probably still worth doing the change, but it's not obvious.
Arnd
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 62c209504c0a..0daad62cfce5 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -1296,7 +1296,6 @@ config HAVE_ARM_SCU
This option enables support for the ARM system coherency unit
config HAVE_ARM_ARCH_TIMER
- bool "Architected timer support"
depends on CPU_V7
select ARM_ARCH_TIMER
select GENERIC_CLOCKEVENTS
diff --git a/drivers/clocksource/Kconfig b/drivers/clocksource/Kconfig
index c346be650892..0314778c992f 100644
--- a/drivers/clocksource/Kconfig
+++ b/drivers/clocksource/Kconfig
@@ -187,7 +187,7 @@ config CLKSRC_STM32
select CLKSRC_MMIO
config ARM_ARCH_TIMER
- bool
+ bool "Architected timer support"
select CLKSRC_OF if OF
select CLKSRC_ACPI if ACPI
^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-04-16 19:59 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-04-01 19:59 [GIT PULL] Qualcomm Driver Updates for v4.7 Andy Gross
2016-04-13 18:10 ` Olof Johansson
2016-04-13 18:21 ` Stephen Boyd
2016-04-13 18:41 ` Olof Johansson
2016-04-13 19:11 ` Stephen Boyd
2016-04-16 19:59 ` Arnd Bergmann
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).