linux-kernel - Re: [PATCH 0/2] iio: adc: Add a current from voltage driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <BN6PR04MB0660FED60AD7BE9AD09B56A8A3BA0@BN6PR04MB0660.namprd04.prod.outlook.com>
Date:   Sat, 16 May 2020 08:01:12 -0700
From:   Jonathan Bakker <xc-racer2@...e.ca>
To:     Lars-Peter Clausen <lars@...afoo.de>, jic23@...nel.org,
        knaack.h@....de, pmeerw@...erw.net, robh+dt@...nel.org,
        linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org,
        devicetree@...r.kernel.org
Cc:     linus.walleij@...aro.org
Subject: Re: [PATCH 0/2] iio: adc: Add a current from voltage driver

Oops, sorry about that, you're absolutely right.  The current-sense-shunt part
of iio-rescale is exactly what I was looking for.

Thanks,
Jonathan

On 2020-05-15 11:28 p.m., Lars-Peter Clausen wrote:
> On 5/16/20 4:26 AM, Jonathan Bakker wrote:
>> In the discussion around adding the GP2A002 light driver, there came
>> up the question of what to do when a system emulates a current ADC
>> by using a voltage ADC and a resistor.  Rather than adding it on
>> a per-driver basis, it was suggested(1) to add a minimal IIO driver
>> to support this situation.
>>
>> The new driver is fairly simple - it simply takes a voltage ADC and
>> a resistor value in ohms exposed as the scale and outputs a current.
>>
>> It has been tested on a first-gen Galaxy S device which has the above
>> mentioned GP2A002 chip connected to the voltage ADC resistor complex.
>>
>> 1) https://lore.kernel.org/linux-iio/20200202150843.762c6897@archlinux/
> 
> Hi,
> 
> There is afe/iio-rescale.c, which I think already implements this functionality.
> 
> - Lars
> 
>