Sansа Clip Zip - Improve battery life

[Mihail Zenkov]

Plus the fact that it happens on the Fuze v2 but not Zip/Clip+

As I mention before Zip also affected but in another case. If I remember right, it was when I try change voltage on pause/play.

[saratoga]

But since it depends on when you do it, that sounds like more of a conflict with the button driver I think?

[Mihail Zenkov]

Not sure but possibly. What about clipv2 problem? Can you test it with my "irq" patch or you already have solution for it?

[johnb]

I am using the one posted by rbert in
http://forums.rockbox.org/index.php/topic,48549.msg233959.html#msg233959
http://gregory.bplaced.net/rockbox/rockbox-fuzev2-1.7z

471ced2M-150209

Runtime: 26h 20min
without SD card.
Album with lame v2 (avg. 160 kbps).

Original ear buds.

Code: [Select]

# .cfg file created by rockbox 471ced2M-150209 - http://www.rockbox.org

volume: -30
repeat: all
backlight timeout: 7
volume display: numeric
battery display: numeric
show icons: off
replaygain noclip: on
crossfeed: meier
button light timeout: off

I bought the fuze ~5 years ago as a refurb, so the battery is somewhat aged, but I don't use it regularly. Working horse for commuting is a clip+.

If I remember correctly runtime with the 3.13 release (more than a year ago) was ~20h.

[404_user_not_found]

Tested with the new fresh sansa clip+. Unstable with current patch set. Solved problem by increasing CVDD2 to 114.

[grums]

Which patch (link?) do I apply on a Clip+ daily build to see the effect of these developments?

[saratoga]

The latest in the gerrit task I believe, although you'll have to remove the changes to as3514.c as those are already committed. 

[johnb]

Updated builds:

http://web.mit.edu/mgg6/www/rockbox-clipv2.7z
http://web.mit.edu/mgg6/www/rockbox-clipplus.7z

Is this compile the most optimized for clip+ as of today?

[saratoga]

Updated builds:

http://web.mit.edu/mgg6/www/rockbox-clipv2.7z
http://web.mit.edu/mgg6/www/rockbox-clipplus.7z

Is this compile the most optimized for clip+ as of today?

No it doesn't have the DMA changes or the I2C fix.  I haven't had time to work on this lately though, so you'll have to grab the patches and build yourself. 

[g.orlandini]

Hi all,

I'm trying to patch last build e606ee4 for my clip+ with patch 7 (http://git.rockbox.org/?p=rockbox.git;a=patch;h=471ced2d0f1df1f6e53824197c93e52563fdac3a;hp=193c5df75d3b6d9e3442e42fa26fd8ccc4c5e3aa) to improve battery life, but I get the following:

patch --dry-run -p1 < "rockbox.git-471ced2d0f1df1f6e53824197c93e52563fdac3a.patch"
checking file firmware/drivers/audio/as3514.c
Reversed (or previously applied) patch detected!  Assume -R? [n]
Apply anyway? [n]
Skipping patch.
6 out of 6 hunks ignored
checking file firmware/export/config/sansaclipplus.h
Hunk #2 succeeded at 194 (offset -1 lines).
checking file firmware/export/config/sansaclipv2.h
checking file firmware/export/config/sansaclipzip.h
Hunk #2 succeeded at 193 (offset -1 lines).
checking file firmware/rolo.c
checking file firmware/target/arm/as3525/clock-target.h
checking file firmware/target/arm/as3525/system-as3525.c
checking file firmware/target/arm/as3525/usb-as3525.c
Hunk #1 FAILED at 33.
1 out of 1 hunk FAILED

Is the patch I'm applying not compatible with the last build e606ee4 or I'm doing something wrong? Where can I download a patched build for clip+ ?

Thanks a lot for your help

[saratoga]

I quickly synced that file (was only one line changed).  I can't login into gerrit at the moment, but here is the patch:

http://mit.edu/mgg6/www/amsv2_scaling_v8.patch

[saratoga]

I also committed the DMA patch, so that should give everyone a very slightly lower CPU usage.  It works for me and in theory is correct, but I'll watch for any problems.

[Mihail Zenkov]

I also committed the DMA patch, so that should give everyone a very slightly lower CPU usage.

Good! Can you also review/commit two bug fix:
http://gerrit.rockbox.org/r/#/c/1190/
http://gerrit.rockbox.org/r/#/c/1146/

[saratoga]

I also committed the DMA patch, so that should give everyone a very slightly lower CPU usage.

Good! Can you also review/commit two bug fix:
http://gerrit.rockbox.org/r/#/c/1190/
http://gerrit.rockbox.org/r/#/c/1146/

Yes I'll commit those shortly.  I can't login into gerrit though, so I can't close the tasks.

[g.orlandini]

I quickly synced that file (was only one line changed).  I can't login into gerrit at the moment, but here is the patch:

http://mit.edu/mgg6/www/amsv2_scaling_v8.patch

Thanks a lot for your patch, Saratoga. Until now it works very well.