iRiver ihp120 failed CF install

[petur]

final answer: we suffer the same problem as ipod nano:

When perform_soft_reset() is done, the next read seems to fail - in my case all reads seem to fail.

As for the shutdown problem: I never saw that, but I see that shutdown occasionally takes longer to perform, none of my fixes ever corrected that. What did you do to prevent that?

[magmaniac]

I ran battery_bench on a CF'ed H120 after applying FrankOtto's patch as well as on a standard (HD) H120. Both have the stock battery.

The CF-H120 lasted 16:18:05.
The HD-H120 lasted 14:05:52.

Note that this was not done on the same battery, just the same type of battery.

I ran battery benchmarks too (constant playing of MP3 320kb/s, stock battery, 3years old), and the runtime that battery_bench.txt reports is 13:51 with the HD and 18:06 with the CF... but scrap those numbers. In the HD case I'm pretty sure that the last lines of the battery_bench.txt were lost due to forced shutdown, while in the CF case I shut the device down voluntarily (it had been reporting "battery level 0%" for some time). Also, in the HD case I had used USB mode briefly before the test, which puts the HD benchmark at a slight disadvantage.

But! I can shift and scale those numbers until the curves coincide
And this tells me that the mentioned disadvantage of the HD amounts to roughly 20 minutes, and that the battery drains approx. 14% slower with CF instead of HD. See attached PDF.

[magmaniac]

As for the shutdown problem: I never saw that, but I see that shutdown occasionally takes longer to perform, none of my fixes ever corrected that. What did you do to prevent that?

petur, I always get the shutdown problem when setting "sleeping=true" is disabled.

shutdown_hw() contains this loop:

Code: [Select]

while(ata_disk_is_active())
   sleep(HZ/10);and ata_disk_is_active() simply returns !sleeping. The only place where sleeping=true gets set is in ata_perform_sleep(). If you disable this entirely, then the loop in shutdown_hw() cannot finish. Eventually another thread forces the unclean shutdown.

[petur]

interesting....so only disabling soft_reset should fix all this... to be tested

[petur]

hrmpf... shutdown was ok, and then one of my last tests there was a freeze
Hard to reproduce, I tried a lot of stuff afterwards without freezing.

NOTE: cleaned up my code and removed some stuff that should have been removed before stability testing. So far no freezing and normal shutdowns....

[magmaniac]

Using the latest ideas of petur, I re-enabled ata_perform_sleep() but disabled perform_soft_reset() as on iPod nano. No freezes in the last two days (normal usage for several hours: playing, stopping, idling, skipping, ...). Looks good.

[Davide-NYC]

I find the ata.c_01-ata80+freeze.patch generally unstable. I get a lot of freeze ups or severe slow downs. This occurs too frewuently to start doing detailed bug reports. It's pretty broken. (sorry)  

[petur]

Really?
Does the patch disable perform_soft_reset() in ata.c ?
(don't know if there were multiple versions of the patch - I just looked at the one currently in the wiki)

[magmaniac]

Does the patch disable perform_soft_reset() in ata.c ?
(don't know if there were multiple versions of the patch - I just looked at the one currently in the wiki)

petur: No, there's only one version of this patch. It is rather small, it just skips the failing powermanagement set_features call, and activates the iPod nano workaround in perform_soft_reset unconditionally.

Davide-NYC: I'm sorry to hear that it's unstable for you, I didn't have a single freeze with this version yet. Just for completeness: Did you do your testing with or without the additonal RTC mod? (but I doubt that these mods interact)

Maybe we have to face the fact that different cards show different issues...

[scharkalvin]

People, please post your results on the wiki.
State what patches you included, which adapter card, which CF card, which player and your results.

[petur]

note: current SVN should now be compatible with CF

[ShinyFalcon]

I need some clarification, as I'm not having any luck searching through the forums, but does this mean that the ATA -80, etc issues have been solved? I'm interested in as to what was causing the problem. Will any CF cards work on the H120 now?

[petur]

yes, but you still need a patched bootloader (see the CFMod wiki page)

afaik, the original firmware does not like the CF, however...

[crv1]

Hello friends my hdd died so I bought PQI x150 CF card but it does not work with Rockbox error pops up ata error -80 can you give me a bearing on the improved soft patch ata.c?