[Mikael Hjelm]

Afraid not, the only patch i have is to disable the disabling of the poll in 

diff src/linux/up-device-supply.c src/linux/up-device-supply.c~

549c549

< 	/* supply->priv->enable_poll = (state != UP_DEVICE_STATE_FULLY_CHARGED); */

---


> 	supply->priv->enable_poll = (state != UP_DEVICE_STATE_FULLY_CHARGED);


In this function you have no knowledge of the AC power state, which is the problem in the first place. 
This function only knows the battery properties and they are too slow to react. 

Is this ok or do you think it should be in the setup_poll function to reenable the poll or is there some way that the battery can know if any AC supply is connected?
If you want a more formal patch i need to talk to some friends of mine who can create a patch from the git tree. 

BTW sorry for the no subject thing i forgot the subject when i copied the old email.

BR 
Mikael Hjelm



> Date: Fri, 1 Jul 2011 18:59:51 +0100
> Subject: Re:
> From: hughsient at gmail.com
> To: j_m_hjelm at hotmail.com
> CC: devkit-devel at lists.freedesktop.org
> 
> On 1 July 2011 16:43, Mikael Hjelm <j_m_hjelm at hotmail.com> wrote:
> > I would like to see a change either in setup_poll to reenable the poll
> > independent of the enable_poll state, or even resetting it to enabled, or do
> > not disable the enable_poll unless the state is
> > UP_DEVICE_STATE_FULLY_CHARGED AND AC power is online.
> 
> Sounds okay I guess. Got a patch please?
> 
> Richard.
> _______________________________________________
> devkit-devel mailing list
> devkit-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/devkit-devel
 		 	   		  
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/devkit-devel/attachments/20110701/095f001c/attachment.html>