[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <51ADDE7A.6090205@ti.com>
Date: Tue, 4 Jun 2013 08:32:58 -0400
From: Eduardo Valentin <eduardo.valentin@...com>
To: Fabio Estevam <festevam@...il.com>
CC: Eduardo Valentin <eduardo.valentin@...com>, <tony@...mide.com>,
<linux-omap@...r.kernel.org>,
Russell King <linux@....linux.org.uk>,
<linux-kernel@...r.kernel.org>,
<linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCHv2 1/3] arm: introduce config HAS_BANDGAP
Fabio,
On 31-05-2013 11:12, Fabio Estevam wrote:
> On Wed, May 15, 2013 at 11:58 AM, Eduardo Valentin
> <eduardo.valentin@...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
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
Download attachment "signature.asc" of type "application/pgp-signature" (296 bytes)
Powered by blists - more mailing lists