Thread (36 messages) 36 messages, 6 authors, 2015-03-27

Re: [PATCH v7 3/5] power: max77843_battery: Add Max77843 fuel gauge device driver

From: Krzysztof Kozlowski <hidden>
Date: 2015-03-09 10:01:44
Also in: linux-devicetree, linux-pm, lkml

2015-03-09 1:36 GMT+01:00 Beomho Seo [off-list ref]:
On 03/08/2015 05:14 AM, Sebastian Reichel wrote:
quoted
Hi,

On Mon, Mar 02, 2015 at 07:10:36PM +0900, Jaewon Kim wrote:
quoted
From: Beomho Seo <redacted>

This patch adds device driver of max77843 fuel gauge.
The driver support for battery fuel gauge in Maxim Max77843.
It is fuel-gauge systems for lithuum-ion batteries in handled and
portable devices.

Cc: Sebastian Reichel <sre@kernel.org>
Signed-off-by: Beomho Seo <redacted>
Reviewed-By: Sebastian Reichel <sre@kernel.org>

I can't take it as is, since it depends on the private header file
of PATCH 1.

-- Sebastian
This patch reviewed by Sebastian.
Could you Please merge that your git tree ?
Hi,

Sorry for late response, but I finally got some time to look at this.
This driver looks very similar to max17042_battery.c fuel gauge
driver. Obtaining some properties looks exactly the same. The
difference seems to be in new properties. The I2C address is the same.

I highly recommend to extend the max17042 driver instead. It already
supports also max17047, max17050 and max77693.

Best regards,
Krzysztof
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help