<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Tahoma
}
--></style>
</head>
<body class='hmmessage'><div dir='ltr'>
The thing is that i do not know what to blame. <br>I consider this change necessary to cope with misbehaving hardware.<br><br>The hardware affected by this is the ones that<br>1: Does not interrupt on state change.<br>2: Reports state change late.<br><br>Because there is no way of knowing if the hardware is fawlty then the polling cannot be disabled.<br>I think that there might be more advanced ways to determine if the polling can be disabled or not,<br>one might imagine that if there has been any interrupt from the battery at all then disabling the polling <br>would be ok. Also one migth consider some other means to know if the AC is connected or not but as<br>i stated in a previous email we are within a battery context function and it has no knowledge of the AC supply<br>at all. <br><br>I have tried to find any information on the battery drivers to see if they are responsible for not reporting <br>discharging fast enough but i do not know where to look to be honest. <br><br>This bug has beeen present since 2009 and affects acer laptops and some other models. I think it depends on <br>how much the hardware filters the charge current it reports.<br><br>Mikael<br><br><br><div>&gt; Date: Mon, 11 Jul 2011 09:15:26 +0100<br>&gt; Subject: Re: [PATCH] Patch to solve issues with some hardware that causes        battery to never update<br>&gt; From: hughsient@gmail.com<br>&gt; To: j_m_hjelm@hotmail.com<br>&gt; CC: devkit-devel@lists.freedesktop.org<br>&gt; <br>&gt; On 10 July 2011 21:34, Mikael Hjelm &lt;j_m_hjelm@hotmail.com&gt; wrote:<br>&gt; &gt; - &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;not if we've guessed the state to be fully charged */<br>&gt; &gt; - &nbsp; &nbsp; &nbsp; supply-&gt;priv-&gt;enable_poll = (state !=<br>&gt; &gt; UP_DEVICE_STATE_FULLY_CHARGED);<br>&gt; &gt; + &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;not if we've guessed the state to be fully charged. This<br>&gt; &gt; function<br>&gt; &gt; + &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;has been disabled since some battery hardware/drivers are very<br>&gt; &gt; + &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;late to report discharging and does not interrupt on state change<br>&gt; &gt; + &nbsp; &nbsp; &nbsp; */<br>&gt; &gt; + &nbsp; &nbsp; &nbsp; /*supply-&gt;priv-&gt;enable_poll = (state !=<br>&gt; &gt; UP_DEVICE_STATE_FULLY_CHARGED);<br>&gt; &gt; + &nbsp; &nbsp; &nbsp; &nbsp;*/<br>&gt; &gt; +<br>&gt; <br>&gt; You've just commented out the function. Surely finding the bug would<br>&gt; be the right thing to do?<br>&gt; <br>&gt; Richard.<br>&gt; _______________________________________________<br>&gt; devkit-devel mailing list<br>&gt; devkit-devel@lists.freedesktop.org<br>&gt; http://lists.freedesktop.org/mailman/listinfo/devkit-devel<br></div>                                               </div></body>
</html>