Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Drivers
  • Assigned To
    Linus Nielsen Feltzing
  • 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 Linus Nielsen Feltzing - 2007-01-18
Last edited by Jens Arnold - 2007-05-23

FS#6549 - Work-in-progress support for iPod 5.5G

Here is the first patch for the ATA driver and the disk partition code for the iPod 5.5G 80Gbyte version. The ATA driver can find out the logical and physical sector size, and disk_init can find the partition correctly. It can not yet mount the filesystem.

Please review and see if you can find any flaws in the code before I commit it.

It should be compatible with all other platforms. I have tried it on iPod 5.5G 80Gb and iPod 5G 60Gb. I would like it tested on as many platforms as possible.

When this is reviewed and approved, I will move on with the (much more difficult) FAT driver.

Closed by  Jens Arnold
2007-05-23 18:00
Reason for closing:  Accepted
Additional comments about closing:  

Further updated version commited to SVN.

Admin
Linus Nielsen Feltzing commented on 2007-01-18 23:09

Here’s a new version which handles the missing udelay() on non-portalplayer platforms.

Paul Collins commented on 2007-01-20 08:49

I built rockbox with ipod55g-2.patch applied for my H120 and everything seems to work fine.

Admin
Linus Nielsen Feltzing commented on 2007-01-20 09:57

Thanks, Paul!

Chris commented on 2007-01-23 20:40

FAT driver - does that mean DMA or is that something else

Admin
Linus Nielsen Feltzing commented on 2007-01-23 21:59

FAT == File Allocation Table. It is the file system.

Chris commented on 2007-01-24 06:40

So, the 5.5Gs aren’t using FAT at the moment?

Chris commented on 2007-01-24 07:15

Hmm no my above comment didn’t make sense xD. Nevermind, got it patched, compiled for 5.5G 30GB Video. What am I looking for, or what should I be testing?

Admin
Linus Nielsen Feltzing commented on 2007-01-24 07:23

Just use it and see if it still works as before. The point is to make sure i didn’t break anything with this change in the ATA driver.

Chris commented on 2007-01-24 07:26

Seems to be okay

Chris commented on 2007-01-24 20:37

I don’t know if this is related, but every now and then after adding music and updating the database, I’ll play maybe 20 minutes of songs, and when I go to play the next one it freezes - not the whole of rockbox, but just the playback stops. If I try to play another song, sometimes it says file not found, sometimes playback just freezes again. To fix this, I have to restart the iPod. Just noting this because it happened again last night ><

Admin
Linus Nielsen Feltzing commented on 2007-01-25 07:42

I don’t think it’s related to my ATA fixes.

Chris commented on 2007-01-27 03:26

Well after testing it for a few days on my 5.5G 30GB iPod it appears to be working without problems. Only hiccup I ran into was when the bootloader booted and then froze when it said Rockbox Loaded (final line) - this only happened once, and I don’t know if it is related to your ATA fixes. Hopefully will be testing on a 2G and 4G Color iPod soon

Jens Arnold commented on 2007-05-21 00:16

Here is a working implementation for those eager to test. You need to build and install both rockbox and a custom bootloader with this patch applied.

The patch is already confirmed working and stress tested on iPod Video G5.5 80GB. It’s also confirmed to not break any archos, any coldfire target, or the ipod mini. Most of the code is simply not used on any other target than iPod Video.

The patch needs to be tested on iPod Video other than 80GB (both G5 and G5.5), in order to see whether the code properly adapts to those more standard hard drives (no large physical sectors).

Jens Arnold commented on 2007-05-21 06:14

Updated patch:

- caching layer in ata.c grouped together
- multivolume builds for iPod video should work again
- invalidate the cache in ata_init(). Cache content cannot be relied upon e.g. after returning from USB (non-issue on ipods)

Lini Ini Mini commented on 2007-05-22 06:32

applied the patch to a 5G 30GB ipod. No problems so far playing mostly mp3s (with cuesheets). I will install the same bootloader/firmware on the 5.5G 80GB later today.

Dieter commented on 2007-05-22 06:42

Installed a patched version of firmware and bootloader on the 5.5G 80GB and it works like a charm. Thanks a lot for this great patch!

Douglas Valentine commented on 2007-05-22 11:09

I have installed the patched version of the bootloader + rockbox on my 5.5G 80GB iPod and it works grate.

I will also test in on my iPod nano tonight when I get home from work.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing