Re: [PATCH 00/14] POWER: BQ27x00: New Properties, fixes, bq27000 support
From: Felipe Contreras
Date: Sun Feb 06 2011 - 19:01:02 EST
On Mon, Feb 7, 2011 at 1:09 AM, Lars-Peter Clausen <lars@xxxxxxxxxx> wrote:
> On 02/06/2011 11:47 PM, Felipe Contreras wrote:
>> Hi,
>>
>> On Sun, Feb 6, 2011 at 2:47 AM, Lars-Peter Clausen <lars@xxxxxxxxxx> wrote:
>>> This patch series contains a few updates for the bq27x00 driver:
>>> * Support for additional power supply properties
>>> * Support for the bq27000 battery which is identical to the bq27200 but is
>>> Âconnected through the HDQ bus.
>>> * Adds a register cache to the driver and introduces polling the batteries state
>>> * Minor improvements and cleanups
>>>
>>> The last patch in this series is not specific to the bq27x00 driver but is
>>> required for uevents to be generated properly for this driver.
>>> The patch makes properties which return -ENODATA to be ignored when generating
>>> uevents. Previously in such a case uevent generation would have been aborted
>>> with an error. But since the bq27x00 return -ENODATA for the TIME_TO_FULL
>>> property when the battery is not charging and for the TIME_TO_EMPTY property
>>> when the battery is not discharging and at least one of them is always true
>>> uevent generation would always fail.
>>>
>>> This series has so far been tested with the bq27000 and the bq27200 battery, but
>>> not with the bq27500 battery, so it would be nice if somebody with a board
>>> containing such a battery could test the patches to make sure that there are no
>>> regressions.
>>
>> This might be a bit off-topic, but I have tried to get the battery on
>> the Nokia N900 working, and so far I'm only able to query the
>> remaining capacity, but not really to enabling charging. This is very
>> annoying, as I can only hack a certain amount of time, and then I have
>> to reflash, reboot, and wait for the batter to charge with the
>> official system (if there's enough power to flash).
>>
>> Do you have any hints how to do that?
>
> Sorry, I've never worked with the N900, so I have no idea. But Pali has tested this
> patch series on his N900, so he might have an idea.
Well, it has a bq27200 AFAIK.
--
Felipe Contreras
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/