Re: [PATCH 2/2] hwmon: Aspeed AST2400/AST2500 ADC

From	Guenter Roeck <linux@roeck-us.net>
Newsgroups	linux.kernel
Subject	Re: [PATCH 2/2] hwmon: Aspeed AST2400/AST2500 ADC
Date	2017-03-05 17:20 +0100
Message-ID	<thH33-2Ka-7@gated-at.bofh.it> (permalink)
References	<tfXlE-1rB-19@gated-at.bofh.it> <tfXlE-1rB-17@gated-at.bofh.it> <tg0My-3uP-7@gated-at.bofh.it> <tg9mP-1zt-37@gated-at.bofh.it> <tgPm1-5Hb-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 03/02/2017 10:21 PM, Rob Herring wrote:
> On Tue, Feb 28, 2017 at 07:45:23PM -0800, Guenter Roeck wrote:
>> On 02/28/2017 04:49 PM, Joel Stanley wrote:
>>> On Wed, Mar 1, 2017 at 6:44 AM, Rick Altherr <raltherr@google.com> wrote:
>>>> Aspeed AST2400/AST2500 BMC SoCs include a 16 channel, 10-bit ADC. This
>>>> driver implements reading the ADC values, enabling channels via device
>>>> tree, and optionally providing channel labels via device tree.  Low and
>>>> high threshold interrupts are supported by the hardware but not
>>>> implemented.
>>>>
>>>> Signed-off-by: Rick Altherr <raltherr@google.com>
>>>
>>> Looks good. Some minor comments below.
>>>
>>> Is there a reason you wrote a hwmon driver instead of an iio driver? I
>>> wasn't sure what the recommended subsystem is.
>>
>> Excellent point. Question is really if there is a plan to add support for
>> thresholds. If not, an iio driver might be more appropriate.
>
> Sigh. We have ADCs in 2 places? Fine for the kernel I guess, but not
> bindings.
>

Almost every {voltage, current, power, temperature} sensor chip is implemented
as ADC. Given that, we have (at least) three places. hwmon, iio, thermal.

Guenter

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH 2/2] hwmon: Aspeed AST2400/AST2500 ADC Guenter Roeck <linux@roeck-us.net> - 2017-03-01 11:10 +0100
  Re: [PATCH 2/2] hwmon: Aspeed AST2400/AST2500 ADC Rick Altherr <raltherr@google.com> - 2017-03-02 05:00 +0100
    Re: [PATCH 2/2] hwmon: Aspeed AST2400/AST2500 ADC Guenter Roeck <linux@roeck-us.net> - 2017-03-05 19:10 +0100
      Re: [PATCH 2/2] hwmon: Aspeed AST2400/AST2500 ADC Rick Altherr <raltherr@google.com> - 2017-03-08 01:00 +0100
  Re: [PATCH 2/2] hwmon: Aspeed AST2400/AST2500 ADC Rob Herring <robh@kernel.org> - 2017-03-03 08:00 +0100
    Re: [PATCH 2/2] hwmon: Aspeed AST2400/AST2500 ADC Guenter Roeck <linux@roeck-us.net> - 2017-03-05 17:20 +0100

csiph-web