[nut-Patches][313910] Add CyberPower OL3000RMXL2U support to powerpanel driver

[nut-patches at alioth.debian.org]

nut-Patches item #313910, was changed at 14/12/2012 23:07 by Arnaud Quette
You can respond by visiting: 
https://alioth.debian.org/tracker/?func=detail&atid=411544&aid=313910&group_id=30602

>Status: Pending
Priority: 3
Submitted By: Timothy Pearson (madscientist-guest)
Assigned to: Arnaud Quette (aquette)
Summary: Add CyberPower OL3000RMXL2U support to powerpanel driver 
Category: Driver fix
Group: None
>Resolution: Fixed


Initial Comment:
This patch adds CyberPower OL3000RMXL2U support to the powerpanel driver.  The following fields are supported:
battery.charge
battery.runtime
battery.voltage
device.mfr
device.model
device.serial
input.frequency
input.frequency.high
input.frequency.low
input.transfer.high
input.transfer.low
input.voltage
input.voltage.nominal
output.frequency
output.voltage
ups.firmware
ups.load
ups.mfr
ups.model
ups.power.nominal
ups.realpower.nominal
ups.serial
ups.status
ups.temperature

----------------------------------------------------------------------

>Comment By: Arnaud Quette (aquette)
Date: 14/12/2012 23:07

Message:
You just did tracked the right patch ;)
Alioth (Fusion Forge) is still a bit rude...

I've committed it as r3829 with a few modifications:
- I've bumped the driver version to 0.26 and added you to the author list,
- I've made a few indentation fixes,
- I've added an entry in the HCL (data/driver.list)

http://trac.networkupstools.org/projects/nut/changeset/3829

----------------------------------------------------------------------

Comment By: Timothy Pearson (madscientist-guest)
Date: 14/12/2012 20:10

Message:
The latest patch is powerpanel_patch.diff

I am not very familiar with Alioth, sorry for the confusion. :-)  For future reference, is there any way to remove an obsolete patch or at least mark it as such so that this does not happen again?

----------------------------------------------------------------------

Comment By: Arnaud Quette (aquette)
Date: 14/12/2012 18:56

Message:
Timothy,

thanks for your patches.
which one is the right one in the end?
I guess cyberpower_olrm_patch.diff, but would like a confirmation

----------------------------------------------------------------------

Comment By: Timothy Pearson (madscientist-guest)
Date: 05/12/2012 03:55

Message:
Use this patch instead of the prior two-patch sequence.

It appears that under  certain conditions the UPS can send out a drastically shortened, yet still valid, status string.  This patch ignores short reads as long as at least one byte was read from the UPS.

----------------------------------------------------------------------

Comment By: Timothy Pearson (madscientist-guest)
Date: 28/11/2012 04:44

Message:
This patch should be applied after cyberpower_olrm_patch.diff; it corrects an issue caused by the fact that this UPS can output a variable-length status string.

----------------------------------------------------------------------

You can respond by visiting: 
https://alioth.debian.org/tracker/?func=detail&atid=411544&aid=313910&group_id=30602