Rockbox

This is a very neat idea. I really don’t think it would be very hard to implement. I’ll take a look later and see if even someone as stupid with coding as me can figure out how to do it.

This is be a great idea as I have read a common fault with iriver H120’s is the eventual breaking on the stop button which is also used as shutdown.

I guess it will also reduce wear on similar buttons on other DAP’s.

Something like this?

   shutdown.patch (8.5 KiB)

Very nice,
(one small issue: the option ‘shut down’ isn’t correct translated to dutch, it says ‘uitgeschakeld’ (what means ‘disabled’) in stead of ‘uitschakelen’ (= shut down)), but nevertheless it looks very good :)

I think it’s LANG_SHUTDOWN in de dutch .lang file (German French and English looks good)

So i’m started with my first rockbox/c programming..
I think i’ve added reboot the patch from obo
It works on my H340, but i can’t test it on other devices
(rolo doesn’t work in the simulator)

   shutdown.patch (8.8 KiB)

patch is not in sync anymore

Sync’d. Change sys_poweroff (and hence clean_shutdown and shutdown_hw) so it can reboot instead of halt - should work on all targets without worrying about the status of ROLO, but maybe a bit OTT.

   20070429-shutdown.patch (14.7 KiB)

Seems out of sync again, at least for the e200. Causes errors when compiling in power_menu.c.

Resync of the patch. The reboot function doesn’t seem to work on my H120 though.

   20070715_shutdown.diff (11.4 KiB)

Resync, and add a “Rebooting…” splash.

#EDIT#

Binary size increase is 472 bytes for a 5g.

   20070729-shutdown.patch (15.1 KiB)

I have found a couple of things wrong here.

1. There is an error with power_menu.c about the lang_shutdown. (I have fixed this in my patch)
2. “Shutting down” is broken when turning off the player. I hope to fix this.
3. I can’t really see the point of “Reboot” Can’t we just add an option “Shuttingdown” to the main menu and forget the other stuff?
That way the patch will be smaller and so will the voice file. Why would someone want too reboot there player?
And then just put the sleep timer back where it was before. This way only one option has been added. Of course we could add a quick menu maybe to the main menu and put “Shutdown” in there. And maybe other options later. But I don’t understand why we need “Reboot” as an option.

So if I remove this will that be ok?

I will post my updated patch soon. Just about done. If you aren’t happy with it let me know and I will maybe change it.

Resync. Add SYS_REBOOT in cases where SYS_POWEROFF is used now to ensure a clean reboot of plugins etc.

As to the point of reboot - it can be useful to get back to the OF, or for a handful of settings that require a reboot of Rockbox to become active. If that is enough to justify the feature or not is another matter.

   20070828-shutdown.patch (19.1 KiB)

Sorry I didn’t get around to updating. But thanks for the update.
I hope to at some stage make the “Rebooting” splash talk. It shouldn’t be too hard I just need to find where the code should go.
But if someone beats me to it I won’t worry about it.
Does the rebooting function work on any players? If so what players. I got the patch working but I needed to remove something like if(reboot) from it so the rebooting didn’t work it just shutdown the player. But if it is working let me know.
Also if you just want a “shutdown” option in the main menu and no reboot option and the sleep timer to stay where it is then try
 FS#7639 : Shutdown item in main menu.
Thanks.

Was getting a hunk error on english.lang kind of resynced. You don’t get a error anymore.

   shutdown-20070829.patch (19 KiB)

Also can this be ported to the gigabeat?

Ok I will see what I can do.
I thought it would work with it. I just quickly looked at the code.
Anyway maybe try
 FS#7639 

Jacob: hunk errors in english.lang aren’t entirely unexpected - any new phrase being added will cause that, just keep moving the ones in this patch to the end. The code /should/ be okay on the Gigabeat - what issues are you seeing? It’s possible that the sys_reboot() function hasn’t been implemented on that platform.

I get the below error when I try to compile.

rolo.c: In function ‘rolo_load’:
rolo.c:330: error: too few arguments to function ‘sys_poweroff’ make[1]: * [/rockbox/build/firmware/rolo.o] Error 1
make: * [build] Error 2

What is line 330 in rolo.c?
Also what happens if you build for the h300?

Jacob: On a clean checkout of SVN, with only this patch applied, I have no problems building for a Gigabeat. firmware/rolo.c is untouched by this patch, (at 328 lines long) so it seems you have one or more other patches applied?

It must be the below patch. It is a Gigabeat fast restart.

http://www.rockbox.org/tracker/task/7113

Change the sys_poweroff() added in the fake-rolo patch to sys_poweroff(false)

Resynced

   shutdown-20071018.patch (18.8 KiB)

Disregard the above patch it won’t compile if applied.

Please resync the patch,
it’s usefull

Yes, please can someone look into getting this patch to work with SVN.

Sync’d (and the shutdown and reboot options appear under the System menu…)

   20080116-shutdown.patch (15.6 KiB)

Thank you very much

Now if we could the fast reboot patch synced it would be fantastic.

The shutdown option works but when you select reboot it shuts off and never comes back on even when you try pressing the power button. To turn it back on you have to turn off/on the battery switch and then press the power button.

patch synced with current svn & it use to always work with the iriver h10 [20GB] from my end thank you.

   shutdown060508.patch (15.6 KiB)

patch break during build process at recording.c

Just a suggestion/request, it would be great if both FS7639 and FS6733 patches were combined to have “Shutdown” in the main menu, “Restart” in the System menu and leave Sleep Timer where it was originally (basically remove that portion of the patch). I’ve used both patches with tdtooke’s Underground build for ipods and these features are very useful!

Resync

   shutdown-080520.patch (15.9 KiB)

this patch still fails to compile at powermgt.c

Try this one

   shutdown-080611.patch (15.5 KiB)

it still fails at recording.c says too few parameters in function “sys_poweroff”

It compiles and runs fine on my player and the sim.

hey im using it on the h10[20GB], would that make any difference ?

Actually from looking at the patch file this doesn’t even touch recording.c
But if you change sys_poweroff() to sys_poweroff(false) it compiles.

Hi. like everyone else, I can’t compile with this patch applied, i’m running an h140, and I can’t compile either with the sim build or the target build.

[quote] But if you change sys_poweroff() to sys_poweroff(false) it compiles.[/quote]

hey yes this works fine compiled at svn.17873, compiles perfect for h10[20GB]

Could a version of this patch be uploaded that includes the change? as i can’t see what bit of code to change, and i’m running the last version uploaded.

Synced.

   20080629-shutdown.patch (16.1 KiB)

added menu to japanese.lang.

   20080820-shutdown.patch (17.3 KiB)

Synced. shutdown-only.patch only enables the existing Archos Player & Recorder shutdown menu entry for all targets. RAM and binsize cost are 128 bytes. shutdown.patch adds a reboot option - RAM cost is 496 bytes, binsize is 480.

   20081202-shutdown.patch (16.3 KiB)
   20081202-shutdown-only.patch (1.9 KiB)

I would like it very much if the ‘shutdown only’ patch would be included into the official build.

hmm.. the shutdown (+) (not shutdown only), does not build properly.. MAy someone please resync it please?

Some things have changed in the destination files. I made some changes in order to apply the patch. I don’t know if I made the right changes but it is working in my Ipod ver 5.5 80GB

   5_20081202-shutdown.patch (16 KiB)

Yes! Thank you! Works pefectly.. The other one was intrusive, this one is perfect… Thanks again!

Could some one re sync this? Please?

Its working fine for me