Rockbox.org home
Downloads
Release release
Dev builds dev builds
Extras extras
themes themes
Documentation
Manual manual
Wiki wiki
Device Status device status
Support
Forums forums
Mailing lists mailing lists
IRC IRC
Development
Bugs bugs
Patches patches
Dev Guide dev guide
Search




Rockbox Technical Forums


Login with username, password and session length
Home Help Search Staff List Login Register
News:

Welcome to the Rockbox Technical Forums!

+  Rockbox Technical Forums
|-+  Support and General Use
| |-+  Hardware
| | |-+  Sansа Clip Zip - Improve battery life
« previous next »
  • Print
Pages: 1 ... 17 18 [19] 20 21 22

Author Topic: Sansа Clip Zip - Improve battery life  (Read 81626 times)

Offline Mihail Zenkov

  • Developer
  • Member
  • *
  • Posts: 374
Re: Sansа Clip Zip - Improve battery life
« Reply #270 on: January 13, 2016, 11:24:35 AM »
fuzpa.bin - bootloader + original firmware, you should update it only if you need changes in bootloader or need new original firmware. Don't update it without good reasons - you can brick player if something going wrong.
For regular update to new rockbox revision - just replace .rockbox/* .
For small internal changes (like you doing with this patch) just replace one file - .rockbox/rockbox.sansa
Logged

Offline oid_maps

  • Artist
  • Member
  • *
  • Posts: 31
Re: Sansа Clip Zip - Improve battery life
« Reply #271 on: January 13, 2016, 11:44:44 AM »
Quote from: Mihail Zenkov on January 13, 2016, 11:24:35 AM
fuzpa.bin - bootloader + original firmware, you should update it only if you need changes in bootloader or need new original firmware. Don't update it without good reasons - you can brick player if something going wrong.
For regular update to new rockbox revision - just replace .rockbox/* .
For small internal changes (like you doing with this patch) just replace one file - .rockbox/rockbox.sansa

Ah OK, thanks. But why then is the bootloader 15 MB big? So it's much more than just a loader. Or does it still have the original firmware for some reason -- which just doesn't get executed?
Logged
My audio players:
* Olympus WS-811 (no rockbox )-:)
* Sansa Fuze v2 (rockbox (git build) (-:)

Offline saratoga

  • Developer
  • Member
  • *
  • Posts: 8761
Re: Sansа Clip Zip - Improve battery life
« Reply #272 on: January 13, 2016, 12:05:38 PM »
The bootloader is a few kilobytes, the rest is the OF.
« Last Edit: January 14, 2016, 01:36:59 PM by saratoga »
Logged

Offline saratoga

  • Developer
  • Member
  • *
  • Posts: 8761
Re: Sansа Clip Zip - Improve battery life
« Reply #273 on: January 14, 2016, 01:38:56 PM »
So if the Fuze v2 is fixed by delay changes (???) the only device that absolutely does not work is the Clipv2. 
Logged

Offline Mihail Zenkov

  • Developer
  • Member
  • *
  • Posts: 374
Re: Sansа Clip Zip - Improve battery life
« Reply #274 on: January 14, 2016, 03:17:59 PM »
Probably we have similar problem with hold/power buttons in clipv2. Can you test attached patch? If it work - try remove second delay.
* fix_button_clipv2.patch.txt (0.62 kB - downloaded 73 times.)
Logged

Offline saratoga

  • Developer
  • Member
  • *
  • Posts: 8761
Re: Sansа Clip Zip - Improve battery life
« Reply #275 on: January 14, 2016, 06:10:57 PM »
Adding the delay fixes boosting (well played for a few minutes at least).

Removing the second delay causes immediate reboot. 

I did notice that during frequent boosting some button inputs are skipped, although maybe that is unrelated.

Edit:  Both delays set to 10 works pretty well, will try other values:

http://mit.edu/mgg6/www/rockbox-clip_10_10delay.7z

Edit2:  diff:  http://mit.edu/mgg6/www/amsv2_scaling_v10.patch
« Last Edit: January 15, 2016, 03:10:39 PM by saratoga »
Logged

Offline oid_maps

  • Artist
  • Member
  • *
  • Posts: 31
fuzev2 optimised button delay values - Re: Sansа Clip Zip - Improve battery life
« Reply #276 on: January 15, 2016, 07:52:00 AM »
Quote from: Mihail Zenkov on January 04, 2016, 04:50:31 AM
First, try find lower value for first delay. Then try same for second delay.

So, I narrowed the things down to get to know the shortest possible delays where the hold-switch still works on my Sansa fuzev2.

Here are the results:


+-------------+--------------+--------+
| First delay | Second delay | works? |
+-------------+--------------+--------+
|          20 |           10 |  yes   |
+-------------+--------------+--------+
|          18 |           09 |   no   |
+-------------+--------------+--------+
|          19 |           10 |  yes   |
+-------------+--------------+--------+
|          18 |           40 |   no   |
+-------------+--------------+--------+
|          20 |           05 |  yes   |
+-------------+--------------+--------+
|          19 |           05 |  yes   |
+-------------+--------------+--------+
|          19 |           01 |   no   |
+-------------+--------------+--------+
|          19 |           03 |   no   | (first, seemed to work, but every few seconds flickers very short)
+-------------+--------------+--------+
|          19 |           02 |   no   |
+-------------+--------------+--------+
|          19 |           04 |  yes   |
+-------------+--------------+--------+


So, shortest possible delays which still work for my device:
1. udelay(): 19, 2. udelay(): 4.

Corresponding patch attached.

@ Mihail Zenkov: Can you include that one in your patchset?

And: Is there a way to make this patches upstream? If not, it would be nice to host it somewhere (that people do not have to dig through all the forum posts to find the correct versions), if hosting is not possible and we have to resort to the forum, then putting everything into one file (.tar-archive, or a single patch file) makes things still more easy.


EDIT 2016-01-15, modified on 2016-01-17: First I thought, first delay 19, second delay 03 are good. But then I noticed very short flickering-on of the screen every few seconds, so I abandoned it.

EDIT 2016-01-17: How I tested: Just switching on the device and see if in "hold"-position screen stays dark.
Are there more/ special conditions I should test?

* fix_hold_fuzev2-2.delaysoptimised.patch.txt (0.55 kB - downloaded 69 times.)
« Last Edit: January 17, 2016, 04:39:40 AM by oid_maps »
Logged
My audio players:
* Olympus WS-811 (no rockbox )-:)
* Sansa Fuze v2 (rockbox (git build) (-:)

Offline oid_maps

  • Artist
  • Member
  • *
  • Posts: 31
Re: Sansа Clip Zip - Improve battery life
« Reply #277 on: January 15, 2016, 07:54:31 AM »
Quote from: Mihail Zenkov on January 13, 2016, 11:24:35 AM
fuzpa.bin - bootloader + original firmware, you should update it only if you need changes in bootloader or need new original firmware. Don't update it without good reasons - you can brick player if something going wrong.
For regular update to new rockbox revision - just replace .rockbox/* .
For small internal changes (like you doing with this patch) just replace one file - .rockbox/rockbox.sansa

Well, but I observe that also when compiling the bootloader, the file 'button-fuzev2.c' get's compiled:

Code: [Select]
$ ./configure --no-ccache --target=63 --type=b
  [...]
$ make
  [...]
  CC firmware/target/arm/as3525/sansa-fuzev2/button-fuzev2.c
  [...]

So, what's the reason that the bootloader takes into account that file?
« Last Edit: January 15, 2016, 08:24:43 AM by oid_maps »
Logged
My audio players:
* Olympus WS-811 (no rockbox )-:)
* Sansa Fuze v2 (rockbox (git build) (-:)

Offline oid_maps

  • Artist
  • Member
  • *
  • Posts: 31
Re: Sansа Clip Zip - Improve battery life
« Reply #278 on: January 15, 2016, 07:56:56 AM »
Quote from: saratoga on January 13, 2016, 12:05:38 PM
The bootloader is a few kilobytes, the rest is the OF.

.. and we still need to append the original firmware when we want to change the bootloader, right? (That's what is documented -- to use "mkamsboot" to patch the original firmware with the rockbox bootloader) Why is it not sufficient to just have a small bootloader ("fuzpa.bin") without the original firmware?

Does it fallback to the original firmware automagically if we don't have the .rockbox-tree on the device?
Logged
My audio players:
* Olympus WS-811 (no rockbox )-:)
* Sansa Fuze v2 (rockbox (git build) (-:)

Offline oid_maps

  • Artist
  • Member
  • *
  • Posts: 31
Re: fuzev2 optimised button delay values - Re: Sansа Clip Zip - Improve battery life
« Reply #279 on: January 15, 2016, 07:59:53 AM »
Quote from: oid_maps on January 15, 2016, 07:52:00 AM
Quote from: Mihail Zenkov on January 04, 2016, 04:50:31 AM
First, try find lower value for first delay. Then try same for second delay.

So, I narrowed the things down to get to know the shortest possible delays where the hold-switch still works on my Sansa fuzev2.

[...]

So, shortest possible delays which still work for my device:
1. udelay(): 19, 2. udelay(): 5. (19, 4 still needs to be tested!)

Corresponding patch attached.

A user asked me if I can provide a working build. The forum does not allow to upload archive files. Is there a way I can upload it somewhere to a subpage of rockbox.org? I feel uncomfortable with hosting it by myself ... I did not dig into the legal issues yet (impressum of the website would be needed, warranty-disclaimer maybe, maybe some attribition to property rights I need to make).

Otherwise, I would email the .tar.xz-archive if people ask me directly (private message, and include your email address).
« Last Edit: January 15, 2016, 08:57:45 AM by oid_maps »
Logged
My audio players:
* Olympus WS-811 (no rockbox )-:)
* Sansa Fuze v2 (rockbox (git build) (-:)

Offline Mihail Zenkov

  • Developer
  • Member
  • *
  • Posts: 374
Re: Sansа Clip Zip - Improve battery life
« Reply #280 on: January 15, 2016, 09:03:36 AM »
Quote from: saratoga on January 14, 2016, 06:10:57 PM
I did notice that during frequent boosting some button inputs are skipped, although maybe that is unrelated.

For fuse v2 we disable irq when check hold/power buttons, probably we should do same for clip v2.
* fix_button_clipv2_irq.patch.txt (0.69 kB - downloaded 59 times.)
Logged

Offline Mihail Zenkov

  • Developer
  • Member
  • *
  • Posts: 374
Re: fuzev2 optimised button delay values - Re: Sansа Clip Zip - Improve battery life
« Reply #281 on: January 15, 2016, 09:12:22 AM »
Quote from: oid_maps on January 15, 2016, 07:52:00 AM
So, I narrowed the things down to get to know the shortest possible delays where the hold-switch still works on my Sansa fuzev2.

Here are the results:

Ok, thanks!

Quote from: oid_maps on January 15, 2016, 07:52:00 AM
@ Mihail Zenkov: Can you include that one in your patchset?

Yes, it should by included in next update.

Quote from: oid_maps on January 15, 2016, 07:52:00 AM
And: Is there a way to make this patches upstream?

We should fix all know issues before.

Quote from: oid_maps on January 15, 2016, 07:52:00 AM
If not, it would be nice to host it somewhere (that people do not have to dig through all the forum posts to find the correct versions), if hosting is not possible and we have to resort to the forum, then putting everything into one file (.tar-archive, or a single patch file) makes things still more easy.

Probably you right and we should provide single patch for user.
Logged

Offline oid_maps

  • Artist
  • Member
  • *
  • Posts: 31
Re: Sansа Clip Zip - Improve battery life
« Reply #282 on: January 15, 2016, 09:15:39 AM »
Quote from: saratoga on January 14, 2016, 06:10:57 PM
Edit:  Both delays set to 10 works pretty well, will try other values:

OK, for me first delay set to 18 and second to 09, but also second to 40, didn't even work (as well as 10, 05 did not work, neither did 12, 06; 14, 07; 16, 08). So maybe either the individual devices differ, or 10, 10 is a combination which would work again? Strange.

Cannot test right now.
Logged
My audio players:
* Olympus WS-811 (no rockbox )-:)
* Sansa Fuze v2 (rockbox (git build) (-:)

Offline Mihail Zenkov

  • Developer
  • Member
  • *
  • Posts: 374
Re: Sansа Clip Zip - Improve battery life
« Reply #283 on: January 15, 2016, 09:21:32 AM »
Quote from: oid_maps on January 15, 2016, 07:54:31 AM
Well, but I observe that also when compiling the bootloader, the file 'button-fuzev2.c' get's compiled:

So, what's the reason that the bootloader takes into account that file?

We check button state in bootloader - to boot in original firmware or boot recovery mode.

Quote from: oid_maps on January 15, 2016, 07:56:56 AM
Why is it not sufficient to just have a small bootloader ("fuzpa.bin") without the original firmware?

We can't do separate update for bootloader and original firmware.

Quote from: oid_maps on January 15, 2016, 07:56:56 AM
Does it fallback to the original firmware automagically if we don't have the .rockbox-tree on the device?

No, it fallback to recovery mode (usb flash drive).
Logged

Offline [Saint]

  • Rockbox Expert
  • Member
  • *
  • Posts: 1662
  • Hayden Pearce
    • Google+
Re: Sansа Clip Zip - Improve battery life
« Reply #284 on: January 15, 2016, 01:06:42 PM »
Quote from: oid_maps on January 15, 2016, 07:59:53 AM
A user asked me if I can provide a working build. The forum does not allow to upload archive files. Is there a way I can upload it somewhere to a subpage of rockbox.org? I feel uncomfortable with hosting it by myself ... I did not dig into the legal issues yet (impressum of the website would be needed, warranty-disclaimer maybe, maybe some attribition to property rights I need to make).

Otherwise, I would email the .tar.xz-archive if people ask me directly (private message, and include your email address).


Why would you not simply host it on one of the many sites like Data File Host or so?

The only real requirement you _must_ undertake is that you need to be able to supply the full source code that generated that binary when asked to do so, as a requirement of the GPL license (note: technically speaking just supplying a patch file and pointing to the Rockbox sources and a given commit that the binary was built from is not actually good enough to satisfy this requirement).


[Saint]
Logged
Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

  • Print
Pages: 1 ... 17 18 [19] 20 21 22
« previous next »
+  Rockbox Technical Forums
|-+  Support and General Use
| |-+  Hardware
| | |-+  Sansа Clip Zip - Improve battery life
 

  • SMF 2.0.6 | SMF © 2013, Simple Machines
  • XHTML
  • RSS
  • WAP2

Page created in 0.109 seconds with 67 queries.