* Arthur D. <spinal.by@xxxxxxxxx> [200316 01:31]: > Hi, Tony. > > It seems like a misunderstanding here. There's no problem in detecting > if the charging is in progress. The green led is switched off and > the battery current sign is changed from "-" to "+" (which means > that the battery is being discharged). So there's no need in additional > checks. For cpcap-battery this situation seems like a battery stopped > charging. And it doesn't matter if that was a user who disconnected > the charger or it was done somewhere in a driver/firmware/hardware. > > The problem is that the charging current cant get to the point <100 mA, > not talking about <50 mA. And that's why I set the value of 112 mA for > the end of charge current: to help the kernel to detect this plateau and > to stop the calibration cycle, so the userspace can get all the battery > parameters I mentioned in the previous mail. OK I guess that's easy to change if we figure out something better :) Maybe add some define for it like CPCAP_BATTERY_FULL_CHARGE_CURRENT or similar? > Please note, that the behaviour I mentioned was observed only when the > conditions written in my last mail were met. The important one was: > > 2) the display backlight is off > > Because when I unlocked the display the charging current was able > to go below 112 mA. Of course I couldn't rely on something like this: > the user should stay with backlight on to have the battery calibrated. > Think about it: waiting for the charging current to drop from 100 mA > to 50 mA can take dozens of minutes (it depends on the age of battery - > the older the battery the longer it will take), and the user should > force somehow the device to not switch off the display hightlight > until the battery is calibrated. > > Of course it's unacceptable, so I decided to set the end of charge > current limit to 112 mA. Which allows the user to just put the device > on a table and to wait until it's fully charged without a need > to interfere the charging process with some action from the user. Yeah OK thanks. Tony
