Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#9969 - Proof of concept USB connection status indicator

Attached to Project: Rockbox
Opened by Frank Gevaerts (fg) - Saturday, 28 February 2009, 15:47 GMT+2
Last edited by Dominik Riebeling (bluebrother) - Sunday, 05 June 2011, 11:36 GMT+2
Task Type Patches
Category Battery/Charging
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Version 3.1
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

This adds proof of concept status indication to the USB screen. It's ugly and probably won't compile for non-colour targets, but it shows how it can be done so someone with artistic and UI skills can do it properly.
Needs r20138
   usb_indicators.patch (1.6 KiB) 
 apps/screens.c |   36 +++++++++++++++++++++++++++++++++++-
 1 file changed, 35 insertions(+), 1 deletion(-)

This task depends upon

Closed by  Dominik Riebeling (bluebrother)
Sunday, 05 June 2011, 11:36 GMT+2
Reason for closing:  Out of Date
Additional comments about closing:  I'd really like to see that but it's not had any activity since 2 years.
Comment by Frank Gevaerts (fg) - Saturday, 28 February 2009, 17:26 GMT+2
This one also shows transfer speeds. It needs r20140. Same caveast apply: Not adjusted to screen size, may only compile for colour targets, looks ugly.
   usb_indicators-2.patch (3.7 KiB) 
 apps/screens.c |   93 ++++++++++++++++++++++++++++++++++++++++++++++++++++-----
 1 file changed, 85 insertions(+), 8 deletions(-)

Comment by Frank Gevaerts (fg) - Saturday, 28 February 2009, 17:44 GMT+2
Updated. For some reason (probably latency things?), this causes linux to reset the bus every now and then unfortunately
   usb_indicators-3.patch (3.7 KiB) 
 apps/screens.c |   96 ++++++++++++++++++++++++++++++++++++++++++++++++++++-----
 1 file changed, 88 insertions(+), 8 deletions(-)

Comment by Frank Gevaerts (fg) - Saturday, 28 February 2009, 20:41 GMT+2
This one (together with r20142) seems more stable. Still not perfect though
   usb_indicators-4.patch (4.1 KiB) 
 apps/screens.c |  103 ++++++++++++++++++++++++++++++++++++++++++++++++++++-----
 1 file changed, 95 insertions(+), 8 deletions(-)

Comment by Dominik Riebeling (bluebrother) - Saturday, 28 February 2009, 21:39 GMT+2
Quite nice, though it seems to not reset the default color correctly -- after a usb connection the foreground color is set to red.
Edit: color is set to red, not read :)
Comment by Tom (frameshift) - Saturday, 14 March 2009, 03:08 GMT+2
seems to not work with the latest revision, patching errors. r20322.
Comment by Maurus Cuelenaere (mcuelenaere) - Sunday, 19 April 2009, 18:03 GMT+2
Sync with r20737
edit:
don't forget to enable l.530 in usb_storage.c
   usb_indicators-5.diff (3.9 KiB) 
 apps/screens.c |   76 ++++++++++++++++++++++++++++++++++++++++++++++++++-------
 1 file changed, 67 insertions(+), 9 deletions(-)

Loading...