- 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
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.
WARNINGS:
* 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.
PP5022_init_after_USB_reset.p...
(1.2 KiB)
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
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
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?
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.
In case people wondered, this doesn’t solve anything on PP5020