Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Utils
  • Assigned To No-one
  • Operating System Another
  • Severity Low
  • Priority Very Low
  • Reported Version Release 3.6
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by jpabloae - 2010-11-09
Last edited by bertrik - 2010-11-10

FS#11747 - tcctool: add support for Cowon J3

Today I have bricked my Cowon J3 player after a failed firmware upgrade. And I’ve been able to unbrick it using tcctool. So first of all thank you for creating such a useful tool!

In order to make it work with Cowon J3 I’ve applied the attached patch. So I’ve thought it would be nice to include it into the tool so that other Cowon J3 users can benefit from it too.

The credits for getting the right values for this player should go to this user:
http://iaudiophile.net/forums/showpost.php?p=296298&postcount=66

Thank you.

Closed by  bertrik
2010-11-10 18:56
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 as SVN r28551

Looks good and ready for commit to me.

Hm, not sure who to credit for this patch (”joecool”?). I don’t know if we accept essentially anonymous patches.

Well, I took the hex value from that forum post. I don’t know if he’s the original person who got the value though. Given that it’s a value “fixed” for this device, I’m not sure if any person can hold the copyright of that. I’d say that no. This is why I see it OK to add this patch, even if we didn’t know who’s the original person that extracted the hex value.

Just my 2 cents.

Admin
fg commented on 2010-11-10 18:20

In case this is a problem, I happened to see the magic sdcfg value for the j3 somewhere (it might even have been in a patch on flyspray!), and I did a cleanroom reimplementation of it (see attachment)

My version even has the advantage of being sorted correctly :)

   j3.patch (0.8 KiB)

hehe, good! Is the “0xb011” instead of “0xb057” correct?

Admin
fg commented on 2010-11-10 18:26

Ah, good catch, sorry. Fixed version (still by hand, no copy/paste here! :)

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing