Phillips GoGear HDD6330

[saratoga]

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. 

[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

[b0hoon]

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.

[scallopedllama]

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.

[b0hoon]

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)?

[scallopedllama]

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.

[scallopedllama]

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

[funman]

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

[b0hoon]

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).

[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.

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.

[b0hoon]

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. 

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.

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.

[scallopedllama]

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.

[lowlight]

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!!

[b0hoon]

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.

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.

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.

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.

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).

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.

[LambdaCalculus]

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.