Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#5639 - Improved battery life on X5 (switching off USBOTG)

Attached to Project: Rockbox
Opened by Ulrich Pegelow (upegelow) - Sunday, 09 July 2006, 17:55 GMT+2
Task Type Patches
Category Battery/Charging
Status Closed
Assigned To No-one
Player Type iAudio X5
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Hi,

this is a patch that switches off the M5636 USBOTG device on iAudio X5.
Battery drain is reduced by ca. 50mA. On a X5L this makes a runtime of typically
24 hours instead of the current 15 hours.

The patch is based on reverse engineering of the OF. Unwanted side effects can not
be excluded. USE AT YOUR OWN RISK.

Ulrich
   x5_USBOTG_off.patch (8.3 KiB) 
 rockbox.new/apps/main.c                       |    6 +
 rockbox.new/firmware/SOURCES                  |    4 -
 rockbox.new/firmware/drivers/m5636.c          |   86 ++++++++++++++++++++++++++
 rockbox.new/firmware/export/config-iaudiox5.h |    3 
 rockbox.new/firmware/export/config.h          |    1 
 rockbox.new/firmware/export/m5636.h           |   62 ++++++++++++++++++
 6 files changed, 161 insertions(+), 1 deletion(-)

This task depends upon

Closed by  Linus Nielsen Feltzing (linusnielsen)
Thursday, 13 July 2006, 09:41 GMT+2
Reason for closing:  Accepted
Comment by Linus Nielsen Feltzing (linusnielsen) - Tuesday, 11 July 2006, 11:01 GMT+2
Good work!

I took the liberty of adjusting your patch to the target tree concept. Please review and test, and then I will commit it.
   x5_usbotg_v2.patch (10 KiB) 
 apps/main.c                                       |    6 ++
 docs/CREDITS                                      |    1 
 firmware/SOURCES                                  |    5 +-
 firmware/drivers/m5636.c                          |   45 +++++++++++++++++++
 firmware/export/config-iaudiox5.h                 |    3 +
 firmware/export/config.h                          |    1 
 firmware/export/m5636.h                           |   25 +++++++++++
 firmware/target/coldfire/iaudio/x5/m5636-target.h |   37 ++++++++++++++++
 firmware/target/coldfire/iaudio/x5/m5636-x5.c     |   50 ++++++++++++++++++++++
 9 files changed, 172 insertions(+), 1 deletion(-)

Comment by Ulrich Pegelow (upegelow) - Tuesday, 11 July 2006, 20:53 GMT+2
Looks reasonable at a first glance. Nevertheless I'd like to check it. Will take a few days due to other obligations.
Comment by Ulrich Pegelow (upegelow) - Thursday, 13 July 2006, 08:26 GMT+2
I checked it: patches fine, compiles fine and runs fine. Most important: the energy saving is as expected.

Revised version hereby approved!

Loading...