Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: Re[2]: cvs: firmware mpeg.c,1.206,1.207
From: Daniel Stenberg (daniel_at_haxx.se)
Date: 2003-03-03


On Mon, 3 Mar 2003, Uwe Freese wrote:

> > amount_to_read = MIN(mp3buflen - mp3buf_write, amount_to_read);
> > +#if MEM == 8
> > + amount_to_read = MIN(0x100000, amount_to_read);
> > +#endif
>
> As I understand it, 1,7 MB (with a 2 MB AJB) is read in one piece when
> filling the buffer the first time. With an 8 MB AJB, it is 7,7 MB. That
> means that bitswapping (or something else) is started too late (after
> reading these 7,7 MB) and data that's not bitswapped (?) is fed into the
> MAS.

That sounds like a bug. The mpeg thread should not feed anything to the MAS
until it is bitswapped.

Linus, you have any comment on this?

If this really is what you need, then why is this 8MB dependent (setting the
limit to 1MB)? Shouldn't this instead be made unconditionally even for 2MB
builds?

> I didn't write that mpeg.c and don't know how it exactly works! Sorry, but
> I thought it's better to have at least a working version (for 8MB) in CVS.
> (no changes when compiling as 2 MB version)

Possibly, we just don't like kludgy fixes in the code without very good
reasons.

-- 
 Daniel Stenberg -- http://rockbox.haxx.se/ -- http://daniel.haxx.se/



Page was last modified "Jan 10 2012" The Rockbox Crew
aaa