<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">On Mar 18, 2015, at 3:57 AM, Philip Taylor <<a href="mailto:philip@kelsotowers.co.uk">philip@kelsotowers.co.uk</a>> wrote:<br><div><br class="Apple-interchange-newline"><blockquote type="cite"><meta http-equiv="Content-Type" content="text/html charset=windows-1252"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Charles, to your specific points :<div class=""><br class=""></div><div class=""><blockquote type="cite" class=""><span class="" style="display: inline !important;">The other HID drivers seem to trigger shutdown by writing to the various timers (e.g. UPS.PowerSummary.DelayBeforeReboot). Your firmware seems to have UPS.PowerSummary.PresentStatus.ShutdownRequested, which according to the HID PDC spec, is read/write. I wonder if the firmware responds to this?</span></blockquote><br class=""></div><div class="">So are you suggesting if I modify openups-hid.c to include this specific command as an instant command, it may respond? I could give it a try if you confirm that’s what you mean.</div></div></blockquote><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class=""><br class=""></div><div class="">It's a thought. Looking further into the code, I'm not sure how well usbhid-ups will handle writing to a single bit, since the reports are collections of bytes. And it would depend how the firmware interprets it (if it is writable at all).</div><div class=""><br></div></div><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class=""><blockquote type="cite" class=""><span class="" style="display: inline !important;">Out of curiosity, which notifications? In the openups_drv_start_onbatt log, the PresentStatus values look the same as the online log. Or do you mean you got that from the ignorelb configuration?</span></blockquote><br class=""></div><div class="">The notifications are : ONLINE ONBATT FSD COMMOK COMMBAD SHUTDOWN (following FSD from command line). I believe it notified LOWBATT too when I used ignoreLB and over-rode battery.charge.low. But I’ve never seen the OpenUPS notify LB via USB. If I use the scenario I proposed I would monitor battery.voltage by polling the online log.</div></div></blockquote><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class=""><br class=""></div><div class="">Maybe it took a little while longer to get the ONBATT status; in both of the logs you sent, ACPresent is 0.01 (more scaling issues, but the code only cares about zero or nonzero).</div><div class=""><br></div></div><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">I’m a little surprised that I seem to be the only person interested in this issue - as some others were interested in the OPENUPS2, in fact you added it to the ‘supported devices’ list fairly recently. I think the inclusion of the OpenUPS as ‘fully supported’ in the hardware compatibility list over-states the capabilities!</div></div></blockquote><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class=""><br></div><div class="">Well, we did add the openups2 recently, but it was mainly because it requires a different set of scale values to fix the voltage and current readings.</div><div class=""><br></div><div class="">"Supported" is an overloaded term. I would agree that given what we know now, the Hardware Compatibility List (HCL) rating of five stars for the openUPS is a bit optimistic. But that was updated in 2012 when the first patches came in, and it was more of a tip-o-the-hat to Mini-Box for providing the driver for NUT.</div><div class=""><br></div><div class="">The idea with the Device Dump Library (DDL) was to provide more quantitative information as to what people can expect from a given combination of UPS and NUT: <a href="http://buildbot.networkupstools.org/~buildbot/cayman/docs/latest/ddl/Mini-Box.com/OPEN-UPS.html">http://buildbot.networkupstools.org/~buildbot/cayman/docs/latest/ddl/Mini-Box.com/OPEN-UPS.html</a></div><div class=""><br></div><div class="">We might consider adding a warning if the UPS does not support basic commands such as shutdown, although we can't do that automatically now, since some of the older dump files simply do not contain the list of commands (even if they support them) since they were collected from mailing list posts.</div><div class=""><br></div></div><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Thanks once again., you have been most helpful.</div></div></blockquote><div><br></div>You're welcome!</div><br><div>
<span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px;"><div>-- </div><div>Charles Lepple</div><div>clepple@gmail</div><div><br></div></span><br class="Apple-interchange-newline">

</div>
<br></body></html>