Bug 678793

Summary: [PATCH] add support for more Matrix-UPS revisions to apcsmart driver
Product: [Fedora] Fedora Reporter: Charles R. Anderson <cra>
Component: nutAssignee: Michal Hlavinka <mhlavink>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: unspecified    
Version: rawhideCC: dwmw2, mhlavink
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-09-04 18:12:06 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Add support for Matrix-UPS firmware 0XM/0XI with additional commands none

Description Charles R. Anderson 2011-02-19 18:44:26 UTC 
Created attachment 479713 [details]
Add support for Matrix-UPS firmware 0XM/0XI with additional commands

Description of problem:
The attached patch adds support for Matrix-UPS revision strings 0XI and 0XM to the apcsmart driver.

The revision string itself isn't completely opaque--it is made up of different parts as documented on http://homepage1.nifty.com/Que/plamo/apc-ups/manual/upsbible.html under "Old Firmware Revision".  This implies that many more combinations of characters could be added to support more revisions & configurations.  Indeed, 0XI and 0XM only differ by the placement of the 208V/240V input voltage select wire.

The first character can be 0 or 5 for Matrix-UPS 3000 vs. Matrix-UPS 5000.  The second character is the actual firmware revision.  I've read in various forums about revisions U, W, X, Y, and Z.  There is probably a V as well.  The last character seems to be I for 240V and M for 208V, which I can change on mine by the placement of the input voltage select wire. 

I've also tested the expanded command list which enables nut to access more functionality on my Matrix-UPS 3000.  I suggest that this same expanded list could be applied to all the other firmware combinations as well.

Please consider adding at least this patch which is limited to 0XI/0XM initially.  Thanks.

Version-Release number of selected component (if applicable):
nut-2.6.0-1.fc14
Comment 1 Michal Hlavinka 2011-02-21 14:02:31 UTC 
Thanks for reporting this. I can see you've already sent this patch to upstream mailing list which is the correct way how get patch applied in nut. I'll add this patch to our nut package once upstream accepts it.
Comment 2 Charles R. Anderson 2011-09-04 18:12:06 UTC 
I see this has been added.  Closing.  Thanks.