Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1276940 > unrolled thread
| Started by | Stephen Boyd <sboyd@codeaurora.org> |
|---|---|
| First post | 2015-11-25 02:10 +0100 |
| Last post | 2015-11-25 14:30 +0100 |
| Articles | 9 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH 0/3] Remove ARCH_MSM* configs Stephen Boyd <sboyd@codeaurora.org> - 2015-11-25 02:10 +0100
[PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible Stephen Boyd <sboyd@codeaurora.org> - 2015-11-25 02:10 +0100
Re: [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible Stephen Boyd <sboyd@codeaurora.org> - 2015-11-25 03:10 +0100
Re: [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible Arnd Bergmann <arnd@arndb.de> - 2015-11-25 10:10 +0100
Re: [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-25 11:20 +0100
Re: [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible Arnd Bergmann <arnd@arndb.de> - 2015-11-25 11:20 +0100
Re: [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-25 13:40 +0100
Re: [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible Arnd Bergmann <arnd@arndb.de> - 2015-11-25 14:00 +0100
Re: [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible Daniel Lezcano <daniel.lezcano@linaro.org> - 2015-11-25 14:30 +0100
| From | Stephen Boyd <sboyd@codeaurora.org> |
|---|---|
| Date | 2015-11-25 02:10 +0100 |
| Subject | [PATCH 0/3] Remove ARCH_MSM* configs |
| Message-ID | <qywKS-7sm-3@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. The first patch can go via Daniel, while the other two can go via Andy. Stephen Boyd (3): clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible ARM: qcom_defconfig: Enable HAVE_ARM_ARCH_TIMER ARM: qcom: Drop ARCH_MSM* configs arch/arm/configs/qcom_defconfig | 1 + arch/arm/mach-qcom/Kconfig | 16 ---------------- drivers/clocksource/Kconfig | 3 ++- 3 files changed, 3 insertions(+), 17 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]
| From | Stephen Boyd <sboyd@codeaurora.org> |
|---|---|
| Date | 2015-11-25 02:10 +0100 |
| Subject | [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible |
| Message-ID | <qywKS-7sm-11@gated-at.bofh.it> |
| In reply to | #1276940 |
We want to remove the ARCH_MSM* configs in mach-qcom/Kconfig because they are mostly proxy configs for selecting the right clocksource driver. Therefore, make CLKSRC_QCOM default to the value of ARCH_QCOM, but also make it visible if ARCH_QCOM=y so that we can turn it off when we don't want it. Suggested-by: Arnd Bergmann <arnd@arndb.de> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> --- drivers/clocksource/Kconfig | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/clocksource/Kconfig b/drivers/clocksource/Kconfig index b423785d6afc..7a5ffaa3e490 100644 --- a/drivers/clocksource/Kconfig +++ b/drivers/clocksource/Kconfig @@ -279,9 +279,10 @@ config EM_TIMER_STI such as EMEV2 from former NEC Electronics. config CLKSRC_QCOM - bool "Qualcomm MSM timer" if COMPILE_TEST + bool "Qualcomm MSM timer" if ARCH_QCOM || COMPILE_TEST depends on ARM select CLKSRC_OF + defbool ARCH_QCOM help This enables the clocksource and the per CPU clockevent driver for the Qualcomm SoCs. -- 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]
| From | Stephen Boyd <sboyd@codeaurora.org> |
|---|---|
| Date | 2015-11-25 03:10 +0100 |
| Subject | Re: [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible |
| Message-ID | <qyxGW-85c-15@gated-at.bofh.it> |
| In reply to | #1276942 |
On 11/24/15 17:08, Stephen Boyd wrote: > We want to remove the ARCH_MSM* configs in mach-qcom/Kconfig > because they are mostly proxy configs for selecting the right > clocksource driver. Therefore, make CLKSRC_QCOM default to the > value of ARCH_QCOM, but also make it visible if ARCH_QCOM=y so > that we can turn it off when we don't want it. > > Suggested-by: Arnd Bergmann <arnd@arndb.de> > Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> > --- > drivers/clocksource/Kconfig | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/clocksource/Kconfig b/drivers/clocksource/Kconfig > index b423785d6afc..7a5ffaa3e490 100644 > --- a/drivers/clocksource/Kconfig > +++ b/drivers/clocksource/Kconfig > @@ -279,9 +279,10 @@ config EM_TIMER_STI > such as EMEV2 from former NEC Electronics. > > config CLKSRC_QCOM > - bool "Qualcomm MSM timer" if COMPILE_TEST > + bool "Qualcomm MSM timer" if ARCH_QCOM || COMPILE_TEST > depends on ARM > select CLKSRC_OF > + defbool ARCH_QCOM Urgh. This should be def_bool -- Qualcomm Innovation Center, Inc. is a member of 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]
| From | Arnd Bergmann <arnd@arndb.de> |
|---|---|
| Date | 2015-11-25 10:10 +0100 |
| Subject | Re: [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible |
| Message-ID | <qyEfo-48i-25@gated-at.bofh.it> |
| In reply to | #1276967 |
On Tuesday 24 November 2015 18:07:20 Stephen Boyd wrote: > On 11/24/15 17:08, Stephen Boyd wrote: > > We want to remove the ARCH_MSM* configs in mach-qcom/Kconfig > > because they are mostly proxy configs for selecting the right > > clocksource driver. Therefore, make CLKSRC_QCOM default to the > > value of ARCH_QCOM, but also make it visible if ARCH_QCOM=y so > > that we can turn it off when we don't want it. > > > > Suggested-by: Arnd Bergmann <arnd@arndb.de> > > Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> > > --- > > drivers/clocksource/Kconfig | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/clocksource/Kconfig b/drivers/clocksource/Kconfig > > index b423785d6afc..7a5ffaa3e490 100644 > > --- a/drivers/clocksource/Kconfig > > +++ b/drivers/clocksource/Kconfig > > @@ -279,9 +279,10 @@ config EM_TIMER_STI > > such as EMEV2 from former NEC Electronics. > > > > config CLKSRC_QCOM > > - bool "Qualcomm MSM timer" if COMPILE_TEST > > + bool "Qualcomm MSM timer" if ARCH_QCOM || COMPILE_TEST > > depends on ARM > > select CLKSRC_OF > > + defbool ARCH_QCOM > > Urgh. This should be def_bool > > With this fixed, the series looks good to me. If Daniel can Ack this, I'd suggest we take the series through arm-soc. 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]
| From | Daniel Lezcano <daniel.lezcano@linaro.org> |
|---|---|
| Date | 2015-11-25 11:20 +0100 |
| Subject | Re: [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible |
| Message-ID | <qyFl7-4Se-7@gated-at.bofh.it> |
| In reply to | #1276942 |
On 11/25/2015 02:08 AM, Stephen Boyd wrote: > We want to remove the ARCH_MSM* configs in mach-qcom/Kconfig > because they are mostly proxy configs for selecting the right > clocksource driver. Therefore, make CLKSRC_QCOM default to the > value of ARCH_QCOM, but also make it visible if ARCH_QCOM=y so > that we can turn it off when we don't want it. I have been removing the ARCH dependencies in the Kconfig file. Why do you have to turn it off manually ? > Suggested-by: Arnd Bergmann <arnd@arndb.de> > Signed-off-by: Stephen Boyd <sboyd@codeaurora.org> > --- > drivers/clocksource/Kconfig | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/clocksource/Kconfig b/drivers/clocksource/Kconfig > index b423785d6afc..7a5ffaa3e490 100644 > --- a/drivers/clocksource/Kconfig > +++ b/drivers/clocksource/Kconfig > @@ -279,9 +279,10 @@ config EM_TIMER_STI > such as EMEV2 from former NEC Electronics. > > config CLKSRC_QCOM > - bool "Qualcomm MSM timer" if COMPILE_TEST > + bool "Qualcomm MSM timer" if ARCH_QCOM || COMPILE_TEST > depends on ARM > select CLKSRC_OF > + defbool ARCH_QCOM > help > This enables the clocksource and the per CPU clockevent driver for the > Qualcomm SoCs. > -- <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] | [next] | [standalone]
| From | Arnd Bergmann <arnd@arndb.de> |
|---|---|
| Date | 2015-11-25 11:20 +0100 |
| Subject | Re: [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible |
| Message-ID | <qyFl8-4Se-17@gated-at.bofh.it> |
| In reply to | #1277156 |
On Wednesday 25 November 2015 11:10:49 Daniel Lezcano wrote: > On 11/25/2015 02:08 AM, Stephen Boyd wrote: > > We want to remove the ARCH_MSM* configs in mach-qcom/Kconfig > > because they are mostly proxy configs for selecting the right > > clocksource driver. Therefore, make CLKSRC_QCOM default to the > > value of ARCH_QCOM, but also make it visible if ARCH_QCOM=y so > > that we can turn it off when we don't want it. > > I have been removing the ARCH dependencies in the Kconfig file. > > Why do you have to turn it off manually ? The background is that this is used only on some of the older MSM SoCs, while the newer ones use the arch timer. We decided to remove the SoC-specific top-level options from mach-msm as they are becoming rather meaningless these days and just a burden to maintain at the rate that new variants get released, so being able to turn off this driver helps make the kernel slightly smaller if you are building a kernel for only the more recent models. 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]
| From | Daniel Lezcano <daniel.lezcano@linaro.org> |
|---|---|
| Date | 2015-11-25 13:40 +0100 |
| Subject | Re: [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible |
| Message-ID | <qyHwD-6ho-39@gated-at.bofh.it> |
| In reply to | #1277157 |
On 11/25/2015 11:17 AM, Arnd Bergmann wrote: > On Wednesday 25 November 2015 11:10:49 Daniel Lezcano wrote: >> On 11/25/2015 02:08 AM, Stephen Boyd wrote: >>> We want to remove the ARCH_MSM* configs in mach-qcom/Kconfig >>> because they are mostly proxy configs for selecting the right >>> clocksource driver. Therefore, make CLKSRC_QCOM default to the >>> value of ARCH_QCOM, but also make it visible if ARCH_QCOM=y so >>> that we can turn it off when we don't want it. >> >> I have been removing the ARCH dependencies in the Kconfig file. >> >> Why do you have to turn it off manually ? > > The background is that this is used only on some of the older > MSM SoCs, while the newer ones use the arch timer. > > We decided to remove the SoC-specific top-level options from > mach-msm as they are becoming rather meaningless these days > and just a burden to maintain at the rate that new variants > get released, so being able to turn off this driver helps make > the kernel slightly smaller if you are building a kernel for > only the more recent models. Ok, thanks for the clarification. I don't really like this approach even if it is correct because it breaks the current approach I am trying to make consistent across the drivers. I would like to have the COMPILE_TEST option available for all the drivers and move this option under the menu config. This patch will prevent to do this code factoring. On the other side, this option is supposed to have a slightly smaller kernel when it is not used. But when does it happen ? When ARCH_MSM8X60=n and ARCH_MSM8960=n. With this patchset, I don't see the ability to turn these SoCs off as the options are removed. So the associated code is not removed, right ? So why allow to turn off the timer but disallow that for the entire SoC ? -- <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] | [next] | [standalone]
| From | Arnd Bergmann <arnd@arndb.de> |
|---|---|
| Date | 2015-11-25 14:00 +0100 |
| Subject | Re: [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible |
| Message-ID | <qyHPY-6p2-23@gated-at.bofh.it> |
| In reply to | #1277348 |
On Wednesday 25 November 2015 13:37:53 Daniel Lezcano wrote: > On 11/25/2015 11:17 AM, Arnd Bergmann wrote: > > On Wednesday 25 November 2015 11:10:49 Daniel Lezcano wrote: > >> On 11/25/2015 02:08 AM, Stephen Boyd wrote: > >>> We want to remove the ARCH_MSM* configs in mach-qcom/Kconfig > >>> because they are mostly proxy configs for selecting the right > >>> clocksource driver. Therefore, make CLKSRC_QCOM default to the > >>> value of ARCH_QCOM, but also make it visible if ARCH_QCOM=y so > >>> that we can turn it off when we don't want it. > >> > >> I have been removing the ARCH dependencies in the Kconfig file. > >> > >> Why do you have to turn it off manually ? > > > > The background is that this is used only on some of the older > > MSM SoCs, while the newer ones use the arch timer. > > > > We decided to remove the SoC-specific top-level options from > > mach-msm as they are becoming rather meaningless these days > > and just a burden to maintain at the rate that new variants > > get released, so being able to turn off this driver helps make > > the kernel slightly smaller if you are building a kernel for > > only the more recent models. > > Ok, thanks for the clarification. > > I don't really like this approach even if it is correct because it > breaks the current approach I am trying to make consistent across the > drivers. > > I would like to have the COMPILE_TEST option available for all the > drivers and move this option under the menu config. This patch will > prevent to do this code factoring. How about moving the option to arch/arm/mach-qcom/Kconfig then? We could have a user-selectable "allow use of qcom clocksource" option there, which would then select the driver. > On the other side, this option is supposed to have a slightly smaller > kernel when it is not used. But when does it happen ? When > ARCH_MSM8X60=n and ARCH_MSM8960=n. With this patchset, I don't see the > ability to turn these SoCs off as the options are removed. So the > associated code is not removed, right ? > > So why allow to turn off the timer but disallow that for the entire SoC ? The timer is the only code that is controlled by those two options at the moment, all the other differences between SoCs are already handled by enabling the respective device drivers. 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]
| From | Daniel Lezcano <daniel.lezcano@linaro.org> |
|---|---|
| Date | 2015-11-25 14:30 +0100 |
| Subject | Re: [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible |
| Message-ID | <qyIj0-6Qa-15@gated-at.bofh.it> |
| In reply to | #1277370 |
On 11/25/2015 01:49 PM, Arnd Bergmann wrote: > On Wednesday 25 November 2015 13:37:53 Daniel Lezcano wrote: >> On 11/25/2015 11:17 AM, Arnd Bergmann wrote: >>> On Wednesday 25 November 2015 11:10:49 Daniel Lezcano wrote: >>>> On 11/25/2015 02:08 AM, Stephen Boyd wrote: >>>>> We want to remove the ARCH_MSM* configs in mach-qcom/Kconfig >>>>> because they are mostly proxy configs for selecting the right >>>>> clocksource driver. Therefore, make CLKSRC_QCOM default to the >>>>> value of ARCH_QCOM, but also make it visible if ARCH_QCOM=y so >>>>> that we can turn it off when we don't want it. >>>> >>>> I have been removing the ARCH dependencies in the Kconfig file. >>>> >>>> Why do you have to turn it off manually ? >>> >>> The background is that this is used only on some of the older >>> MSM SoCs, while the newer ones use the arch timer. >>> >>> We decided to remove the SoC-specific top-level options from >>> mach-msm as they are becoming rather meaningless these days >>> and just a burden to maintain at the rate that new variants >>> get released, so being able to turn off this driver helps make >>> the kernel slightly smaller if you are building a kernel for >>> only the more recent models. >> >> Ok, thanks for the clarification. >> >> I don't really like this approach even if it is correct because it >> breaks the current approach I am trying to make consistent across the >> drivers. >> >> I would like to have the COMPILE_TEST option available for all the >> drivers and move this option under the menu config. This patch will >> prevent to do this code factoring. > > How about moving the option to arch/arm/mach-qcom/Kconfig then? > > We could have a user-selectable "allow use of qcom clocksource" > option there, which would then select the driver. Yes, why not. >> On the other side, this option is supposed to have a slightly smaller >> kernel when it is not used. But when does it happen ? When >> ARCH_MSM8X60=n and ARCH_MSM8960=n. With this patchset, I don't see the >> ability to turn these SoCs off as the options are removed. So the >> associated code is not removed, right ? >> >> So why allow to turn off the timer but disallow that for the entire SoC ? > > The timer is the only code that is controlled by those two options at > the moment, all the other differences between SoCs are already handled > by enabling the respective device drivers. Ok, I see. Thanks. -- Daniel -- <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