Received-SPF: pass (google.com: domain of linux-kernel+bounces-200550-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <630d943e-3bf3-46ab-b3b8-cc44a86cab64@collabora.com>
Date: Tue, 4 Jun 2024 13:57:54 +0200
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v1 2/4] iio: adc: Add support for MediaTek MT6357/8/9
 Auxiliary ADC
To: Andy Shevchenko <andy.shevchenko@gmail.com>
Cc: Jonathan Cameron <jic23@kernel.org>, lars@metafoo.de, robh@kernel.org,
 krzk+dt@kernel.org, conor+dt@kernel.org, matthias.bgg@gmail.com,
 lee@kernel.org, andy@kernel.org, nuno.sa@analog.com, bigunclemax@gmail.com,
 dlechner@baylibre.com, marius.cristea@microchip.com,
 marcelo.schmitt@analog.com, fr0st61te@gmail.com, mitrutzceclan@gmail.com,
 mike.looijmans@topic.nl, marcus.folkesson@gmail.com,
 linux-iio@vger.kernel.org, devicetree@vger.kernel.org,
 linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
 linux-mediatek@lists.infradead.org, kernel@collabora.com
References: <20240530093410.112716-1-angelogioacchino.delregno@collabora.com>
 <20240530093410.112716-3-angelogioacchino.delregno@collabora.com>
 <20240602111141.0058f39e@jic23-huawei>
 <60e55919-2a8c-4d83-89a1-6e4ae156d34d@collabora.com>
 <CAHp75Vf5a8VVyOXQRt9P1QnM6GHZ3rLuvnBF63H_83QBHOTJ9w@mail.gmail.com>
From: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Content-Language: en-US
In-Reply-To: <CAHp75Vf5a8VVyOXQRt9P1QnM6GHZ3rLuvnBF63H_83QBHOTJ9w@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit

Il 04/06/24 12:55, Andy Shevchenko ha scritto:
> On Tue, Jun 4, 2024 at 12:42 PM AngeloGioacchino Del Regno
> <angelogioacchino.delregno@collabora.com> wrote:
>> Il 02/06/24 12:11, Jonathan Cameron ha scritto:
>>> On Thu, 30 May 2024 11:34:08 +0200
>>> AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> wrote:
> 
> ...
> 
>>> What are IMP channels?
>>
>> Honestly? Well, it's called like that. I don't have any clear description of that
>> and not even datasheets are unrolling the meaning of "IMP". So.. I don't know.
>>
>> What I know is what I wrote in the driver, and this is:
>> * IMP has IBAT and VBAT ADCs
>> * It needs different handling from the other ADCs, as shown.
>>
>> ...and nothing else :-(
> 
> I could speculate with confidence that this means IMPedance (since it's ADC).
> 
>  From MTK6329  datasheet:
> "The hardware also includes necessary modes to allow for simultaneous
> current and voltage measurement
> which can be utilized to estimate the battery impedance."
> And googling in vendor trees also suggests the same.
> 

That does make a lot of sense. Thanks for that, I don't have the 6329 datasheet.

Great, one more piece added to the puzzle!

Cheers,
Angelo