• Status Closed
  • Percent Complete
  • Task Type Patches
  • Category Drivers
  • Assigned To No-one
  • Operating System PortalPlayer-based
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by dreamlayers - 2009-02-17
Last edited by fg - 2009-02-17

FS#9923 - PP5022-specific USB init which may fix signal quality issues

USB high speed mode never worked for me. When I add this initialization, it always works. A USB controller reset undoes this initialization, so I put it at the end of usb_drv_reset() in usb-drv-arc.c.

* I recommend making sure you have backup copies of files on your iPod before using Rockbox USB mass storage mode. (I have not encountered any problems; I just think it’s the responsible thing to do.)
* USB charging on iPods may not work or may be limited to 100 mA. See  FS#8802 . Your battery may discharge while the disk is being used. This may empty it, cause a shutdown in the middle of disk activity and result in filesystem problems. Note that the status line displays battery status even in USB mode.

Closed by  fg
2009-02-17 22:25
Reason for closing:  Accepted
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Thanks a lot! This is a huge step forward

MikeS commented on 2009-02-17 20:32

Nice. I take it this may help all PP502x players?

EDIT: Remove question the author explained about already (moving too fast today).

The OF checks PP_VER1 for the ‘2’. I have no idea if it would help on other PP502x chips. It’s easy to edit the “#if CONFIG_CPU == PP5022” line and try it. (Note that the chips in the 5G iPod and 1G Nano are labelled as PP5021C-TDF, but they identify as PP5022C and Rockbox sets CONFIG_CPU to PP5022.)

BTW. Should I check PP_VER1 at runtime instead of CONFIG_CPU at compile time?

fg commented on 2009-02-17 22:12

Compile time is fine (everything else we do for different PP chips is compile time)

This seems to work on c200, e200, and ipod video at least.

fg commented on 2009-02-17 22:18

In case people wondered, this doesn’t solve anything on PP5020


Available keyboard shortcuts


Task Details

Task Editing