Hi,

On Fri, Mar 24, 2017 at 03:44:51AM -0700, Liam Breck wrote:

On Fri, Mar 24, 2017 at 2:39 AM, Sebastian Reichel [off-list ref] wrote:

quoted

On Thu, Mar 23, 2017 at 01:47:23PM -0700, Liam Breck wrote:

quoted

On Thu, Mar 23, 2017 at 3:52 AM, Sebastian Reichel [off-list ref] wrote:

quoted

That still leaves the health status reading, though. From
userspace's point-of-view this should be merged into the
battery device, if fuel-gauge can't provide that information.

You mean merged into the -charger device? -battery is the one
we're dropping.

If you did mean -battery, could you elaborate?

I was not talking about bq24190-battery. I meant the battery
device provided by the fuel-gauge. From userspace's point of
view a battery device, which is usually derived from the
fuel-gauge should contain battery related information.

It doesn't sound right to ask every gauge to absorb features from a
specific charger. Userspace apps are already watching both USB and
battery power supply types for a given battery. They'll pick up a
health issue if we merge -battery and -charger health. Isn't that
enough?

Technically it's not correct, since high battery temperature is a
much bigger safety risk than high charger chip temperature. I guess
it's good enough, though. I guess the battery temperature health is
related to charging anyways, which is different from the ones making
the battery itself dangerous. So I'm fine with that solution.

quoted

Since the battery health information is battery related it
should somehow be merged into the data from the fuel-gauge.
That's what Hans implemented with the function-pointer
construct and what you suggested to implement in this mail:

http://marc.info/?l=linux-pm&m=149026871313238&w=2

The merge I suggested was of properties from all power drivers into a
single sysfs device. Which I would work on if you are eager to have
it. I realize it would take a while to be merged.

Only the devices belonging to the same battery should be
exposed as one device. Also charger is extra device, since
it may be used to charge multiple batteries. So far its
pretty uncommon, that charger chip provides information
about the battery, so you would spent lots of time for
a single corner case. So while I would like to have it,
I don't think its worth the time to implement it.

-- Sebastian