[SOLVED] (iPod Mini 2g) ATA error: -82 on latest 20240612 build, not on 3.15

[kaaaas]

Power management and Adv power management are shown as enabled in disk info.
Does that still mean its working?

[speachy]

Power management and Adv power management are shown as enabled in disk info.
Does that still mean its working?

In this case, it means it's lying. 

[adriaanjelle]

I believe I've found (and fixed) the problem.

The root cause is that some CF cards lie about supporting a particular feature/command (Advanced Power Management) and throw an error when it's issued.

There is a special case in the ATA driver to catch and ignore this, but it was checking against the index of the driver's internal feature table.  When the table was altered as part of the iflash corruption fixes, the offset changed rendering the test incorrect.

...The fix is to check against the command ID instead of the table index.

This will be in tomorrow's (ie 20240615) daily build but if you can't wait that long you can grab the latest dev build (updated after every commit) from https://build.rockbox.org/ instead.

Great to see it's been resolved! Now I can continue with my various battery tests. I'll let you know how it goes! It seems to be working so far.