Rockbox

Forgot to mention - this patch takes effect only when USB is connected while RB is on. If connecting while device is off, OF still boots as usual.

the usb.c and usb.h changes need to be fixed. we have a rule that the firmware level doesnt call any apps level stuff (including global_settings). look at how the backlight settings work for a good way to do it.

done. hopefully this is how you meant it should be fixed :)

   usbconfig.patch (4.5 KiB)

I don't mind the functionality, but I strongly object to some of the terminology and naming used.

"usb charging mode" does not at all imply what the setting does, and "usb_charging_by_default" seems to imply that if it's set to false, the device will not charge.

These settings and variable names seem to imply that rockbox will not charge while connected in MSC mode, which is plainly not the case. Also, some players (such as the Ondio) can run on USB power but can not charge from USB, so for these the entire "charge" wording is even more wrong.

The patch also misses the necessary changes to the manual.

sure, what terminology would you prefer?

How do I change the manual..? there was no reference to doing so in RB's development guide :(

I'd use e.g. usb_default_to_power_only (or similar) instead of usb_charging_by_default (power instead of charging, and "power only" doesn't imply that the alternative doesn't do power).

For the setting string, it's a bit more difficult to fit in a yes/no thing. I'd really prefer something like "USB default connection mode: Power only/Full connection" (or "MSC", or something like that). I don't know if that's easy to do in the settings code.

For the manual, I guess http://www.rockbox.org/wiki/ManualHowto might be useful. If you can't manage it, feel free to join us on irc to ask for help, or at least provide the text the manual should have here

alright, to avoid any misunderstandings with different platforms I think I'll use something along the lines of 'use alternative USB power mode' or 'reverse USB power operation' - since this is the only way to truly describe what that option does - switches the way rockbox behaves when connecting exteral power and pressing the 'select' button.

I'll get that fixed, update the manual and repost the patch.

Ok, made the requested code/interface changes, tested, and updated the manual accordingly.

   usbconfig.patch (6.1 KiB)

Storage on USB attach: (When button is bressed / When no button is pressed)
maybe?

Make it about the presentation of storage, rather than about power/charging.

Well I think this is becoming somewhat redundant… I honestly don't mind how this option is called, I just think it has to be user controlled.

If you guys wish, we could have a vote among all rockbox devs to decide the best phrasing, but I think that right now the interface describes exactly what the option does (regardless of actual functionality).
This menu option causes rockbox to switch the two possible modes of handling USB connections, that's it. and that's why it's called the way it is.

Moreover, since the complete description is found in the manual, I don't see the point in contemplating other possible ways to phrase this in the GUI… the current phrasing is close enough, and anyone interested should read the manual to fully understand it.