Bug 14076 - Add more diagnostics to the diagnostics
: Add more diagnostics to the diagnostics
Status: CLOSED FIXED
Product: SB Radio
Classification: Unclassified
Component: Diagnostics/Tests
: Include FW version in comment
: PC Other
: P1 normal (vote)
: 7.4.0
Assigned To: Wadzinski Tom
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-09-15 23:03 UTC by Caleb Crome
Modified: 2009-10-05 14:33 UTC (History)
5 users (show)

See Also:
Category: ---


Attachments
patch (1.14 KB, patch)
2009-09-24 08:52 UTC, Wadzinski Tom
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Caleb Crome 2009-09-15 23:03:50 UTC
I think the following could be useful for in-field debugging of battery problems:

MSP version (from /sys/bus/i2c/devices/1-0010/fw)
battery voltages from same path:  battery_voltage, battery_vmon1, battery_vmon2, wall_voltage, battery_temperature, power_mode, charger_state
Comment 1 James Richardson 2009-09-16 06:04:50 UTC
Tom, is this yours to implement?  feel free to adjust the priority/target
Comment 2 Wadzinski Tom 2009-09-16 06:14:31 UTC
Sure, I'll take it. My thinking is to a a battery submenu off the main diagnostics, since the main list is getting pretty long. 

So I taking from this:
a) one top level param for MSP version
b) a battery submenu with 7 params: battery_voltage, battery_vmon1,
battery_vmon2, wall_voltage, battery_temperature, power_mode, charger_state
Comment 3 SVN Bot 2009-09-19 05:13:24 UTC
 == Auto-comment from SVN commit #7664 to the jive repo by tom ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7664 ==

Bug: 14076
- Add msp version and power submenu
Comment 4 Wadzinski Tom 2009-09-19 05:16:08 UTC
All params have been added in r7644.
I wonder if we can massage/map the battery values any to make them more useful, 
for instance, wall voltage comes up as 18253 and power mode is "3" and charge 
state is "1"
Comment 5 Caleb Crome 2009-09-19 09:31:45 UTC
Sure, I'll take a look.
Comment 6 SVN Bot 2009-09-20 14:36:33 UTC
 == Auto-comment from SVN commit #7668 to the jive repo by ccrome ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7668 ==

Fixed #14076.  Prettified the power diags.
Comment 7 Wadzinski Tom 2009-09-24 08:48:51 UTC
MSP version is baby only, and should be inside Battery
Comment 8 Wadzinski Tom 2009-09-24 08:52:20 UTC
Created attachment 5916 [details]
patch
Comment 9 Wadzinski Tom 2009-09-24 08:52:51 UTC
Assigned to Richard to review attached patch
Comment 10 Richard Titmuss 2009-09-24 08:54:22 UTC
patch acked.
Comment 11 SVN Bot 2009-09-24 08:58:10 UTC
 == Auto-comment from SVN commit #7741 to the jive repo by tom ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7741 ==

Fixed Bug: 14076
Description:
- moved msp version inside power menu.
Comment 12 James Richardson 2009-10-05 14:33:32 UTC
This bug has been marked as fixed in the 7.4.0 release version of SqueezeBox Server!
    * SqueezeCenter: 28672
    * Squeezebox 2 and 3: 130
    * Transporter: 80
    * Receiver: 65
    * Boom: 50
    * Controller: 7790
    * Radio: 7790  

Please see the Release Notes for all the details: http://wiki.slimdevices.com/index.php/Release_Notes

If you haven't already, please download and install the new version from http://www.logitechsqueezebox.com/support/download-squeezebox-server.html

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.