| Message ID | 1368629928-6723-2-git-send-email-eduardo.valentin@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Tony, Russell, On 15-05-2013 10:58, Eduardo Valentin wrote: > Introduce HAS_BANDGAP config entry. This config is a > boolean value so that arch code can flag is they > feature a bandgap device. > > This config entry follows the same idea behind > ARCH_HAS_CPUFREQ. > > Cc: Russell King <linux@arm.linux.org.uk> > Cc: Tony Lindgren <tony@atomide.com> > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-omap@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com> Gentle reminder on this one. > --- > arch/arm/Kconfig | 3 +++ > arch/arm/mach-omap2/Kconfig | 1 + > 2 files changed, 4 insertions(+) > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig > index d423d58..bcbdec9 100644 > --- a/arch/arm/Kconfig > +++ b/arch/arm/Kconfig > @@ -174,6 +174,9 @@ config ARCH_HAS_CPUFREQ > and that the relevant menu configurations are displayed for > it. > > +config ARCH_HAS_BANDGAP > + bool > + > config GENERIC_HWEIGHT > bool > default y > diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig > index f49cd51..8620ab5 100644 > --- a/arch/arm/mach-omap2/Kconfig > +++ b/arch/arm/mach-omap2/Kconfig > @@ -4,6 +4,7 @@ config ARCH_OMAP > config ARCH_OMAP2PLUS > bool "TI OMAP2/3/4/5 SoCs with device tree support" if (ARCH_MULTI_V6 || ARCH_MULTI_V7) > select ARCH_HAS_CPUFREQ > + select ARCH_HAS_BANDGAP > select ARCH_HAS_HOLES_MEMORYMODEL > select ARCH_OMAP > select ARCH_REQUIRE_GPIOLIB >
On Wed, May 15, 2013 at 11:58 AM, Eduardo Valentin <eduardo.valentin@ti.com> wrote: > Introduce HAS_BANDGAP config entry. This config is a > boolean value so that arch code can flag is they > feature a bandgap device. > > This config entry follows the same idea behind > ARCH_HAS_CPUFREQ. It would be nice if you explain what BANDGAP means here, since you introduce this option in this patch.
Fabio, On 31-05-2013 11:12, Fabio Estevam wrote: > On Wed, May 15, 2013 at 11:58 AM, Eduardo Valentin > <eduardo.valentin@ti.com> wrote: >> Introduce HAS_BANDGAP config entry. This config is a >> boolean value so that arch code can flag is they >> feature a bandgap device. >> >> This config entry follows the same idea behind >> ARCH_HAS_CPUFREQ. > > It would be nice if you explain what BANDGAP means here, since you > introduce this option in this patch. > > Yeah, I didnt do it because bandgap is a common term used in digital circuits [1,2]. Also, as in this patch we do not add term different from the common used bandgap term, I believe it is self explanatory. I any case, I am resending with minor explanation. [1] - http://en.wikipedia.org/wiki/Bandgap_voltage_reference [2] - http://en.wikipedia.org/wiki/Silicon_bandgap_temperature_sensor
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index d423d58..bcbdec9 100644 --- a/arch/arm/Kconfig +++ b/arch/arm/Kconfig @@ -174,6 +174,9 @@ config ARCH_HAS_CPUFREQ and that the relevant menu configurations are displayed for it. +config ARCH_HAS_BANDGAP + bool + config GENERIC_HWEIGHT bool default y diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig index f49cd51..8620ab5 100644 --- a/arch/arm/mach-omap2/Kconfig +++ b/arch/arm/mach-omap2/Kconfig @@ -4,6 +4,7 @@ config ARCH_OMAP config ARCH_OMAP2PLUS bool "TI OMAP2/3/4/5 SoCs with device tree support" if (ARCH_MULTI_V6 || ARCH_MULTI_V7) select ARCH_HAS_CPUFREQ + select ARCH_HAS_BANDGAP select ARCH_HAS_HOLES_MEMORYMODEL select ARCH_OMAP select ARCH_REQUIRE_GPIOLIB
Introduce HAS_BANDGAP config entry. This config is a boolean value so that arch code can flag is they feature a bandgap device. This config entry follows the same idea behind ARCH_HAS_CPUFREQ. Cc: Russell King <linux@arm.linux.org.uk> Cc: Tony Lindgren <tony@atomide.com> Cc: linux-arm-kernel@lists.infradead.org Cc: linux-omap@vger.kernel.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com> --- arch/arm/Kconfig | 3 +++ arch/arm/mach-omap2/Kconfig | 1 + 2 files changed, 4 insertions(+)