From: Björn Stenberg <bjorn_at_haxx.se>
Date: Tue, 5 Aug 2003 15:30:07 +0200

Mike Holden wrote:
> > I haven't done this because I don't like the idea of breaking the ATA
> > specification only to lessen the impact of our own bugs.
>
> Fair enough. Does the spec state we must wait 10 seconds? If so, then I'm
> sure we must comply with that.

I couldn't find it with a quick glance, but I recall reading it in the spec.

> At the very least, can we get rid of that awful perform_soft_reset()
> function that takes 90 seconds to complete, as this makes the impact of
> the problem far more severe than it needs to be. As I have stated earlier,
> it loops through a 10 second wait 9 times, before returning an error to
> the mpeg layer. Rockbox then skips to the next track and again waits 90
> seconds and so on to the end of the playlist/directory.

I doubt this will help much either. We can cut down the wait time to 31 seconds[1] per reset call, but is still means a very long time hung when playing a moderately-sized playlist. Far longer than most users are willing to wait.

[1] This number is specified in section 9.2 of the ATA spec: "All actions required in this state shall be completed within 31 s.", page 234 of v5.

-- 
Björn

Received on 2003-08-05