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: hughsi...@gmail.com
> To: j_m_hj...@hotmail.com
> CC: devkit-devel@lists.freedesktop.org
> 
> On 1 July 2011 16:43, Mikael Hjelm <j_m_hj...@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@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/devkit-devel
                                          
_______________________________________________
devkit-devel mailing list
devkit-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/devkit-devel

Reply via email to