Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <20190107190555.GA18865@amd> <20190311102325.GB14118@amd>
In-Reply-To: <20190311102325.GB14118@amd>
From:   Baolin Wang <baolin.wang@linaro.org>
Date:   Mon, 11 Mar 2019 19:11:24 +0800
Message-ID: <CAMz4kuJznvDiU8-mL=gS3NEJ1Q=4KP+uHq6B3jQsbUUrbHzXRw@mail.gmail.com>
Subject: Re: 5.0-rc1 Open circuit voltage tables -- 3afb50d7125bc
To:     Pavel Machek <pavel@ucw.cz>
Cc:     kernel list <linux-kernel@vger.kernel.org>,
        Linus Walleij <linus.walleij@linaro.org>,
        Sebastian Reichel <sebastian.reichel@collabora.com>,
        Sebastian Reichel <sre@kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hi Pavel,

On Mon, 11 Mar 2019 at 18:23, Pavel Machek <pavel@ucw.cz> wrote:
>
> On Mon 2019-01-07 20:05:55, Pavel Machek wrote:
> > Hi!
> >
> > Can we do something with that code?
> >
> > First -- tables should not be necceessary. Formula for li-ion open
> > circuit voltage is well-known, and does not really depend on the
> > device -- only on the chemistry of battery.
> >
> > state of charge = f( voltage, current, internal resistance )
> >
> > (And yes, internal resistance depends on temperature and health of the
> > battery).

As we know, experiments had proved that the OCV had a steady
relationship with the state of charge for a given chemical
characteristic. So many companies' charging method will use OCV table
to look up the initial capacity of the battery according to the
current OCV value. That's why we introduce the OCV table.

> >
> > Second, even if we need tables, why is it doing multiple allocations?
> >
> > info->ocv_temp[index]        = -EINVAL;
> >
> > That's just wrong. Ocv_temp is temperature in celsius. Temperatures go
> > below zero.

This requirement is from Qualcomm. Since they have multiple OCV tables
in different temperatures.

>
> Ping? That code is just bad. Does someone care?

Sorry, I missed your previous email.
--
Baolin Wang
Best Regards