Sansа Clip Zip - Improve battery life

[rbert]

Does anyone want to try raising the voltages one by one and see which one was set too low?

OK, but it would be helpful, if I had your source code changes.
I tried setting CVDD2 and CVDD1 by changing two lines in system-as3525.c and I still had crashes.
Here's what I tried:

Code: [Select]

/* Set CVDD2 power supply for fuzev2 */
ascodec_write_pmu(0x17, 2, 0x80 | 116);


Code: [Select]

/* Set CVDD1 power supply for fuzev2 */
ascodec_write_pmu(0x17, 1, 0x80 | 47);

[saratoga]

All I did was comment out the voltage reduction on unboost.

Edit CPU voltage, cvdd1.

[PagaN]

Latest build with all patches for clip zip: http://gdurl.com/c_OQ/download

tested this (f65baf9M-150129) firmware on my 2 year used ClipZip 4gb 100% charged,
16gb MicroSD full of mp3 files on standard sansa headphones volume (-15dB),

I get best result 18:29h and no errors! very unstable (next day I walk on street 1:30h and it freeze and give error 5-6 times) reverted on last devbuild.

[rbert]

All I did was comment out the voltage reduction on unboost.

Edit CPU voltage, cvdd1.

I can't set CVDD1 on unboost without crashing my fuzev2 in main menu.
But if I comment that line out, everything works fine.

I tried most voltages between

Code: [Select]

ascodec_write_pmu(0x17, 1, 0x80 | 20);and

Code: [Select]

ascodec_write_pmu(0x17, 1, 0x80 | 47);

[Mihail Zenkov]

Very strange. What clock setting you have in "system > debug >  view hw info" ?

[rbert]

Very strange. What clock setting you have in "system > debug >  view hw info" ?

[Mihail Zenkov]

rbert:
Can you try force normal (unboost) mode - replace  if(frequency == CPUFREQ_MAX) to if(0) and try find lower stable CVDD1?

[rbert]

still no luck, if I try to set CVDD1 during unboosted mode

[Mihail Zenkov]

Even if you set ascodec_write_pmu(0x17, 1, 0x80 | 47); ?

[rbert]

Even if you set ascodec_write_pmu(0x17, 1, 0x80 | 47); ?

yes, exactly
something's strange

[Mihail Zenkov]

Yes, very strange. Can you try  0x80 | 47 + attached patch?

EDIT: wrong patch, reattached.

[rbert]

Yes, very strange. Can you try  0x80 | 47 + attached patch?

Thank you very much, it works.
Now I can test lower voltages.

EDIT:
no problems with 0x80 | 20
I can now confirm, that my fuzev2 variant0 works like a charm with
- Patch Set 7 471ced2d0f1df1f6e53824197c93e52563fdac3a
- rockbox-restore_irq.patch.txt
- the same changes in firmware/export/config/sansafuzev2.h as in the other config files

EDIT2:
for anyone else to test:
http://gregory.bplaced.net/rockbox/rockbox-fuzev2-1.7z

[saratoga]

Yes, very strange. Can you try  0x80 | 47 + attached patch?

EDIT: wrong patch, reattached.

That looks like an interesting race condition indeed. 

Edit:  Does it make sense to also protect the ascodec_read_pmu call from an IRQ?  Seems like both could be corrupted.

[Mihail Zenkov]

I read IRC and I agree with jhMikeS - we should try fix it deeper.  But it to hard for me

Zip also have problem with ascodec_write_pmu - I do this changes in patch V4, but I could not reproduce this behavior again and revert in V5.

[jhMikeS]

Okiedokie: http://gerrit.rockbox.org/1145/

Whatever the case if it's hard to reproduce, the driver has a problem that this fixes.