On 10.03.2009 01:05, Kevin O'Connor wrote: > On Mon, Mar 09, 2009 at 10:55:28AM -0700, Myles Watson wrote: > >> SeaBIOS won't work with SimNOW unless you make the error checking in >> ata.c less strict. >> >> This is the message you get: >> ata_transfer : no sectors left (status 00000008) >> >> I reported it to the SimNOW team since it seems like they're not >> following the spec, but it's been a long time and the bug is still >> there. To get past it: >> dprintf(6, "ata_transfer : no sectors left (status %02x)\n", status); >> + if (status != 0x8) >> return -7; >> > > Yeah, I've run into this too. It looks like the SimNOW CDROM doesn't > clear the DRQ bit - even though the spec definitely says it must be > cleared. > > I make the same change locally (just warn and not abort the transfer) > when I test with SimNOW. >
Maybe add a conditional message: "Working around SimNow bug" so users and coders know what's happening. Regards, Carl-Daniel -- http://www.hailfinger.org/ -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
