Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1280460 > unrolled thread

[PATCH v3 0/3] Remove ARCH_MSM* configs

Started byStephen Boyd <sboyd@codeaurora.org>
First post2015-12-01 02:30 +0100
Last post2015-12-01 13:20 +0100
Articles 6 — 3 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH v3 0/3] Remove ARCH_MSM* configs Stephen Boyd <sboyd@codeaurora.org> - 2015-12-01 02:30 +0100
    [PATCH v3 3/3] ARM: qcom: Drop ARCH_MSM* configs Stephen Boyd <sboyd@codeaurora.org> - 2015-12-01 02:30 +0100
    [PATCH v3 2/3] ARM: qcom_defconfig: Enable HAVE_ARM_ARCH_TIMER Stephen Boyd <sboyd@codeaurora.org> - 2015-12-01 02:30 +0100
    [PATCH v3 1/3] ARM: qcom: Make an option for qcom 'a-family' platforms Stephen Boyd <sboyd@codeaurora.org> - 2015-12-01 02:30 +0100
    Re: [PATCH v3 0/3] Remove ARCH_MSM* configs Arnd Bergmann <arnd@arndb.de> - 2015-12-01 13:00 +0100
    Re: [PATCH v3 0/3] Remove ARCH_MSM* configs Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-12-01 13:20 +0100

#1280460 — [PATCH v3 0/3] Remove ARCH_MSM* configs

FromStephen Boyd <sboyd@codeaurora.org>
Date2015-12-01 02:30 +0100
Subject[PATCH v3 0/3] Remove ARCH_MSM* configs
Message-ID<qAHVv-2Vf-1@gated-at.bofh.it>
This patch series allows us to remove the ARCH_MSM* configs that live
in mach-qcom/Kconfig. They're mostly proxy configs for user selectable
clocksource configurations anyway.

Changes from v2:
 * Renamed config in patch 1 to QCOM_ARCH_A_FAMILY to be less confusing

Changes from v1:
 * Patch 1 replaced with one that moves config to mach-qcom/Kconfig
 * Applied Arnd's acks to patches 2 and 3

Stephen Boyd (3):
  ARM: qcom: Make an option for qcom 'a-family' platforms
  ARM: qcom_defconfig: Enable HAVE_ARM_ARCH_TIMER
  ARM: qcom: Drop ARCH_MSM* configs

 arch/arm/Makefile               |  3 +--
 arch/arm/configs/qcom_defconfig |  1 +
 arch/arm/mach-qcom/Kconfig      | 17 +++++++++--------
 3 files changed, 11 insertions(+), 10 deletions(-)

-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1280461 — [PATCH v3 3/3] ARM: qcom: Drop ARCH_MSM* configs

FromStephen Boyd <sboyd@codeaurora.org>
Date2015-12-01 02:30 +0100
Subject[PATCH v3 3/3] ARM: qcom: Drop ARCH_MSM* configs
Message-ID<qAHVw-2Vf-19@gated-at.bofh.it>
In reply to#1280460
Drop these configs now that we select the clocksources we need
via the defconfig.

Acked-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
---
 arch/arm/mach-qcom/Kconfig | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/arch/arm/mach-qcom/Kconfig b/arch/arm/mach-qcom/Kconfig
index 230c7e8a32fb..9cd3b50d891f 100644
--- a/arch/arm/mach-qcom/Kconfig
+++ b/arch/arm/mach-qcom/Kconfig
@@ -23,16 +23,4 @@ config ARCH_QCOM_A_FAMILY
 	  B-family includes all Snapdragon 2xx/4xx/6xx/8xx models starting
 	  in 2013 with the MSM8x74 SoC.
 
-config ARCH_MSM8X60
-	bool "Enable support for MSM8X60"
-	select CLKSRC_QCOM
-
-config ARCH_MSM8960
-	bool "Enable support for MSM8960"
-	select CLKSRC_QCOM
-
-config ARCH_MSM8974
-	bool "Enable support for MSM8974"
-	select HAVE_ARM_ARCH_TIMER
-
 endif
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1280463 — [PATCH v3 2/3] ARM: qcom_defconfig: Enable HAVE_ARM_ARCH_TIMER

FromStephen Boyd <sboyd@codeaurora.org>
Date2015-12-01 02:30 +0100
Subject[PATCH v3 2/3] ARM: qcom_defconfig: Enable HAVE_ARM_ARCH_TIMER
Message-ID<qAHVw-2Vf-15@gated-at.bofh.it>
In reply to#1280460
Enable this config in the defconfig so that we can delete the
selection logic surrounding it that exists in mach-qcom/Kconfig.

Acked-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
---
 arch/arm/configs/qcom_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/qcom_defconfig b/arch/arm/configs/qcom_defconfig
index ee54a706e8a3..6f34da7afd75 100644
--- a/arch/arm/configs/qcom_defconfig
+++ b/arch/arm/configs/qcom_defconfig
@@ -22,6 +22,7 @@ CONFIG_ARCH_MSM8X60=y
 CONFIG_ARCH_MSM8960=y
 CONFIG_ARCH_MSM8974=y
 CONFIG_SMP=y
+CONFIG_HAVE_ARM_ARCH_TIMER=y
 CONFIG_PREEMPT=y
 CONFIG_AEABI=y
 CONFIG_HIGHMEM=y
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1280464 — [PATCH v3 1/3] ARM: qcom: Make an option for qcom 'a-family' platforms

FromStephen Boyd <sboyd@codeaurora.org>
Date2015-12-01 02:30 +0100
Subject[PATCH v3 1/3] ARM: qcom: Make an option for qcom 'a-family' platforms
Message-ID<qAHVw-2Vf-21@gated-at.bofh.it>
In reply to#1280460
We want to remove the ARCH_MSM* configs in mach-qcom/Kconfig
because they are proxy configs for selecting the right
clocksource driver and adjusting the text offset and there is no
difference between ARCH_MSM8X60 and ARCH_MSM8960 anymore.
Furthermore, there aren't configs for all the SoCs supported by
the mainline kernel, leading to confusion about which SoCs are
supported.

So, add a new config that covers all the platforms that use the
qcom clocksource driver and need the 2MB text offset adjustment
and make it def_bool y so that the defconfigs keep working
properly.

Cc: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
---
 arch/arm/Makefile          |  3 +--
 arch/arm/mach-qcom/Kconfig | 13 +++++++++++++
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/arch/arm/Makefile b/arch/arm/Makefile
index 2c2b28ee4811..dd5799dbcf26 100644
--- a/arch/arm/Makefile
+++ b/arch/arm/Makefile
@@ -148,8 +148,7 @@ textofs-$(CONFIG_PM_H1940)      := 0x00108000
 ifeq ($(CONFIG_ARCH_SA1100),y)
 textofs-$(CONFIG_SA1111) := 0x00208000
 endif
-textofs-$(CONFIG_ARCH_MSM8X60) := 0x00208000
-textofs-$(CONFIG_ARCH_MSM8960) := 0x00208000
+textofs-$(CONFIG_ARCH_QCOM_A_FAMILY) := 0x00208000
 textofs-$(CONFIG_ARCH_AXXIA) := 0x00308000
 
 # Machine directory name.  This list is sorted alphanumerically
diff --git a/arch/arm/mach-qcom/Kconfig b/arch/arm/mach-qcom/Kconfig
index 2256cd1e25d1..230c7e8a32fb 100644
--- a/arch/arm/mach-qcom/Kconfig
+++ b/arch/arm/mach-qcom/Kconfig
@@ -10,6 +10,19 @@ menuconfig ARCH_QCOM
 
 if ARCH_QCOM
 
+config ARCH_QCOM_A_FAMILY
+	bool "Support a-family chipsets (msm8660, msm8960, apq8064)"
+	default y
+	select CLKSRC_QCOM
+	help
+	  Select this option if you want to support a-family platforms.
+
+	  A-family includes all Snapdragon S1/S2/S3/S4 chips before 2013,
+	  up to the MSM8x60 and APQ8064 SoCs.
+
+	  B-family includes all Snapdragon 2xx/4xx/6xx/8xx models starting
+	  in 2013 with the MSM8x74 SoC.
+
 config ARCH_MSM8X60
 	bool "Enable support for MSM8X60"
 	select CLKSRC_QCOM
-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1280804

FromArnd Bergmann <arnd@arndb.de>
Date2015-12-01 13:00 +0100
Message-ID<qARLb-Ht-1@gated-at.bofh.it>
In reply to#1280460
On Monday 30 November 2015 17:25:20 Stephen Boyd wrote:
> This patch series allows us to remove the ARCH_MSM* configs that live
> in mach-qcom/Kconfig. They're mostly proxy configs for user selectable
> clocksource configurations anyway.
> 
> 

Acked-by: Arnd Bergmann <arnd@arndb.de>

I suppose we could have a QCOM_B_FAMILY option as well, but it looks
like we don't need that. Maybe mention in the help text that B_FAMILY
is always supported.

	Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1280817

FromDaniel Lezcano <daniel.lezcano@linaro.org>
Date2015-12-01 13:20 +0100
Message-ID<qAS4y-145-3@gated-at.bofh.it>
In reply to#1280460
On 12/01/2015 02:25 AM, Stephen Boyd wrote:
> This patch series allows us to remove the ARCH_MSM* configs that live
> in mach-qcom/Kconfig. They're mostly proxy configs for user selectable
> clocksource configurations anyway.
>
> Changes from v2:
>   * Renamed config in patch 1 to QCOM_ARCH_A_FAMILY to be less confusing
>
> Changes from v1:
>   * Patch 1 replaced with one that moves config to mach-qcom/Kconfig
>   * Applied Arnd's acks to patches 2 and 3
>
> Stephen Boyd (3):
>    ARM: qcom: Make an option for qcom 'a-family' platforms
>    ARM: qcom_defconfig: Enable HAVE_ARM_ARCH_TIMER
>    ARM: qcom: Drop ARCH_MSM* configs
>
>   arch/arm/Makefile               |  3 +--
>   arch/arm/configs/qcom_defconfig |  1 +
>   arch/arm/mach-qcom/Kconfig      | 17 +++++++++--------
>   3 files changed, 11 insertions(+), 10 deletions(-)

Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org>

Even it does not hurt, ARCH_MSM8X60, ARCH_MSM8960 and ARCH_MSM8974 could 
be also removed from the defconfig. Better to do it right now, so they 
won't be forgotten.


-- 
  <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web