Rockbox.org home
Downloads
Release release
Dev builds dev builds
Extras extras
themes themes
Documentation
Manual manual
Wiki wiki
Device Status device status
Support
Forums forums
Mailing lists mailing lists
IRC IRC
Development
Bugs bugs
Patches patches
Dev Guide dev guide
Search



Donate

Rockbox Technical Forums


Login with username, password and session length
Home Help Search Staff List Login Register
News:

Welcome to the Rockbox Technical Forums!

+  Rockbox Technical Forums
|-+  Rockbox Development
| |-+  New Ports
| | |-+  Phillips GoGear HDD6330
« previous next »
  • Print
Pages: 1 [2] 3

Author Topic: Phillips GoGear HDD6330  (Read 32836 times)

Offline saratoga

  • Developer
  • Member
  • *
  • Posts: 8968
Re: Phillips GoGear HDD6330
« Reply #15 on: June 13, 2010, 03:57:47 AM »
Quote from: scallopedllama on June 13, 2010, 03:39:54 AM
I would love to help with the development effort but I don't really know where to start.

Well figuring out why playback doesn't work for you, but evidently did for whoever wrote the wiki page would be a good start. 
Logged

Offline scallopedllama

  • Member
  • *
  • Posts: 11
Re: Phillips GoGear HDD6330
« Reply #16 on: June 14, 2010, 05:33:00 AM »
I haven't figured out why audio is broken, but I did manage to fix the buttons: http://www.rockbox.org/tracker/task/11401
Logged

Offline b0hoon

  • Developer
  • Member
  • *
  • Posts: 71
Re: Phillips GoGear HDD6330
« Reply #17 on: June 14, 2010, 08:16:02 AM »
Quote from: scallopedllama
I haven't figured out why audio is broken, but I did manage to fix the buttons: http://www.rockbox.org/tracker/task/11401

Playback doesn't work because there is no correct define for this target in the wmcodec-pp.c file (i2c audio address should be 0x1b). Additionally there must be some gpio pin (i haven't found it yet) which enables "full sound" (like in the PB vibe or iriver h10).

Your solution is correct, however i have a full touchpad driver in my local svn for this target, including buttonlights and it need's some additional changes for the mep driver. I'll try to commit this code after my vacation.
Logged

Offline scallopedllama

  • Member
  • *
  • Posts: 11
Re: Phillips GoGear HDD6330
« Reply #18 on: June 14, 2010, 09:36:32 AM »
Oh well, it was an interesting learning experience and it got me a little bit acquainted with the code..

Also, You Rock. I know have a usable audio player.
Logged

Offline b0hoon

  • Developer
  • Member
  • *
  • Posts: 71
Re: Phillips GoGear HDD6330
« Reply #19 on: June 14, 2010, 10:49:56 AM »
Quote from: scallopedllama
I know have a usable audio player.

So.. did you add the define and now have a sound? :) It's working for me and if it's working fine for you too, i'll fix it later (i don't have an access to my build environment atm). Is the sound crunchy and quiet (for me it is) or you have a full sound like in the OF? Which exactly model do you have: HDD6330 (with radio) or HDD6320 (without)?
Logged

Offline scallopedllama

  • Member
  • *
  • Posts: 11
Re: Phillips GoGear HDD6330
« Reply #20 on: June 14, 2010, 10:29:19 PM »
Define added and audio now works. Sound quality is certainly not on par with the OF.. I guess crunchy and quiet describes it ok.

Specifically I have an HDD6330 with the radio. I haven't tested everything yet though.
Logged

Offline scallopedllama

  • Member
  • *
  • Posts: 11
Re: Phillips GoGear HDD6330
« Reply #21 on: June 18, 2010, 10:56:41 PM »
I'm pretty sure I figured the rest of this stuff out.
With the added define from the previous post, sound was working, but it wasn't very loud and the radio did not work. To fix those problems, the same kind of fix is needed as was needed to get audio working, adding a "|| defined(PHILIPS_HDD6330)" to a line.

For audio: /firmware/target/arm/wmcodec-pp.c at line 36
For the tuner: /firmware/drivers/tuner/tea5767.c at line 31
For the preamp: /firmware/drivers/audio/wm8731.c at line 204

With those three defines added, audio sound pretty good out of my hdd6330 and the radio works. The patch I submitted isn't very good, and the keymap is still pretty bad though. I have hopes for the code that h0hoon eventually commits.

I made diffs for these changes and put it in the tracker: http://www.rockbox.org/tracker/task/11417
Logged

Offline funman

  • Developer
  • Member
  • *
  • Posts: 645
Re: Phillips GoGear HDD6330
« Reply #22 on: June 18, 2010, 11:53:29 PM »
I committed the patch, thanks

For the next patch please run svn diff from the root of your checkout so the full path to patched files is included in the patches
Logged
a wise man said: "a wise man said"

Offline b0hoon

  • Developer
  • Member
  • *
  • Posts: 71
Re: Phillips GoGear HDD6330
« Reply #23 on: June 19, 2010, 06:04:56 AM »
scallopedllama: thanks for figuring out the rest of the audio problems!

Additionally i would like to change the behavior of the vertical scrollstrip, as it should be - sliding a finger down or up should scroll down or up the lists and menus (like in the OF), without the button_select in the middle (unnatural), which should be removed and remapped. The scrollstrip was designed to act like that IMO (it works great like that in the PB Vibe).
Logged

Offline scallopedllama

  • Member
  • *
  • Posts: 11
Re: Phillips GoGear HDD6330
« Reply #24 on: June 21, 2010, 02:57:23 AM »
I agree with making the scrollstrip a scrollstrip and I would love to help out with that but I'd really like to see your code for properly using the scroll strip before starting. Can you get it committed or provide a patch?

EDIT: Working off of my scrollstrip enabled code, I basically took the code used in the pb vibe for scrolling and stuck it in here. It works, but not very well. Maybe because my original scrollstrip enabled code wasn't very good.

On an unrelated note, I noticed that the configure script indicated that the hdd6330 only has 32M of ram while the wiki page says it rocks 64M.

I attached a patch that will apply both of these changes from the latest trunk. It's not very good, but it's something..

As for other work to be done, the keymap still needs a lot of work... unless you already that. I also noticed that I can't boot into the OF. It gets to the line "hdd1630 key used" and freezes.
* scroll_start.txt (9.34 kB - downloaded 208 times.)
« Last Edit: June 21, 2010, 05:36:29 AM by scallopedllama »
Logged

Offline b0hoon

  • Developer
  • Member
  • *
  • Posts: 71
Re: Phillips GoGear HDD6330
« Reply #25 on: June 25, 2010, 04:17:31 AM »
Quote from: scallopedllama
I agree with making the scrollstrip a scrollstrip and I would love to help out with that but I'd really like to see your code for properly using the scroll strip before starting. Can you get it committed or provide a patch?

EDIT: Working off of my scrollstrip enabled code, I basically took the code used in the pb vibe for scrolling and stuck it in here. It works, but not very well. Maybe because my original scrollstrip enabled code wasn't very good.

I'm surprised it doesn't work  :-\, basically it's the same code as i want to commit, except i have a "btn = BUTTON_NONE" after queue_post() to prevent queuing the button_up or down twice. I must test it and i will commit my touchpad code soon, please be patient for a while yet.  ;)

Quote from: scallopedllama
On an unrelated note, I noticed that the configure script indicated that the hdd6330 only has 32M of ram while the wiki page says it rocks 64M.

Should be 64MB, fixed in svn.

Quote from: scallopedllama
I also noticed that I can't boot into the OF. It gets to the line "hdd1630 key used" and freezes.

Same for me but right now i don't have a clue what's wrong with it. TEA key is correct cause it's the same for hdd1630 and hdd6330. I'll try to investigate it later.
Logged

Offline scallopedllama

  • Member
  • *
  • Posts: 11
Re: Phillips GoGear HDD6330
« Reply #26 on: June 28, 2010, 05:41:51 AM »
Quote from: b0hoon on June 25, 2010, 04:17:31 AM
I'm surprised it doesn't work  :-\
I didn't say It doesn't work. I just don't feel it works well. I added your line and it feels a bit better though. It just doesn't feel as smooth as the scrolling is in OF.

Have you done any work on the keymaps for this player? I'd be happy to help with that a bit if you haven't.
Logged

Offline lowlight

  • Developer
  • Member
  • *
  • Posts: 194
Re: Phillips GoGear HDD6330
« Reply #27 on: June 30, 2010, 11:06:31 PM »
Good to see someone has picked up this port. Unfortunately, real life got the better of me and I was never able to finish.

Odd that I missed some defines as I don't recall any audio problems from my build; however, I remember that at the time I was working from a months old svn tree, so might have miss something.

Have you actually checked that 64MB works?  I know that the documentation says 64, but when I tried before it didn't work. Try loading a playlist so that it will buffer more than 32MB, like a bunch of flac files.

For the OF not booting, try removing the "magic" from firmware/target/arm/crt0-pp-bl.S. If that doesn't work, maybe we need different "magic". I can't remember if I tried booting the OF before, or if I just assumed it was the same as the hdd130.

Can't wait to try your scrollpad patch, I've always wanted to have scrolling support like the OF. Can you still "tap" up/down? Is the code generic enough incorporate in the Synaptics mep driver? There are several players with a Synaptics scrollpad (Philips hdd1630/hdd6330/sa9200, Olympus m:robe 100, and, of course, the PB Vibe).

If one of you has a 6320, perhaps you could look at dynamically checking for the radio and disabling it so it doesn't show in the menu.

Anyway, good work!!
Logged

Offline b0hoon

  • Developer
  • Member
  • *
  • Posts: 71
Re: Phillips GoGear HDD6330
« Reply #28 on: July 04, 2010, 10:59:26 AM »
Quote from: scallopedllama on June 28, 2010, 05:41:51 AM
I didn't say It doesn't work. I just don't feel it works well. I added your line and it feels a bit better though. It just doesn't feel as smooth as the scrolling is in OF.

Yes. It didn't work properly because of vertical scrollstrip additional packet data, anyway modified touchpad code is in SVN, please try it. I haven't apply your changes for the keymaps but i will do it ASAP.

Quote from: scallopedllama on June 28, 2010, 05:41:51 AM
Have you done any work on the keymaps for this player? I'd be happy to help with that a bit if you haven't.

No i haven't and will be happy to get some help from you especially with the plugin keymaps, but first the main keymaps must be reworked somehow.

Quote from: lowlight on June 30, 2010, 11:06:31 PM
Good to see someone has picked up this port. Unfortunately, real life got the better of me and I was never able to finish.

Odd that I missed some defines as I don't recall any audio problems from my build; however, I remember that at the time I was working from a months old svn tree, so might have miss something.

Have you actually checked that 64MB works?  I know that the documentation says 64, but when I tried before it didn't work. Try loading a playlist so that it will buffer more than 32MB, like a bunch of flac files.

Hi lowlight.

Good to know about this memory problem, i will check it of course.

Quote from: lowlight on June 30, 2010, 11:06:31 PM
For the OF not booting, try removing the "magic" from firmware/target/arm/crt0-pp-bl.S. If that doesn't work, maybe we need different "magic". I can't remember if I tried booting the OF before, or if I just assumed it was the same as the hdd130.

Another good hint, i will check it too.

Quote from: lowlight on June 30, 2010, 11:06:31 PM
Can't wait to try your scrollpad patch, I've always wanted to have scrolling support like the OF. Can you still "tap" up/down? Is the code generic enough incorporate in the Synaptics mep driver? There are several players with a Synaptics scrollpad (Philips hdd1630/hdd6330/sa9200, Olympus m:robe 100, and, of course, the PB Vibe).

You can try it now :) You can't tap up or down now but it can be implemented on a tap packets (0b,02,00,00) eventually. The code is rather generic but in fact every touchpad is different a bit and needs some special caution in a case of implementing it. HDD6330 is the best example of it (additional packet must be received from the vertical scrollstrip, the same like after the buttons).

Quote from: lowlight on June 30, 2010, 11:06:31 PM
If one of you has a 6320, perhaps you could look at dynamically checking for the radio and disabling it so it doesn't show in the menu.

Anyway, good work!!

Yes, i thought about that, it should be easy, especially when the method is described in the service manual IIRC, which i put on the Wiki.
Logged

Offline LambdaCalculus

  • Member
  • *
  • Posts: 2257
  • Dreaming of Turing Machines...
    • The Nostalgia Roadtrip
Re: Phillips GoGear HDD6330
« Reply #29 on: October 18, 2010, 03:15:56 PM »
Seeing all of the progress on the HDD6330 port makes me want to get another unit for myself. I gave mine to lowlight.

If I find another unit, I'll come back in full swing for testing and contributions again.

(EDIT) Would anyone say that the port is far along enough to be good for everyday use? If so, I'm thinking of adding in support for installing builds via Rockbox Utility.
« Last Edit: October 18, 2010, 03:50:07 PM by LambdaCalculus379 »
Logged
Former Rockbox dev. Rising from the ashes...

Players: iPod Video /w 128GB SSD mod, H320 /w 128GB SSD mod

  • Print
Pages: 1 [2] 3
« previous next »
+  Rockbox Technical Forums
|-+  Rockbox Development
| |-+  New Ports
| | |-+  Phillips GoGear HDD6330
 

  • SMF 2.0.17 | SMF © 2019, Simple Machines
  • Rockbox Privacy Policy
  • XHTML
  • RSS
  • WAP2

Page created in 0.107 seconds with 20 queries.