Details

I think it would be nice if there is an option/submenu shutdown in the main menu.
This can contain the following things
- Shutdown player
- Reboot player (execute the rockbox.iriver file)
- Sleep timer (which is now under info)

I'm owning the iRiver H340, but i think its the same for the other players

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 :)

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.

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 tryFS#7639: Shutdown item in main menu.
Thanks.

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.

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?

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.

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!