Re: [PATCH] iio: afe: iio-rescale: Support processed channels

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2020-12-12 13:26, Linus Walleij wrote:
> On Mon, Nov 2, 2020 at 12:22 AM Linus Walleij <linus.walleij@xxxxxxxxxx> wrote:
> 
>> It happens that an ADC will only provide raw or processed
>> voltage conversion channels. (adc/ab8500-gpadc.c).
>> On the Samsung GT-I9070 this is used for a light sensor
>> and current sense amplifier so we need to think of something.
>>
>> The idea is to allow processed channels and scale them
>> with 1/1 and then the rescaler can modify the result
>> on top.
>>
>> Cc: Peter Rosin <peda@xxxxxxxxxx>
>> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
> 
> Did we reach any conclusion on this? I really need to use
> the rescaler on an ADC that only handles processed channels...
> 
> I'm sorry that I can't make this ADC disappear :D

Hi!

My conclusion was that the patch is buggy since it presents inconsistent
information. That needs to be fixed one way or the other. If the offending
information cannot be filtered out for some reason, I don't know what to
do. Details in my previous comment [1]. BTW, I still do not know the answer
to the .read_avail question at the end of that message, and I don't have
time to dig into it. Sorry.

Cheers,
Peter

[1] https://lore.kernel.org/linux-iio/320464d8-659c-01de-0e08-34e4c744ef16@xxxxxxxxxx/



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux