Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Operating System/Drivers
  • Assigned To No-one
  • Operating System iPod 5G
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Petey Leinonen - 2006-05-31

FS#5472 - ipod 64MB memory

This patch remaps the full 64MB memory available on the 60GB ipod video. This will make all 64MB available to rockbox.

Closed by  Daniel Stenberg
2007-05-19 14:35
Reason for closing:  Fixed
Additional comments about closing:  

version three used, slightly edited and now committed

Dave Chapman commented on 2006-05-31 09:07

Nice work on figuring out the change needed. Can you update your patch to also include the required change to tools/configure which allows the user to specify the amount of RAM available?

But IMO we don’t want to apply this patch to CVS and add yet another build to the system - the desirable solution is a single 5g build that will detect and use all available RAM.

The issue with that is that currently the codecs and plugins are compiled and linked in order to be loaded to fixed memory locations at the end of RAM. This approach of course fails when the end of RAM is not known at compile-time.

A solution would be to reorganise the memory allocation so that the codecs and plugins are located at the start of RAM.

But getting Rockbox working correctly with 64MB RAM in the first place is a good start.

Petey Leinonen commented on 2006-06-03 01:41

linuxstb, done. File attached is the modified patch.

As there are no plans to apply it to CVS I haven’t spent too much time modifying configure. Now it has another build option (22) which is the ipodvideo 64MB build. If done properly I would have written a function to ask the user which memory model they want.

Dave Chapman commented on 2006-06-03 08:28

As you say, an extra build option is enough. It’s just to help those people who want to use your patch to compile their own 64MB builds.

Thomas Paul Diffenbach commented on 2006-06-08 01:19

I installed the second version of the 64 MB patch: http://www.rockbox.org/tracker/?getfile=11753 configured build 22, and built.

However Menu|Info|Rockbox Info still shows a < 32 MB buffer (specifically, 28.843MB).

Is this a bug in the displayed buffer size, a bug with the patch, or my error?

Thanks.

Thomas Paul Diffenbach commented on 2006-06-08 03:02

New build directory fixes this for me. Please ignore previous comment.

Anders Hunstad commented on 2006-08-02 22:38

I think there is something wrong with this patch, it does not work well with the resent change of compile optimization of libmad.
Playback of mp3 files fails after some time of playing (usually minutes of play but I have observed the error as late as an hour of play).

Petey Leinonen commented on 2006-08-03 11:03

Anders,

do you have *any* other patches applied as well?

I have a build that includes the change to libmad optimization and have played mp3s heavily over the previous week without any problems.

Petey Leinonen commented on 2006-08-03 12:34

I’ve updated the patch to adjust the build option to 24 as a couple of new builds have been added some time after the 24 July 2006.

Anders, try again with this patch (though I don’t think this will fix your problem). Also can you please look at Info/Rockbox Info and tell me what your buffer is set to.

Ryan Sawhill commented on 2006-08-03 12:41

Yeah–I’ve compiled a new build for myself since the libmad commit with not problems….

Admin
Daniel Stenberg commented on 2006-08-03 16:41

I would rather see this using the already present configure prompt asking for memory size after you select the target device. We already have numerous (Archos) players with selectable memory size to build for and IMHO, it would make sense to use the same approach for this build - if it is to be committed.

Michael DiFebbo commented on 2006-08-03 19:03

Daniel, is something like this what you had in mind?

Michael DiFebbo commented on 2006-08-03 19:25

Actually, this is probably a much better way to do it:

Admin
Daniel Stenberg commented on 2006-08-03 20:08

Yeps, that’s exactly how I meant it!

Petey Leinonen commented on 2006-08-04 02:40

bagder,
just an explanation why I did it the way I did. As linuxstb and Llorean had already stated that they did NOT want to commit this, preferring to detect how much memory is available instead, I did the simplest change I could. I had planned to use the memory prompt, and may have changed the patch later if I was having to do many changes because of new platforms.

Nice change Michael.

Petey Leinonen commented on 2006-08-04 02:55

I’ve made a small change to the patch to add the relative path from the rockbox root directory for the changes to the configure script. This way we can run patch from the rockbox root without falling over.

Admin
Daniel Stenberg commented on 2006-08-04 07:14

Well, this way it is also more likely to survive further changes of configure

todd commented on 2006-09-09 04:56

I think this is broken now…. I used the select_ipod_memory_2.patch and it’s not letting me view jpg’s now. It says incompatible model or something like that… without the patch it works fine. It compiles without errors…

todd commented on 2006-09-10 20:49

Also you get codec errors when trying to play music. I tried looking at the code but I’m clueless. I downloaded a clean cvs and patched it with select_ipod_memory_2.patch and 30 gig compile works fine. Compile a 60 gig and I get those errors on my 60 gig ipod video.

Chris Banes commented on 2006-09-11 14:51

Resync, should be working as before now ;)

Petey Leinonen commented on 2006-09-18 03:18

Cheers senab,

I came to update the patch and noticed your changes.

Thanks.

Chuckles commented on 2006-10-08 03:29

I’m still getting the codec failure with the latest patch
using daily build20060829
also with album art, bmp resizing and scrolling margins patches

Chuckles commented on 2006-10-08 03:58

sorry, daily build above is 20061006 (was reading a patch date before :-) )

Petey Leinonen commented on 2006-10-15 02:26

Chuckles,

I had no problems (related to the memory patch) compiling and using a build from the 6th October. There may have been a change to the Rockbox source since this date that may have broken the patch. I’ll have a look in a couple of days and report back. I’m trying to sort out another issue at the moment (not related to this patch).

I just want to confirm that you are using a 60GB 5th gen ipod (video ipod)

Chuckles commented on 2006-10-15 02:42

Yep, i’m using a 60GB video ipod

I haven’t tried again with any recent daily releases, i’ll give it a shot with only the memory patch

Petey Leinonen commented on 2006-10-16 11:18

Just compiled and used this patch fine. Source is “20061014 01:33”, only patch is this one.

Chuckles commented on 2006-11-16 01:53

Ok i’ve tried again with fresh daily source

rockbox: daily-20061115
select_ipod_memory_2_(senab_sync).patch
scroll margins: 20061114
bmp_resize: v0.99.6-MM.20061102
album art: v5.2_20061114

patches applied in above order

and i still get the same codec failure

Chuckles commented on 2006-11-16 02:23

I just rebuilt rockbox: daily-20061115 with only select_ipod_memory_2_(senab_sync).patch and i still get the codec failure

(and i can confirm again that i’m using an ipod 60gb, the drive holds 60gb and there is a big embossed 60gb on the back)

Chuckles commented on 2006-11-16 04:39

Just to test that it wasn’t me compiling wrong i grabbed ace241’s unsupported build, it also throws a codec error when i try and play a song.

by the way i’m trying to play both ogg and mp3 files

also, i cleared the settings (using rockbox menu) and reset, still wont work

Chuckles commented on 2006-11-16 05:14

Ok, I started from scratch, wiped all rockbox related files off the ipod and reinstalled and it worked, something somewhere was hanging around between installs

Thankyou all for your ears and apologies for all the posts, (and for anyone with the same issue, do a clean out)

Petey Leinonen commented on 2006-11-17 09:51

Good to hear you got it going.

robin commented on 2007-03-04 19:05

This Patch Is Now Broken by the latest build has it now been commited?

David Hall commented on 2007-03-04 20:12

I believe this will fix it. (I’ve been wrong before.)

Robert Kukla commented on 2007-03-04 21:45

Add ipod video (60GB) as separate build target without extra memory size prompt.

Petey Leinonen commented on 2007-03-06 10:26

Thanks Soap and roolku,

I fixed the patch the other day, and was going to upload the latest changes now, but I see you guys beat me to it.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing