- Status Closed
- Percent Complete
- Task Type Patches
- Category Rbutil
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Rbutil git
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#11732 - Rockbox Utility: Add support for Philips GoGear HDD16x0 and HDD63x0 series
This patch adds support to install Rockbox on the Philips GoGear HDD16x0 and HDD63x0 series. As the bootloader install method is similar to other PP-based targets (MI4-based), simply updating rbutil.ini should be sufficient.
I may commit this soon if it tests clean.
Closed by RMenes379
2011-01-06 18:29
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
2011-01-06 18:29
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
Committed.
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
I can't try the installation itself, but two comments:
- the section [0471014b] is unnecessary. The sections named after the USB IDs are only used for detecting incompatible players (or if a player is in an incompatible mode that can be detected via a different USB ID, like the Nano 2g in DFU mode)
- you're writing the USB IDs as hex code with upper case letters, while all other entries use lower case letters. ini files are case sensitive, and I'm somewhat sure the code uses lower case representations (this doesn't make a difference for the usbid value since that is converted into a number, but it does make a difference for the USB ID named sections). IMO it would be best to simply stick to lower case representations – IMO making the code case insensitive is not worth the additional code for this.
Thanks for the feedback. I'll make those fixes and try it again.
New version with the fixes made.