Rockbox

Tasklist

FS#11732 - Rockbox Utility: Add support for Philips GoGear HDD16x0 and HDD63x0 series

Attached to Project: Rockbox
Opened by Robert Menes (RMenes379) - Sunday, 07 November 2010, 03:26 GMT
Last edited by Robert Menes (RMenes379) - Thursday, 06 January 2011, 18:29 GMT
Task Type Patches
Category Rbutil
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Rbutil SVN
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

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.
This task depends upon

Closed by  Robert Menes (RMenes379)
Thursday, 06 January 2011, 18:29 GMT
Reason for closing:  Accepted
Additional comments about closing:  Committed.
Comment by Dominik Riebeling (bluebrother) - Sunday, 07 November 2010, 21:12 GMT
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.
Comment by Robert Menes (RMenes379) - Monday, 08 November 2010, 10:32 GMT
Thanks for the feedback. I'll make those fixes and try it again.
Comment by Robert Menes (RMenes379) - Monday, 08 November 2010, 12:45 GMT
New version with the fixes made.

Loading...